opendevreview | Michał Dulko proposed openstack/kuryr-kubernetes master: Update minimum openstacksdk version to 0.59.0 https://review.opendev.org/c/openstack/kuryr-kubernetes/+/848749 | 07:30 |
---|---|---|
dulek | Looks like something broke ovs jobs on master: `ERROR neutron.common.experimental [-] Feature 'linuxbridge' is experimental and has to be explicitly enabled in 'cfg.CONF.experimental'` | 10:47 |
dulek | Do we run with linuxbridge there? I'm a bit surprised. | 10:47 |
dulek | gryf: ^ | 10:47 |
gryf | hm. | 10:49 |
gryf | let me look. | 10:49 |
dulek | https://review.opendev.org/c/openstack/neutron/+/845181 - I guess it's this thing. | 10:50 |
dulek | Maybe it's time to stop OVS testing…? | 10:50 |
gryf | actually, we do: https://zuul.opendev.org/t/openstack/build/33ffb5b0301947e0a098a77f16157be5/log/controller/logs/local_conf.txt#86 | 10:52 |
gryf | dulek, I'm not sure what's the reason but must be some interconnectivity thing between nodes. | 10:53 |
dulek | Hm, I wonder if we should stop it. Okay, let's try a few things here then. | 10:54 |
gryf | for sure we don't do that for ovn multinode, maybe it's not needed anymore? | 10:55 |
opendevreview | Michał Dulko proposed openstack/kuryr-kubernetes master: Disable linuxbridge on the OVS gates https://review.opendev.org/c/openstack/kuryr-kubernetes/+/848836 | 10:56 |
dulek | https://review.opendev.org/c/openstack/kuryr-kubernetes/+/848836 | 10:56 |
dulek | Let's see. | 10:56 |
dulek | Looks like we just carried that for a longer period. | 10:57 |
dulek | I'll think more about this if the patch fails, otherwise let's just disable it. | 10:57 |
opendevreview | Roman Dobosz proposed openstack/kuryr-kubernetes master: Remove experimental linuxbridge from mutlinode OVS job. https://review.opendev.org/c/openstack/kuryr-kubernetes/+/848838 | 11:41 |
gryf | dulek, ^^ | 11:41 |
dulek | gryf: Doesn't it duplicate my above patch, just doesn't remove it from base ovs? | 11:59 |
gryf | aww | 12:01 |
gryf | ok, I'll abandon mine. | 12:02 |
gryf | (didn't notice that, sorry) | 12:02 |
dulek | I do it all the time. ;) | 13:07 |
dulek | gryf, ltomasbo: This looks wonderfully green now: https://review.opendev.org/c/openstack/kuryr-kubernetes/+/848836. | 15:23 |
dulek | Also looks like the e2e NP job will no longer work: `github.com/onsi/ginkgo@v1.16.5: missing go.sum entry; to add it:` | 15:24 |
ltomasbo | nice! | 15:24 |
gryf | as for e2e - there might be outdated now. and I've started to work for support "new" framework, but never got it run on gates for some reason. https://review.opendev.org/c/openstack/kuryr-kubernetes/+/831941 | 15:53 |
opendevreview | Merged openstack/kuryr-kubernetes master: Disable linuxbridge on the OVS gates https://review.opendev.org/c/openstack/kuryr-kubernetes/+/848836 | 17:06 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!