| ralonsoh | lajoskatona, slaweq good morning! if you have some minutes (no rush, this has low prio): https://review.opendev.org/c/openstack/neutron/+/907313 | 06:55 |
|---|---|---|
| ralonsoh | thanks in advance! | 06:56 |
| opendevreview | yatin proposed openstack/neutron-tempest-plugin master: Temporary switch lb/ovs/ovn scenario jobs to cirros-0.5.3 https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/924124 | 06:58 |
| slaweq | ralonsoh: +2, thx for addressing my comments there :) | 08:27 |
| *** sfinucan is now known as stephenfin | 09:01 | |
| *** elodilles is now known as elodilles_afk | 10:00 | |
| ralonsoh | ykarel, I think we should add another not in https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/924124/1/zuul.d/base-nested-switch.yaml#36, pointing to LP#2045549 | 10:09 |
| ralonsoh | do you agree? | 10:09 |
| ralonsoh | just to have this change documented | 10:09 |
| ykarel | ralonsoh, sure can add, thought to have this patch short lived | 10:10 |
| ralonsoh | perfect! | 10:10 |
| opendevreview | yatin proposed openstack/neutron-tempest-plugin master: Temporary switch lb/ovs/ovn scenario jobs to cirros-0.5.3 https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/924124 | 10:12 |
| ykarel | done | 10:12 |
| ralonsoh | +2 | 10:12 |
| ykarel | thx | 10:12 |
| frickler | oh, wow, I wasn't aware that this is such a common issue. 500 hits in 7d. and I must admit I forgot to check the cirros patch last week | 11:19 |
| ykarel | frickler, out of those 500, more than half looks some patch specfics, but yes we are hitting this issure randomly in every few runs | 11:49 |
| *** elodilles_afk is now known as elodilles | 12:07 | |
| *** haleyb|out is now known as haleyb | 12:32 | |
| opendevreview | Lucas Alvares Gomes proposed openstack/neutron master: [OVN] Add support for broadcast_arps_to_all_routers https://review.opendev.org/c/openstack/neutron/+/923391 | 13:26 |
| seba | haleyb, we've been talking about v6 only servers with cloud-init and I said I couldn't get this to work, due to the link local address. Apparently there now is a patch in cloud init that actually uses the zone identifier, getting this to work. https://github.com/canonical/cloud-init/commit/2ea276548e05dd1ee465bb71fc4c0f5305ff0bf4 | 14:45 |
| seba | I still wonder how this ever worked before | 14:45 |
| opendevreview | Merged openstack/tap-as-a-service master: Devstack: Make sure that taas ini file used by Neutron https://review.opendev.org/c/openstack/tap-as-a-service/+/923837 | 15:30 |
| ihrachys | haleyb is refreshing nested snat patch to handle Terry's comments about updates not handled on your shortlist? let me know if you'll need someone else to take it over. | 16:16 |
| opendevreview | Jakub Libosvar proposed openstack/neutron master: Don't print traceback if standard attr is missing on update https://review.opendev.org/c/openstack/neutron/+/922043 | 19:15 |
| haleyb | seba: interesting - i remember it working before, don't know if something changed, but glad it's addressed | 20:42 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!