| *** dviroel|out is now known as dviroel | 00:05 | |
| *** dviroel is now known as dviroel|out | 00:20 | |
| OutBackDingo | jrosser: ok so 2 hosts can ping each other and every other host on that network, from bond0.1696 but cannot ping its own .1 gateway | 01:44 |
|---|---|---|
| OutBackDingo | which says everything on the switch is properly routed, except the gateway :) | 01:45 |
| jrosser | OutBackDingo: check the gw is configured on the switch next - which vlan is 1696? | 05:29 |
| OutBackDingo | @jrosser yeah, well then i was told its vlan4816... so i changed it, and now the host cant ping anything, so im thinking the switch isnt configured right | 05:30 |
| OutBackDingo | and im not cumulus savvy | 05:30 |
| OutBackDingo | plus its running ospf | 05:31 |
| jrosser | is there just the one switch? | 05:31 |
| OutBackDingo | give me a juniper and im happy | 05:31 |
| OutBackDingo | 2 leafs | 05:31 |
| jrosser | so some hosts in one leaf and some in another with L3 between the leafs? | 05:32 |
| jrosser | or L2 / MLAG as you’ve got bonds? | 05:33 |
| OutBackDingo | L2 | 05:33 |
| jrosser | so you’d only need a gateway in vlans you want to route further on | 05:34 |
| OutBackDingo | https://pastebin.com/VKWA2nhP | 05:34 |
| OutBackDingo | they appear to be there | 05:34 |
| jrosser | probably only the mgmt vlan….. but not storage or vxlan | 05:34 |
| OutBackDingo | no their all their | 05:34 |
| OutBackDingo | https://pastebin.com/1KRTT9Z2 | 05:36 |
| jrosser | well like I say you only need the gw if you actually want to route rather than just switch | 05:36 |
| jrosser | these vlans should be isolated from each other | 05:37 |
| jrosser | and unless they are in different vrf having a gateway will break that | 05:37 |
| OutBackDingo | i show 48.1 on 00:02 all the others are on 00:03 | 05:39 |
| OutBackDingo | mac wise | 05:39 |
| OutBackDingo | seems i need a cumulus guru | 05:40 |
| jrosser | indeed, I usually use Cisco and can do a bit of juniper but no cumulus | 05:41 |
| admin1 | OutBackDingo, where does your gatway live ? is it a real router, or in a switch with L3 function ? | 09:04 |
| admin1 | anyone using zun ? | 09:11 |
| noonedeadpunk | admin1: I think you can ask andrewbonney if they do | 10:15 |
| jrosser | admin1: we did a lot of work with zun but didnt deploy it to users in the end | 10:40 |
| OutBackDingo | admin1: switch | 11:04 |
| *** dviroel_ is now known as dviroel | 11:22 | |
| OutBackDingo | ip a | grep 48.1 | 11:33 |
| OutBackDingo | inet 10.16.48.1/32 metric 1024 scope global vlan1648-v0 | 11:33 |
| mgariepy | can we merge this to unblock test sync ? https://review.opendev.org/q/topic:workaround-setuptools-autodiscovery | 12:14 |
| mgariepy | OutBackDingo, i migth be wrong but i don't thing you need to set the mac for the interface: https://paste.openstack.org/show/bPMgRMw8l2oTLEjiNI5O/ | 12:23 |
| noonedeadpunk | I wonder if they are needed. As I can see some changes to land after just patching tests repo... Let me re-check some patches to see if they pass now | 12:24 |
| noonedeadpunk | Like https://review.opendev.org/q/topic:osa%252Fkeypairs | 12:24 |
| mgariepy | noonedeadpunk, not all the repo have setup.py | 12:25 |
| noonedeadpunk | um, and do we need it in others?:) | 12:25 |
| mgariepy | https://zuul.opendev.org/t/openstack/build/efb6e33f168f40fd9184f21a1dba1e10 | 12:26 |
| mgariepy | it's to fix that. | 12:26 |
| noonedeadpunk | maybe instead of pathching it we should just drop setup.py for these? | 12:26 |
| noonedeadpunk | as I dunno why we have that for roles tbh | 12:27 |
| noonedeadpunk | I do understand for integrated repo though | 12:28 |
| opendevreview | Marc Gariépy proposed openstack/openstack-ansible-os_tempest master: test cleanup instead of patching https://review.opendev.org/c/openstack/openstack-ansible-os_tempest/+/835969 | 12:31 |
| mgariepy | what is the purpose? | 12:32 |
| noonedeadpunk | we run python setup.py install somewhere iirc for integrated repo during bootstrap | 12:33 |
| noonedeadpunk | I believe for that to be available https://opendev.org/openstack/openstack-ansible/src/branch/master/setup.cfg#L27 | 12:34 |
| noonedeadpunk | and to avoid dealing with realtive path imports | 12:34 |
| mgariepy | yeah ok to install some python pkg that does make sense i guess. but all the other roles are not python apps | 12:35 |
| noonedeadpunk | yup | 12:36 |
| mgariepy | ok so cleanup instead of patching ! | 12:36 |
| noonedeadpunk | well I guess for others it was more like way to install roles before galaxy was a thing | 12:37 |
| OutBackDingo | mgariepy: yeah the new configs dont set the mac | 12:37 |
| mgariepy | let's me fix those. | 12:37 |
| OutBackDingo | right now im down to debugging / fixing a cumulus | 12:37 |
| noonedeadpunk | as it was installing roles into ansible roles paths | 12:37 |
| noonedeadpunk | but today it's really weird and cleanup is way better imo | 12:37 |
| mgariepy | yep | 12:39 |
| opendevreview | Marc Gariépy proposed openstack/ansible-role-python_venv_build master: Cleanup setup.py config https://review.opendev.org/c/openstack/ansible-role-python_venv_build/+/835992 | 12:45 |
| opendevreview | Marc Gariépy proposed openstack/ansible-role-systemd_mount master: Cleanup setup.py config https://review.opendev.org/c/openstack/ansible-role-systemd_mount/+/835993 | 12:45 |
| opendevreview | Marc Gariépy proposed openstack/ansible-role-systemd_service master: Cleanup setup.py config https://review.opendev.org/c/openstack/ansible-role-systemd_service/+/835994 | 12:45 |
| opendevreview | Marc Gariépy proposed openstack/openstack-ansible-ceph_client master: Cleanup setup.py config https://review.opendev.org/c/openstack/openstack-ansible-ceph_client/+/835995 | 12:45 |
| opendevreview | Marc Gariépy proposed openstack/openstack-ansible-os_aodh master: Cleanup setup.py config https://review.opendev.org/c/openstack/openstack-ansible-os_aodh/+/835996 | 12:45 |
| opendevreview | Marc Gariépy proposed openstack/openstack-ansible-os_ceilometer master: Cleanup setup.py config https://review.opendev.org/c/openstack/openstack-ansible-os_ceilometer/+/835997 | 12:45 |
| opendevreview | Marc Gariépy proposed openstack/openstack-ansible-os_glance master: Cleanup setup.py config https://review.opendev.org/c/openstack/openstack-ansible-os_glance/+/835998 | 12:45 |
| opendevreview | Marc Gariépy proposed openstack/openstack-ansible-os_gnocchi master: Cleanup setup.py config https://review.opendev.org/c/openstack/openstack-ansible-os_gnocchi/+/835999 | 12:45 |
| opendevreview | Marc Gariépy proposed openstack/openstack-ansible-os_horizon master: Cleanup setup.py config https://review.opendev.org/c/openstack/openstack-ansible-os_horizon/+/836000 | 12:46 |
| opendevreview | Marc Gariépy proposed openstack/openstack-ansible-os_magnum master: Cleanup setup.py config https://review.opendev.org/c/openstack/openstack-ansible-os_magnum/+/836001 | 12:46 |
| opendevreview | Marc Gariépy proposed openstack/openstack-ansible-os_manila master: Cleanup setup.py config https://review.opendev.org/c/openstack/openstack-ansible-os_manila/+/836002 | 12:46 |
| opendevreview | Marc Gariépy proposed openstack/openstack-ansible-os_neutron master: Cleanup setup.py config https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/836003 | 12:46 |
| opendevreview | Marc Gariépy proposed openstack/openstack-ansible-os_octavia master: Cleanup setup.py config https://review.opendev.org/c/openstack/openstack-ansible-os_octavia/+/836004 | 12:46 |
| opendevreview | Marc Gariépy proposed openstack/openstack-ansible-os_sahara master: Cleanup setup.py config https://review.opendev.org/c/openstack/openstack-ansible-os_sahara/+/836005 | 12:46 |
| opendevreview | Marc Gariépy proposed openstack/openstack-ansible-os_tacker master: Cleanup setup.py config https://review.opendev.org/c/openstack/openstack-ansible-os_tacker/+/836006 | 12:46 |
| opendevreview | Marc Gariépy proposed openstack/openstack-ansible-os_trove master: Cleanup setup.py config https://review.opendev.org/c/openstack/openstack-ansible-os_trove/+/836007 | 12:46 |
| opendevreview | Marc Gariépy proposed openstack/openstack-ansible-os_zun master: Cleanup setup.py config https://review.opendev.org/c/openstack/openstack-ansible-os_zun/+/836008 | 12:46 |
| opendevreview | Marc Gariépy proposed openstack/openstack-ansible-tests master: Cleanup setup.py config https://review.opendev.org/c/openstack/openstack-ansible-tests/+/836009 | 12:48 |
| opendevreview | Marc Gariépy proposed openstack/openstack-ansible-os_tempest master: Updated from OpenStack Ansible Tests https://review.opendev.org/c/openstack/openstack-ansible-os_tempest/+/835724 | 12:53 |
| mgariepy | good for now :D. i don't have time to rebase all the other patch but i guess we can merge these then recheck the others.. | 12:53 |
| noonedeadpunk | yep | 13:01 |
| noonedeadpunk | oh, hm https://zuul.opendev.org/t/openstack/build/fa918c977f5c4740a8348a7e2fc2e23b | 13:23 |
| noonedeadpunk | so seems mgariepy we can't do cleanup and ou was right originally... | 13:25 |
| noonedeadpunk | and I guess we just missing projects from https://review.opendev.org/q/topic:workaround-setuptools-autodiscovery topic - will populate these then.... | 13:26 |
| mgariepy | so the setup.py is needed for the doc. | 14:01 |
| mgariepy | then how comes that we don't have it for some repos ? | 14:01 |
| mgariepy | noonedeadpunk, https://github.com/openstack/openstack-ansible-galera_server/commit/1ba6cc04f9ddf5e07e11ff340f3df728e4d6620b | 14:11 |
| mgariepy | this plus the cleanup ? | 14:12 |
| opendevreview | Marc Gariépy proposed openstack/openstack-ansible-os_tacker master: Cleanup setup.py config https://review.opendev.org/c/openstack/openstack-ansible-os_tacker/+/836006 | 14:16 |
| opendevreview | Marc Gariépy proposed openstack/openstack-ansible-os_tacker master: Cleanup setup.py config https://review.opendev.org/c/openstack/openstack-ansible-os_tacker/+/836006 | 14:17 |
| roham | Hi all. Is there a way to connect a second host to a fully working Openstack-Ansible all-in-one deployment (vers. Victoria)? I can't connect a bridge to the AIO node in spite of removing the iptables masquerading stuff on the eth interface | 14:27 |
| noonedeadpunk | mgariepy: mmmmm | 14:31 |
| noonedeadpunk | I can recall doing that :) | 14:31 |
| noonedeadpunk | *same | 14:31 |
| mgariepy | lol | 14:31 |
| mgariepy | i'll resubmit the cleanup with that. | 14:31 |
| mgariepy | tacker passed. | 14:31 |
| noonedeadpunk | I will abandon patches again then :D | 14:32 |
| mgariepy | ok thanks | 14:32 |
| NeilHanlon | :D | 14:33 |
| opendevreview | Marc Gariépy proposed openstack/ansible-role-python_venv_build master: Cleanup setup.py config https://review.opendev.org/c/openstack/ansible-role-python_venv_build/+/835992 | 14:34 |
| NeilHanlon | roham: AIOs aren't particularly flexible in terms of growing them later on, at least.. in my experience | 14:34 |
| opendevreview | Marc Gariépy proposed openstack/ansible-role-systemd_mount master: Cleanup setup.py config https://review.opendev.org/c/openstack/ansible-role-systemd_mount/+/835993 | 14:34 |
| opendevreview | Marc Gariépy proposed openstack/ansible-role-systemd_service master: Cleanup setup.py config https://review.opendev.org/c/openstack/ansible-role-systemd_service/+/835994 | 14:35 |
| opendevreview | Marc Gariépy proposed openstack/openstack-ansible-ceph_client master: Cleanup setup.py config https://review.opendev.org/c/openstack/openstack-ansible-ceph_client/+/835995 | 14:36 |
| opendevreview | Marc Gariépy proposed openstack/openstack-ansible-os_aodh master: Cleanup setup.py config https://review.opendev.org/c/openstack/openstack-ansible-os_aodh/+/835996 | 14:38 |
| opendevreview | Marc Gariépy proposed openstack/openstack-ansible-os_ceilometer master: Cleanup setup.py config https://review.opendev.org/c/openstack/openstack-ansible-os_ceilometer/+/835997 | 14:38 |
| opendevreview | Marc Gariépy proposed openstack/openstack-ansible-os_glance master: Cleanup setup.py config https://review.opendev.org/c/openstack/openstack-ansible-os_glance/+/835998 | 14:38 |
| opendevreview | Marc Gariépy proposed openstack/openstack-ansible-os_gnocchi master: Cleanup setup.py config https://review.opendev.org/c/openstack/openstack-ansible-os_gnocchi/+/835999 | 14:39 |
| opendevreview | Marc Gariépy proposed openstack/openstack-ansible-os_horizon master: Cleanup setup.py config https://review.opendev.org/c/openstack/openstack-ansible-os_horizon/+/836000 | 14:39 |
| opendevreview | Marc Gariépy proposed openstack/openstack-ansible-os_magnum master: Cleanup setup.py config https://review.opendev.org/c/openstack/openstack-ansible-os_magnum/+/836001 | 14:39 |
| opendevreview | Marc Gariépy proposed openstack/openstack-ansible-os_manila master: Cleanup setup.py config https://review.opendev.org/c/openstack/openstack-ansible-os_manila/+/836002 | 14:39 |
| opendevreview | Marc Gariépy proposed openstack/openstack-ansible-os_neutron master: Cleanup setup.py config https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/836003 | 14:40 |
| opendevreview | Marc Gariépy proposed openstack/openstack-ansible-os_octavia master: Cleanup setup.py config https://review.opendev.org/c/openstack/openstack-ansible-os_octavia/+/836004 | 14:40 |
| opendevreview | Marc Gariépy proposed openstack/openstack-ansible-os_sahara master: Cleanup setup.py config https://review.opendev.org/c/openstack/openstack-ansible-os_sahara/+/836005 | 14:40 |
| opendevreview | Marc Gariépy proposed openstack/openstack-ansible-os_trove master: Cleanup setup.py config https://review.opendev.org/c/openstack/openstack-ansible-os_trove/+/836007 | 14:40 |
| opendevreview | Marc Gariépy proposed openstack/openstack-ansible-os_zun master: Cleanup setup.py config https://review.opendev.org/c/openstack/openstack-ansible-os_zun/+/836008 | 14:41 |
| opendevreview | Marc Gariépy proposed openstack/openstack-ansible-tests master: Cleanup setup.py config https://review.opendev.org/c/openstack/openstack-ansible-tests/+/836009 | 14:41 |
| opendevreview | Marc Gariépy proposed openstack/openstack-ansible-os_tempest master: cleanup setup.py config https://review.opendev.org/c/openstack/openstack-ansible-os_tempest/+/835969 | 14:43 |
| mgariepy | should be all. | 14:44 |
| *** dviroel_ is now known as dviroel | 15:54 | |
| jrosser | roham: you would need to extend the vlans from the br-<...> on the AIO node to your second node across some physical interface / switch | 16:00 |
| jrosser | i've done "all in two" type things like that before | 16:01 |
| jrosser | then add the second node to openstack_user_config as a compute or whatever you want it for | 16:01 |
| *** dviroel is now known as dviroel|lunch | 16:14 | |
| mgariepy | jrosser, if you have a couple of minutes :Dhttps://review.opendev.org/q/topic:cleanup-setup | 16:54 |
| *** dviroel|lunch is now known as dviroel | 17:18 | |
| roham | @jrosser, yes the bridges are done, but there is no connection betweeen the host bridges br-mgmt, br-storage and the others, I must be missing something in the systemd-networkd stuff of the AIO node. The openstack_user_config edit is ready to be added | 17:23 |
| jrosser | roham: have you attached the bridges to physical interfaces, that’s normally not needed on an AIO so you will have to add that | 18:07 |
| mgariepy | thanks jrosser | 19:44 |
| opendevreview | Merged openstack/openstack-ansible-os_gnocchi master: Cleanup setup.py config https://review.opendev.org/c/openstack/openstack-ansible-os_gnocchi/+/835999 | 19:49 |
| opendevreview | Merged openstack/openstack-ansible-os_sahara master: Cleanup setup.py config https://review.opendev.org/c/openstack/openstack-ansible-os_sahara/+/836005 | 19:49 |
| opendevreview | Merged openstack/openstack-ansible-os_tempest master: cleanup setup.py config https://review.opendev.org/c/openstack/openstack-ansible-os_tempest/+/835969 | 19:50 |
| opendevreview | Merged openstack/openstack-ansible-os_zun master: Cleanup setup.py config https://review.opendev.org/c/openstack/openstack-ansible-os_zun/+/836008 | 19:50 |
| opendevreview | Merged openstack/openstack-ansible-os_neutron master: Cleanup setup.py config https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/836003 | 19:50 |
| opendevreview | Merged openstack/openstack-ansible-os_octavia master: Cleanup setup.py config https://review.opendev.org/c/openstack/openstack-ansible-os_octavia/+/836004 | 19:50 |
| opendevreview | Merged openstack/openstack-ansible-os_trove master: Cleanup setup.py config https://review.opendev.org/c/openstack/openstack-ansible-os_trove/+/836007 | 19:51 |
| opendevreview | Merged openstack/openstack-ansible-os_aodh master: Cleanup setup.py config https://review.opendev.org/c/openstack/openstack-ansible-os_aodh/+/835996 | 19:52 |
| opendevreview | Merged openstack/openstack-ansible-os_magnum master: Cleanup setup.py config https://review.opendev.org/c/openstack/openstack-ansible-os_magnum/+/836001 | 19:52 |
| opendevreview | Merged openstack/ansible-role-python_venv_build master: Cleanup setup.py config https://review.opendev.org/c/openstack/ansible-role-python_venv_build/+/835992 | 19:53 |
| opendevreview | Merged openstack/openstack-ansible-os_ceilometer master: Cleanup setup.py config https://review.opendev.org/c/openstack/openstack-ansible-os_ceilometer/+/835997 | 19:54 |
| opendevreview | Merged openstack/ansible-role-systemd_mount master: Cleanup setup.py config https://review.opendev.org/c/openstack/ansible-role-systemd_mount/+/835993 | 19:54 |
| opendevreview | Merged openstack/openstack-ansible-ceph_client master: Cleanup setup.py config https://review.opendev.org/c/openstack/openstack-ansible-ceph_client/+/835995 | 19:57 |
| opendevreview | Merged openstack/openstack-ansible-os_tacker master: Cleanup setup.py config https://review.opendev.org/c/openstack/openstack-ansible-os_tacker/+/836006 | 19:57 |
| opendevreview | Merged openstack/ansible-role-systemd_service master: Cleanup setup.py config https://review.opendev.org/c/openstack/ansible-role-systemd_service/+/835994 | 19:58 |
| opendevreview | Merged openstack/openstack-ansible-os_glance master: Cleanup setup.py config https://review.opendev.org/c/openstack/openstack-ansible-os_glance/+/835998 | 20:04 |
| opendevreview | Merged openstack/openstack-ansible-os_manila master: Cleanup setup.py config https://review.opendev.org/c/openstack/openstack-ansible-os_manila/+/836002 | 20:10 |
| opendevreview | Marc Gariépy proposed openstack/ansible-role-systemd_service master: Updated from OpenStack Ansible Tests https://review.opendev.org/c/openstack/ansible-role-systemd_service/+/835737 | 20:13 |
| opendevreview | Marc Gariépy proposed openstack/openstack-ansible-os_neutron master: Updated from OpenStack Ansible Tests https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/835715 | 20:14 |
| opendevreview | Marc Gariépy proposed openstack/openstack-ansible-os_aodh master: Updated from OpenStack Ansible Tests https://review.opendev.org/c/openstack/openstack-ansible-os_aodh/+/835698 | 20:14 |
| opendevreview | Marc Gariépy proposed openstack/openstack-ansible-os_ceilometer master: Updated from OpenStack Ansible Tests https://review.opendev.org/c/openstack/openstack-ansible-os_ceilometer/+/835701 | 20:15 |
| opendevreview | Marc Gariépy proposed openstack/openstack-ansible-os_magnum master: Updated from OpenStack Ansible Tests https://review.opendev.org/c/openstack/openstack-ansible-os_magnum/+/835711 | 20:15 |
| opendevreview | Marc Gariépy proposed openstack/openstack-ansible-os_sahara master: Updated from OpenStack Ansible Tests https://review.opendev.org/c/openstack/openstack-ansible-os_sahara/+/835720 | 20:15 |
| opendevreview | Marc Gariépy proposed openstack/ansible-role-python_venv_build master: Updated from OpenStack Ansible Tests https://review.opendev.org/c/openstack/ansible-role-python_venv_build/+/835734 | 20:15 |
| opendevreview | Marc Gariépy proposed openstack/openstack-ansible-os_zun master: Updated from OpenStack Ansible Tests https://review.opendev.org/c/openstack/openstack-ansible-os_zun/+/835726 | 20:15 |
| opendevreview | Marc Gariépy proposed openstack/openstack-ansible-os_tacker master: Updated from OpenStack Ansible Tests https://review.opendev.org/c/openstack/openstack-ansible-os_tacker/+/835723 | 20:15 |
| opendevreview | Marc Gariépy proposed openstack/openstack-ansible-os_manila master: Updated from OpenStack Ansible Tests https://review.opendev.org/c/openstack/openstack-ansible-os_manila/+/835712 | 20:15 |
| opendevreview | Marc Gariépy proposed openstack/openstack-ansible-ceph_client master: Updated from OpenStack Ansible Tests https://review.opendev.org/c/openstack/openstack-ansible-ceph_client/+/835668 | 20:15 |
| opendevreview | Marc Gariépy proposed openstack/ansible-role-systemd_mount master: Updated from OpenStack Ansible Tests https://review.opendev.org/c/openstack/ansible-role-systemd_mount/+/835735 | 20:15 |
| opendevreview | Marc Gariépy proposed openstack/openstack-ansible-os_trove master: Updated from OpenStack Ansible Tests https://review.opendev.org/c/openstack/openstack-ansible-os_trove/+/835725 | 20:15 |
| opendevreview | Marc Gariépy proposed openstack/openstack-ansible-os_gnocchi master: Updated from OpenStack Ansible Tests https://review.opendev.org/c/openstack/openstack-ansible-os_gnocchi/+/835706 | 20:15 |
| opendevreview | Marc Gariépy proposed openstack/openstack-ansible-os_glance master: Updated from OpenStack Ansible Tests https://review.opendev.org/c/openstack/openstack-ansible-os_glance/+/835705 | 20:15 |
| opendevreview | Marc Gariépy proposed openstack/openstack-ansible-os_octavia master: Updated from OpenStack Ansible Tests https://review.opendev.org/c/openstack/openstack-ansible-os_octavia/+/835717 | 20:15 |
| opendevreview | Marc Gariépy proposed openstack/openstack-ansible-os_tempest master: Updated from OpenStack Ansible Tests https://review.opendev.org/c/openstack/openstack-ansible-os_tempest/+/835724 | 20:16 |
| opendevreview | Merged openstack/openstack-ansible-os_horizon master: Cleanup setup.py config https://review.opendev.org/c/openstack/openstack-ansible-os_horizon/+/836000 | 20:23 |
| *** dviroel is now known as dviroel|afk | 20:24 | |
| opendevreview | Marc Gariépy proposed openstack/openstack-ansible-os_horizon master: Updated from OpenStack Ansible Tests https://review.opendev.org/c/openstack/openstack-ansible-os_horizon/+/835708 | 20:25 |
| opendevreview | Dariusz Smigiel proposed openstack/openstack-ansible-os_tempest master: Revert "cleanup setup.py config" https://review.opendev.org/c/openstack/openstack-ansible-os_tempest/+/835977 | 20:35 |
| jrosser | ^ thats unfortunate - means we can't clean up os_tempest or config_template | 20:52 |
| mgariepy | waht does that install ? | 20:54 |
| jrosser | well, seems that tripleo install it with...... | 20:55 |
| jrosser | pip | 20:55 |
| mgariepy | well. | 20:59 |
| mgariepy | i guess we need to bring back the the other patch. | 20:59 |
| jrosser | the revert will fail the tests anyway | 20:59 |
| mgariepy | yep, it needs the other one also. | 21:00 |
| mgariepy | ansible-role-python_venv_build will also need the revert. | 21:01 |
| opendevreview | Marc Gariépy proposed openstack/ansible-role-python_venv_build master: Revert "Cleanup setup.py config" https://review.opendev.org/c/openstack/ansible-role-python_venv_build/+/835978 | 21:02 |
| opendevreview | Marc Gariépy proposed openstack/ansible-role-python_venv_build master: Disable setuptools auto discovery https://review.opendev.org/c/openstack/ansible-role-python_venv_build/+/835892 | 21:02 |
| jrosser | i think you need to quash those two | 21:03 |
| jrosser | *squash | 21:03 |
| mgariepy | ha. | 21:03 |
| jrosser | the revert will fail openstack-tox-docs just like before | 21:03 |
| mgariepy | i whould have though that they would merge together | 21:03 |
| jrosser | they would both have to pass the jobs to do that | 21:04 |
| jrosser | i would expect the first to fail and the second to pass | 21:04 |
| opendevreview | Marc Gariépy proposed openstack/openstack-ansible-os_tempest master: Revert "cleanup setup.py config" https://review.opendev.org/c/openstack/openstack-ansible-os_tempest/+/835977 | 21:07 |
| opendevreview | Marc Gariépy proposed openstack/ansible-role-python_venv_build master: Revert "Cleanup setup.py config" https://review.opendev.org/c/openstack/ansible-role-python_venv_build/+/836091 | 21:08 |
| mgariepy | like this ? | 21:08 |
| jrosser | perhaps - though how installing it at all with pip works i'm not sure | 21:12 |
| mgariepy | i tested it.. and it does work... | 21:12 |
| jrosser | does it just copy all the files? | 21:13 |
| mgariepy | https://paste.openstack.org/show/bVnFTVliMDc0PCqSvA8D/ | 21:13 |
| jrosser | blimey | 21:14 |
| jrosser | how did it know to do that :) | 21:14 |
| mgariepy | $ pip install git+https://opendev.org/openstack/openstack-ansible-os_tempest.git@2402146bf778cbd7e140958294e91348c94b5ec3 | 21:14 |
| jrosser | ooooh i see setup.cfg | 21:14 |
| mgariepy | no clue on this. | 21:14 |
| mgariepy | ha | 21:15 |
| mgariepy | shall we revert then ? lol | 21:15 |
| mgariepy | probably need to discuss that. | 21:16 |
| mgariepy | not today tho. food/family time for me. | 21:16 |
| opendevreview | Merged openstack/openstack-ansible-os_tempest master: Revert "cleanup setup.py config" https://review.opendev.org/c/openstack/openstack-ansible-os_tempest/+/835977 | 22:13 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!