*** spatel has quit IRC | 00:03 | |
openstackgerrit | Brian Haley proposed openstack/ovn-octavia-provider master: Fix gate failure https://review.opendev.org/c/openstack/ovn-octavia-provider/+/765872 | 00:07 |
---|---|---|
openstackgerrit | Brian Haley proposed openstack/ovn-octavia-provider master: Fix gate failure https://review.opendev.org/c/openstack/ovn-octavia-provider/+/765872 | 00:12 |
openstackgerrit | Brian Haley proposed openstack/ovn-octavia-provider master: Fix gate failure https://review.opendev.org/c/openstack/ovn-octavia-provider/+/765872 | 00:19 |
openstackgerrit | Brian Haley proposed openstack/ovn-octavia-provider master: Fix gate failure https://review.opendev.org/c/openstack/ovn-octavia-provider/+/765872 | 00:29 |
*** macz_ has quit IRC | 00:46 | |
*** zzzeek has quit IRC | 00:47 | |
*** zzzeek has joined #openstack-neutron | 00:49 | |
*** openstackgerrit has quit IRC | 00:58 | |
*** sapd1 has joined #openstack-neutron | 01:14 | |
*** sapd1 has quit IRC | 01:19 | |
*** sapd1 has joined #openstack-neutron | 01:21 | |
*** sapd1 has quit IRC | 01:25 | |
*** openstackgerrit has joined #openstack-neutron | 01:32 | |
openstackgerrit | Terry Wilson proposed openstack/networking-ovn stable/train: Ensure ovsdb_probe_interval set before connect() https://review.opendev.org/c/openstack/networking-ovn/+/765896 | 01:32 |
*** tbachman has quit IRC | 02:01 | |
*** tbachman_ has joined #openstack-neutron | 02:02 | |
*** zzzeek has quit IRC | 02:13 | |
*** zzzeek has joined #openstack-neutron | 02:14 | |
*** zzzeek has quit IRC | 02:28 | |
*** zzzeek has joined #openstack-neutron | 02:29 | |
*** zzzeek has quit IRC | 02:34 | |
*** hemanth_n has joined #openstack-neutron | 02:36 | |
*** zzzeek has joined #openstack-neutron | 02:36 | |
*** hemanth_n has quit IRC | 02:49 | |
*** tamas_erdei has joined #openstack-neutron | 02:54 | |
*** terdei has quit IRC | 02:57 | |
*** hemanth_n has joined #openstack-neutron | 03:29 | |
*** hemanth_n has quit IRC | 03:33 | |
*** zzzeek has quit IRC | 03:44 | |
*** zzzeek has joined #openstack-neutron | 03:44 | |
*** hamalq_ has quit IRC | 03:45 | |
*** rcernin has quit IRC | 03:46 | |
*** hemanth_n has joined #openstack-neutron | 03:48 | |
*** rcernin has joined #openstack-neutron | 03:50 | |
*** sapd1 has joined #openstack-neutron | 03:58 | |
*** zzzeek has quit IRC | 04:26 | |
*** zzzeek has joined #openstack-neutron | 04:27 | |
*** zzzeek has quit IRC | 04:41 | |
*** zzzeek has joined #openstack-neutron | 04:45 | |
*** jlibosva has joined #openstack-neutron | 04:53 | |
*** jlibosva has quit IRC | 04:57 | |
*** ratailor has joined #openstack-neutron | 05:03 | |
*** lajoskatona has joined #openstack-neutron | 05:22 | |
*** rcernin has quit IRC | 05:39 | |
*** ociuhandu has joined #openstack-neutron | 05:40 | |
*** ociuhandu has quit IRC | 05:44 | |
*** benj_ has quit IRC | 06:04 | |
*** benj_ has joined #openstack-neutron | 06:05 | |
*** rcernin has joined #openstack-neutron | 06:08 | |
*** gcheresh has joined #openstack-neutron | 06:24 | |
*** elod_pto has joined #openstack-neutron | 06:31 | |
*** waleedm has joined #openstack-neutron | 06:37 | |
*** elod_pto has quit IRC | 06:38 | |
*** zzzeek has quit IRC | 06:41 | |
*** hamalq has joined #openstack-neutron | 06:44 | |
*** zzzeek has joined #openstack-neutron | 06:45 | |
*** rcernin has quit IRC | 06:45 | |
*** elod_pto has joined #openstack-neutron | 06:45 | |
*** rcernin has joined #openstack-neutron | 07:01 | |
*** zzzeek has quit IRC | 07:03 | |
*** zzzeek has joined #openstack-neutron | 07:05 | |
*** lxkong has quit IRC | 07:09 | |
*** zzzeek has quit IRC | 07:24 | |
*** rcernin has quit IRC | 07:25 | |
*** abdysn has joined #openstack-neutron | 07:26 | |
*** zzzeek has joined #openstack-neutron | 07:28 | |
*** jlibosva has joined #openstack-neutron | 07:30 | |
openstackgerrit | Mamduh proposed openstack/os-vif stable/train: Refactor code of linux_net to more cleaner and increase performace https://review.opendev.org/c/openstack/os-vif/+/765912 | 07:37 |
openstackgerrit | Mamduh proposed openstack/os-vif stable/stein: Refactor code of linux_net to more cleaner and increase performace https://review.opendev.org/c/openstack/os-vif/+/765914 | 07:52 |
*** lpetrut has joined #openstack-neutron | 07:55 | |
*** tmorin has joined #openstack-neutron | 08:03 | |
*** luksky has joined #openstack-neutron | 08:04 | |
*** slaweq has joined #openstack-neutron | 08:08 | |
*** rpittau|afk is now known as rpittau | 08:10 | |
*** tesseract has joined #openstack-neutron | 08:13 | |
*** bengates has joined #openstack-neutron | 08:21 | |
*** jlibosva has quit IRC | 08:28 | |
*** jlibosva has joined #openstack-neutron | 08:29 | |
*** ociuhandu has joined #openstack-neutron | 08:35 | |
hemanth_n | hi looking for reviews on https://review.opendev.org/c/openstack/neutron/+/752248 initial reviews are done with +2 .. appreciate further reviews.. thanks | 08:40 |
*** baojg has quit IRC | 08:44 | |
*** ociuhandu has quit IRC | 08:44 | |
openstackgerrit | Rico Lin proposed openstack/neutron master: Switch to openstack-python3-wallaby-jobs https://review.opendev.org/c/openstack/neutron/+/765923 | 08:51 |
openstackgerrit | Rico Lin proposed openstack/neutron master: Add openstack-python3-wallaby-jobs-arm64 job https://review.opendev.org/c/openstack/neutron/+/742101 | 08:53 |
*** zzzeek has quit IRC | 08:57 | |
*** zzzeek has joined #openstack-neutron | 08:59 | |
*** lxkong has joined #openstack-neutron | 09:00 | |
*** tmorin has quit IRC | 09:05 | |
*** baojg has joined #openstack-neutron | 09:05 | |
*** baojg has quit IRC | 09:08 | |
*** baojg has joined #openstack-neutron | 09:09 | |
*** lucasagomes has joined #openstack-neutron | 09:10 | |
openstackgerrit | Slawek Kaplonski proposed openstack/neutron master: Implement secure RBAC for trunk API https://review.opendev.org/c/openstack/neutron/+/765390 | 09:10 |
*** hamalq has quit IRC | 09:10 | |
*** hamalq has joined #openstack-neutron | 09:14 | |
*** hamalq_ has joined #openstack-neutron | 09:16 | |
*** hamalq has quit IRC | 09:18 | |
*** fnordahl has joined #openstack-neutron | 09:19 | |
*** tmorin has joined #openstack-neutron | 09:20 | |
*** hamalq_ has quit IRC | 09:20 | |
openstackgerrit | Slawek Kaplonski proposed openstack/neutron master: Implement secure RBAC for subnet pool API https://review.opendev.org/c/openstack/neutron/+/765391 | 09:21 |
openstackgerrit | Slawek Kaplonski proposed openstack/neutron master: Implement secure RBAC for the subnet API https://review.opendev.org/c/openstack/neutron/+/765392 | 09:27 |
*** ociuhandu has joined #openstack-neutron | 09:28 | |
openstackgerrit | Slawek Kaplonski proposed openstack/neutron master: Implement secure RBAC for the segment API https://review.opendev.org/c/openstack/neutron/+/765393 | 09:31 |
*** tmorin has quit IRC | 09:38 | |
*** tmorin has joined #openstack-neutron | 09:56 | |
*** baojg has quit IRC | 09:57 | |
*** baojg has joined #openstack-neutron | 09:58 | |
*** hamalq has joined #openstack-neutron | 10:00 | |
openstackgerrit | Slawek Kaplonski proposed openstack/neutron master: Implement secure RBAC for security group API https://review.opendev.org/c/openstack/neutron/+/765394 | 10:00 |
openstackgerrit | Lajos Katona proposed openstack/neutron master: [Docs] Guide about running and debugging tempest tests https://review.opendev.org/c/openstack/neutron/+/765958 | 10:04 |
*** hamalq has quit IRC | 10:04 | |
openstackgerrit | Bernard Cafarelli proposed openstack/neutron master: Update major release checklist https://review.opendev.org/c/openstack/neutron/+/765959 | 10:06 |
openstackgerrit | Lajos Katona proposed openstack/neutron master: [Docs] Guide about running and debugging tempest tests https://review.opendev.org/c/openstack/neutron/+/765958 | 10:10 |
zigo | Where should I file bugs about neutron-dynamic-routing ? | 10:14 |
openstackgerrit | Slawek Kaplonski proposed openstack/neutron master: Implement secure RBAC for the router API https://review.opendev.org/c/openstack/neutron/+/765395 | 10:14 |
openstackgerrit | Mamduh proposed openstack/os-vif stable/rocky: Refactor code of linux_net to more cleaner and increase performace https://review.opendev.org/c/openstack/os-vif/+/765962 | 10:21 |
lajoskatona | zigo: https://launchpad.net/neutron | 10:30 |
zigo | lajoskatona: Is there a way to tell it's for neutron-d-r ? | 10:30 |
lajoskatona | zigo: I suppose add to the subject/sumamry line that its n-d-r and there's a tag (https://docs.openstack.org/neutron/latest/contributor/policies/bugs.html#proposing-new-tags ) l3-bgp which you can use | 10:34 |
zigo | Bug filled: https://bugs.launchpad.net/neutron/+bug/1907232 | 10:35 |
openstack | Launchpad bug 1907232 in neutron "dynamic-routing: "dragent add speaker" is buggy" [Undecided,New] | 10:35 |
*** zzzeek has quit IRC | 10:41 | |
*** zzzeek has joined #openstack-neutron | 10:42 | |
*** jangutter has joined #openstack-neutron | 10:44 | |
*** jangutter_ has quit IRC | 10:48 | |
openstackgerrit | Flavio Fernandes proposed openstack/neutron master: Rely on worker count for HashRing caching https://review.opendev.org/c/openstack/neutron/+/765874 | 11:01 |
*** dviroel has joined #openstack-neutron | 11:10 | |
*** ociuhandu has quit IRC | 11:14 | |
*** ociuhandu has joined #openstack-neutron | 11:15 | |
openstackgerrit | Mamduh proposed openstack/os-vif stable/queens: Refactor code of linux_net to more cleaner and increase performace https://review.opendev.org/c/openstack/os-vif/+/765941 | 11:17 |
openstackgerrit | Mamduh proposed openstack/os-vif stable/ussuri: Fix - os-vif fails to get the correct UpLink Representor https://review.opendev.org/c/openstack/os-vif/+/765967 | 11:22 |
openstackgerrit | Mamduh proposed openstack/os-vif stable/train: Fix - os-vif fails to get the correct UpLink Representor https://review.opendev.org/c/openstack/os-vif/+/765970 | 11:24 |
*** ociuhandu has quit IRC | 11:25 | |
*** ociuhandu has joined #openstack-neutron | 11:36 | |
openstackgerrit | Mamduh proposed openstack/os-vif stable/stein: Fix - os-vif fails to get the correct UpLink Representor https://review.opendev.org/c/openstack/os-vif/+/765974 | 11:37 |
slaweq | lajoskatona: bcafarel: new CI failure: https://bugs.launchpad.net/neutron/+bug/1907242 | 11:42 |
openstack | Launchpad bug 1907242 in neutron "ERROR: No matching distribution found for wrapt==1.11.* in the CI jobs" [Critical,Confirmed] | 11:42 |
slaweq | :/ | 11:42 |
slaweq | I don't understand why it happens like that in the gate | 11:42 |
slaweq | locally when I tried it works fine for me | 11:42 |
slaweq | :/ | 11:42 |
openstackgerrit | Mamduh proposed openstack/os-vif stable/rocky: Fix - os-vif fails to get the correct UpLink Representor https://review.opendev.org/c/openstack/os-vif/+/765977 | 11:46 |
lajoskatona | slaweq: This morning thought it was just a glitch in some mirror or similar | 11:49 |
slaweq | lajoskatona: I also though that | 11:49 |
slaweq | but if You will check logstash query it seems that it happens on many different providers | 11:50 |
bcafarel | :/ | 11:52 |
*** sapd1 has quit IRC | 11:52 | |
bcafarel | slaweq: just fired it up locally, as you said it passes though I see it picked wrapt==1.12.1 | 11:53 |
lajoskatona | slaweq: gibi pointed to infra where was some discussion (http://eavesdrop.openstack.org/irclogs/%23openstack-infra/latest.log.html) to bump l-c.txt | 11:55 |
openstackgerrit | Mamduh proposed openstack/os-vif stable/queens: Fix - os-vif fails to get the correct UpLink Representor https://review.opendev.org/c/openstack/os-vif/+/765983 | 11:55 |
slaweq | bcafarel: I had exactly same thing | 11:56 |
slaweq | it passed with wrapt 1.12.1 | 11:57 |
slaweq | but when I installed 1.11.1 in the same venv, it was fine too | 11:57 |
*** gcheresh has quit IRC | 11:57 | |
bcafarel | oh so buming l-c.txt may not be the fix then? :( | 11:58 |
*** raildo has joined #openstack-neutron | 11:58 | |
*** tbachman_ has quit IRC | 11:58 | |
*** hamalq has joined #openstack-neutron | 12:01 | |
*** gcheresh has joined #openstack-neutron | 12:03 | |
*** hamalq has quit IRC | 12:05 | |
*** baojg has quit IRC | 12:07 | |
*** baojg has joined #openstack-neutron | 12:07 | |
*** ociuhandu has quit IRC | 12:09 | |
*** ociuhandu has joined #openstack-neutron | 12:25 | |
*** ratailor has quit IRC | 12:29 | |
*** ociuhandu has quit IRC | 12:30 | |
openstackgerrit | Lajos Katona proposed openstack/neutron master: lower-constraints: Bump wrapt to 1.10.11 https://review.opendev.org/c/openstack/neutron/+/765988 | 12:31 |
lajoskatona | slaweq, bcafarel: ^---- let's see if l-c bump has a chance or it's not enough | 12:32 |
slaweq | lajoskatona: thx | 12:40 |
*** sapd1 has joined #openstack-neutron | 12:40 | |
*** ociuhandu has joined #openstack-neutron | 12:40 | |
*** ociuhandu has quit IRC | 12:41 | |
*** ociuhandu has joined #openstack-neutron | 12:45 | |
*** ociuhandu has quit IRC | 12:49 | |
*** ociuhandu has joined #openstack-neutron | 12:58 | |
*** Luzi has joined #openstack-neutron | 13:00 | |
*** ociuhandu has quit IRC | 13:02 | |
*** gcheresh has quit IRC | 13:02 | |
*** ociuhandu has joined #openstack-neutron | 13:02 | |
openstackgerrit | Slawek Kaplonski proposed openstack/neutron master: Add script which can clean patch ports between bridges https://review.opendev.org/c/openstack/neutron/+/765994 | 13:03 |
*** ociuhandu has quit IRC | 13:09 | |
kklimonda_ | If I delete ovn's nb and sb databases, will neutron synchronize everything on restart? | 13:10 |
*** zzzeek has quit IRC | 13:10 | |
*** zzzeek has joined #openstack-neutron | 13:11 | |
numans | kklimonda_, neutron folks can better answer it, but from my previous experience, neutron supports regenerates the ovn nb db in 2 ways. 1. Use neutron-ovn-db-sync-util. 2. Restart neutron with a config option to repair the db. | 13:11 |
*** gcheresh has joined #openstack-neutron | 13:12 | |
numans | kklimonda_, https://opendev.org/openstack/neutron/src/branch/master/neutron/conf/plugins/ml2/drivers/ovn/ovn_conf.py#L86 | 13:12 |
numans | kklimonda_, So even if for some reason, the ovn nb db is inconsistent with the neutron db, you can definitely regenerate it back. Although it may take some time if the deployment has lot of resources. | 13:14 |
kklimonda_ | thanks, I'll have to take a look at the ovn-sync util - I think last time I've tried using it, it bailed out with some weird exceptions. But if one of those options will make full sync then I'll just debug issues if needed | 13:15 |
kklimonda_ | yeah, it's a pretty small deployment for now, and not yet production, so I'm not worried about time it takes to synchronize everything | 13:16 |
*** zzzeek has quit IRC | 13:16 | |
*** zzzeek has joined #openstack-neutron | 13:18 | |
*** spatel has joined #openstack-neutron | 13:19 | |
*** tbachman has joined #openstack-neutron | 13:23 | |
*** zzzeek has quit IRC | 13:26 | |
*** zzzeek has joined #openstack-neutron | 13:28 | |
*** tbachman_ has joined #openstack-neutron | 13:32 | |
*** tbachman has quit IRC | 13:33 | |
*** tbachman_ is now known as tbachman | 13:33 | |
bcafarel | lajoskatona: looks like 1.10.11 did not change lot - maybe try with some 1.12? to see if it changes result | 13:33 |
bcafarel | https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_c4f/765988/1/check/openstack-tox-pep8/c4f89a4/job-output.txt | 13:34 |
lajoskatona | bcafarel: hmmm, I selected 1.10.11 as that seemed to be used in a lot of project's l-c file | 13:35 |
lajoskatona | bcafarel: yeah and I just checked l-c job and pep8 and it failed with 2 different mirrors | 13:36 |
lajoskatona | bcafarel: 1.12 support py38, so can be good | 13:38 |
bcafarel | lajoskatona: yay, going in modern land :) | 13:38 |
*** ociuhandu has joined #openstack-neutron | 13:39 | |
bcafarel | not sure it will be proper fix, but at least if it helps it could make it a bit clearer | 13:39 |
openstackgerrit | Lajos Katona proposed openstack/neutron master: lower-constraints: Bump wrapt to 1.10.11 https://review.opendev.org/c/openstack/neutron/+/765988 | 13:40 |
*** ociuhandu has quit IRC | 13:43 | |
*** obondarev has joined #openstack-neutron | 13:49 | |
*** hemanth_n has quit IRC | 13:50 | |
bcafarel | well it was worth the try... | 13:53 |
*** elvira has joined #openstack-neutron | 13:53 | |
*** ociuhandu has joined #openstack-neutron | 13:55 | |
*** elvira has quit IRC | 13:58 | |
*** ociuhandu has quit IRC | 14:00 | |
bcafarel | pep8 installdeps: -c/home/zuul/src/opendev.org/openstack/requirements/upper-constraints.txt, -r/home/zuul/src/opendev.org/openstack/neutron/requirements.txt, -r/home/zuul/src/opendev.org/openstack/neutron/test-requirements.txt | 14:00 |
bcafarel | hmm changing l-c will probably not impact that one then | 14:00 |
*** elvira has joined #openstack-neutron | 14:01 | |
*** hamalq has joined #openstack-neutron | 14:02 | |
bcafarel | https://github.com/PyCQA/astroid/releases 2.4.0 has "Relax upper bound on wrapt" | 14:02 |
mlavalle | slaweq: no meeting today? | 14:02 |
*** slaweq has quit IRC | 14:03 | |
bcafarel | lajoskatona: slaweq: it may be time to bump astroid then (hoping there is not too many new errors) | 14:03 |
*** slaweq has joined #openstack-neutron | 14:04 | |
*** sapd1 has quit IRC | 14:05 | |
*** tmorin has quit IRC | 14:05 | |
*** hamalq has quit IRC | 14:07 | |
haleyb | bcafarel: i started seeing gate failures last night and never did figure it out for the ovn-provider, strange error like "ERROR: Could not find a version that satisfies the requirement pylint>=2.6.0" - but i know there is a pylint 2.6.0 :-/ | 14:10 |
bcafarel | :/ | 14:10 |
bcafarel | haleyb: I wonder if resolvers in pip changed recently then, at least for astroid it looks similar to our error: https://github.com/PyCQA/astroid/issues/755 | 14:11 |
*** tmorin has joined #openstack-neutron | 14:11 | |
haleyb | bcafarel: i did see a wrapt failure too in a neutron change | 14:12 |
openstackgerrit | Bernard Cafarelli proposed openstack/neutron master: Bump astroid test requirement to 2.4.0 https://review.opendev.org/c/openstack/neutron/+/766000 | 14:12 |
lajoskatona | haleyb: yeah that's in neutron, but other projects has similar issues with other dependencies | 14:12 |
slaweq | haleyb: bcafarel frickler just told me that he was able to reproduce that locally | 14:12 |
slaweq | please check on #opendev channel | 14:13 |
haleyb | ack, i just eventually went to bed figuring it was a mirror problem or something | 14:13 |
*** ociuhandu has joined #openstack-neutron | 14:14 | |
slaweq | haleyb: I don't think it's mirror issue as problem with wrapt was on many different providers | 14:15 |
*** hamalq has joined #openstack-neutron | 14:17 | |
*** ociuhandu has quit IRC | 14:19 | |
openstackgerrit | Merged openstack/neutron-tempest-plugin master: Make neutron-tempest-plugin-bgpvpn-bagpipe voting again https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/761594 | 14:20 |
*** hamalq has quit IRC | 14:22 | |
*** nweinber has joined #openstack-neutron | 14:28 | |
*** ociuhandu has joined #openstack-neutron | 14:30 | |
openstackgerrit | Brian Haley proposed openstack/ovn-octavia-provider master: Fix gate failure https://review.opendev.org/c/openstack/ovn-octavia-provider/+/765872 | 14:35 |
*** tkajinam has quit IRC | 14:47 | |
*** tkajinam has joined #openstack-neutron | 14:47 | |
openstackgerrit | Brian Haley proposed openstack/ovn-octavia-provider master: Fix gate failure https://review.opendev.org/c/openstack/ovn-octavia-provider/+/765872 | 14:49 |
*** ociuhandu has quit IRC | 14:50 | |
openstackgerrit | Bernard Cafarelli proposed openstack/neutron master: Bump astroid test requirement to 2.4.0 https://review.opendev.org/c/openstack/neutron/+/766000 | 14:53 |
openstackgerrit | Brian Haley proposed openstack/ovn-octavia-provider master: Fix gate failure https://review.opendev.org/c/openstack/ovn-octavia-provider/+/765872 | 15:03 |
openstackgerrit | Bernard Cafarelli proposed openstack/neutron master: Bump astroid test requirement to 2.4.0 https://review.opendev.org/c/openstack/neutron/+/766000 | 15:05 |
openstackgerrit | Flavio Fernandes proposed openstack/ovn-octavia-provider master: Remove work around change for test_port_forwarding https://review.opendev.org/c/openstack/ovn-octavia-provider/+/766014 | 15:07 |
openstackgerrit | Flavio Fernandes proposed openstack/ovn-octavia-provider master: Remove work around change for test_port_forwarding https://review.opendev.org/c/openstack/ovn-octavia-provider/+/766014 | 15:13 |
*** obondarev has quit IRC | 15:13 | |
openstackgerrit | Brian Haley proposed openstack/ovn-octavia-provider master: Fix gate failure https://review.opendev.org/c/openstack/ovn-octavia-provider/+/765872 | 15:17 |
*** tkajinam has quit IRC | 15:18 | |
*** ociuhandu has joined #openstack-neutron | 15:21 | |
*** ociuhandu has quit IRC | 15:26 | |
jlibosva | lucasagomes: hi, I removed WIP from https://review.opendev.org/c/openstack/neutron/+/752906 based on our findings yesterday | 15:27 |
*** Luzi has quit IRC | 15:31 | |
lucasagomes | haleyb, ^ | 15:42 |
*** macz_ has joined #openstack-neutron | 15:44 | |
haleyb | lucasagomes: ack, just did a recheck to make sure zuul is still happy | 15:47 |
lucasagomes | sounds good | 15:47 |
lucasagomes | thanks | 15:47 |
jlibosva | thanks | 15:47 |
jlibosva | otherwiseguy: do you think we should still merge https://review.opendev.org/c/openstack/neutron/+/753847 or shall we understand the race first? | 15:48 |
*** macz_ has quit IRC | 15:48 | |
sean-k-mooney | slaweq: by the way if you are checking out that image i dont think it has cloud init so if you boot it you log in with root and no password by default | 15:48 |
otherwiseguy | jlibosva: we don't understand the race? | 15:48 |
*** macz_ has joined #openstack-neutron | 15:48 | |
slaweq | sean-k-mooney: :/ | 15:49 |
slaweq | that's not good | 15:49 |
jlibosva | otherwiseguy: is it related to the hash ring caching? | 15:49 |
sean-k-mooney | slaweq: it wont bring up networking by default so you have to run the inital setup script | 15:49 |
otherwiseguy | jlibosva: shouldn't be. | 15:49 |
slaweq | it would probably require a lot of changes from us | 15:49 |
jlibosva | otherwiseguy: then I shouldn't have used "we" | 15:49 |
sean-k-mooney | slaweq: the plan was to have disimage builder just instlal cloud-init/glean | 15:49 |
sean-k-mooney | which woudl do everythign we need | 15:49 |
slaweq | sean-k-mooney: and for that Your patches are needed | 15:50 |
sean-k-mooney | i was just having issue with the initramfs finding the root disk | 15:50 |
otherwiseguy | jlibosva: this one is that a Row object being passed in and used in multiple threads can be modified by a transaction in one thread, then in the RowEventHandler thread when it is run--after being matched on a condition--could be modified so that it doesn't match that condition before we call run(). | 15:50 |
sean-k-mooney | slaweq: yep again that is more of a long term solution | 15:50 |
slaweq | agree | 15:50 |
otherwiseguy | jlibosva: So when you try to take an action based on some external id changing etc., the row that you pass in and try to grab that value in that you matched on ends up throwing an exception because it isn't there. | 15:51 |
sean-k-mooney | we could use something like ubuntu or fedora but they are boot much much larger and need more ram | 15:51 |
slaweq | sean-k-mooney: and without nested virt it is slow like a hell | 15:51 |
slaweq | :/ | 15:51 |
slaweq | we were using ubuntu in neutron-tempest-plugin some time ago | 15:51 |
otherwiseguy | So this uses an immutable object for passing into the RowEventHandler so that transactions in other threads don't affect it. | 15:51 |
slaweq | and 3h timeout wasn't enough often | 15:52 |
sean-k-mooney | slaweq: ya alpine was built for embeded usecase like network aplinces so it really light weight | 15:52 |
otherwiseguy | jlibosva: Someday, we will get enough fixes in that don't also break CI for CI to be fixed to allow fixes in. :P | 15:52 |
sean-k-mooney | so hopefully would not have that problem | 15:52 |
slaweq | hopefully | 15:53 |
openstackgerrit | Brian Haley proposed openstack/ovn-octavia-provider master: Fix gate failure https://review.opendev.org/c/openstack/ovn-octavia-provider/+/765872 | 15:54 |
jlibosva | otherwiseguy: I think my brain just raised an exception after trying to understand the race and after reading that last sentence | 15:55 |
otherwiseguy | jlibosva: lol. it needs more 'fixes'. Maybe: Someday we will merge enough fixes in that don't need fixing so that we can fix CI to allow fixes in. | 15:56 |
otherwiseguy | jlibosva: I am tired of race conditions. | 15:57 |
otherwiseguy | My brain can't handle too much more of them. | 15:57 |
jlibosva | let's make everything serial | 15:57 |
jlibosva | maybe they won't notice | 15:57 |
otherwiseguy | :D | 15:57 |
* otherwiseguy converts neutron to functional programming style w/ no side effects | 15:58 | |
otherwiseguy | Like db writes :p | 15:58 |
otherwiseguy | Or interacting with users. | 15:58 |
otherwiseguy | I think that would cut down on bugs. | 15:58 |
slaweq | jlibosva: otherwiseguy what an interesting discussion :P | 15:59 |
otherwiseguy | slaweq: I can't be held responsible--my brain is partially melted. | 15:59 |
otherwiseguy | Every time I think I'm almost done getting a patch merged/backported and ready for a customer fix, upstream CI finds a new way to break. | 16:01 |
*** ociuhandu has joined #openstack-neutron | 16:01 | |
jlibosva | otherwiseguy: which is better than customer finding it :) | 16:01 |
jlibosva | otherwiseguy: so back to my original question - you understand the race condition. Do you still want to merge it? It's approved but I'm afraid it's gonna destabilize the CI | 16:02 |
otherwiseguy | jlibosva: I must have missed why it is going to destabilize CI. | 16:03 |
jlibosva | otherwiseguy: it introduces test failures, doesn't it? | 16:04 |
openstackgerrit | Brian Haley proposed openstack/ovn-octavia-provider master: Fix gate failure https://review.opendev.org/c/openstack/ovn-octavia-provider/+/765872 | 16:04 |
sean-k-mooney | slaweq: is there a bug for this by the way | 16:04 |
otherwiseguy | Not to my knowledge. | 16:04 |
otherwiseguy | jlibosva: ^ | 16:04 |
slaweq | otherwiseguy: You should play a bit with migration to new engine facade | 16:04 |
* otherwiseguy looks again | 16:04 | |
slaweq | every time You fix one failing UT other 10 starts failing ;) | 16:04 |
slaweq | sean-k-mooney: nope, I didn't report anything | 16:05 |
openstackgerrit | Slawek Kaplonski proposed openstack/neutron master: Switch to new engine facade for l3_hamode_db leftovers https://review.opendev.org/c/openstack/neutron/+/764920 | 16:05 |
otherwiseguy | jlibosva: I see the test_distributed_lock failure, but that is happening elsewhere too, right? | 16:05 |
sean-k-mooney | ok ill just push a quick patch so and ill file one depening on gibi's reponce | 16:05 |
jlibosva | otherwiseguy: I thought you said with this patch it fails much more often | 16:05 |
otherwiseguy | I reproduce it about 1% of the time without this patch. | 16:05 |
*** ociuhandu has quit IRC | 16:05 | |
otherwiseguy | jlibosva: at one time I thought that it might be causing it somehow, but then I reproduced it without. I've seen it hit a few times. I can see if it happens faster with that patch applied, though. | 16:06 |
jlibosva | otherwiseguy: can or can't ? | 16:07 |
otherwiseguy | We could also rebase it on the patch that supposedly fixes that issue. | 16:07 |
jlibosva | otherwiseguy: which one is that? | 16:08 |
otherwiseguy | I can reproduce it without the frozen row patch. I can also test the patch locally with --until-fail to see if it happens more often with the frozen row patch. | 16:09 |
otherwiseguy | jlibosva: https://review.opendev.org/c/openstack/neutron/+/765874/ | 16:09 |
otherwiseguy | is the thing that should fix test_distributed_lock | 16:10 |
jlibosva | otherwiseguy: you just told me few minutes back it's not related to that :-D | 16:10 |
jlibosva | 16:49:16 jlibosva │ otherwiseguy: is it related to the hash ring caching? | 16:10 |
jlibosva | 16:49:22 otherwiseguy │ jlibosva: shouldn't be. | 16:10 |
otherwiseguy | the frozen row patch isn't related to it. it just happens to be held up by the failing test that that fixes. | 16:11 |
otherwiseguy | at least occasionally held up. | 16:11 |
otherwiseguy | it's hit that failure a couple of times and the rechecks are annoying. | 16:11 |
otherwiseguy | but other patches have hit it as well--it's just unknown whether the frozen row patch hits it more often than other patches. | 16:12 |
otherwiseguy | sorry, this has been very confusing. :p | 16:12 |
otherwiseguy | jlibosva: ^ | 16:13 |
openstackgerrit | Brian Haley proposed openstack/ovn-octavia-provider master: Fix gate failure https://review.opendev.org/c/openstack/ovn-octavia-provider/+/765872 | 16:13 |
zigo | Hi there ! When I add a floating IP to a VM (I'm using floating IP routed over the next hop of a segment), I get this error message in neutron-rpc-server: | 16:14 |
zigo | ERROR oslo_messaging.rpc.server neutron.services.segments.exceptions.HostNotConnectedToAnySegment: Host pub1-compute-6.infomaniak.ch is not connected to any segments on routed provider network '19e9adf5-ab0d-4f4b-badf-e770e002f199'. It should be connected to one. | 16:14 |
zigo | Though my compute node is well on the matching physical network of the segment, so I don't get why I have that. | 16:15 |
zigo | Can anyone give me a clue on where to search ? | 16:15 |
*** hamalq has joined #openstack-neutron | 16:18 | |
*** ociuhandu has joined #openstack-neutron | 16:22 | |
otherwiseguy | jlibosva: maybe my use of the words 'not related' is problematic. More precise would be "The frozen row patch is definitely not causing the race condition." | 16:22 |
openstackgerrit | Bernard Cafarelli proposed openstack/neutron master: Update requirements for recent pip failures https://review.opendev.org/c/openstack/neutron/+/766000 | 16:23 |
*** hamalq has quit IRC | 16:23 | |
jlibosva | otherwiseguy: my concern was to not make the CI stability worse but you said we don't have a proof it makes it fail more often, right? | 16:25 |
otherwiseguy | bcafarel: Is your new upload going to fix the ERROR: Could not find a version that satisfies the requirement PyYAML>=5.3.1 (from bandit) with your astroid/wrapt fix? | 16:25 |
otherwiseguy | jlibosva: we don't. but it should be something that I can check fairly quickly once my devstack vm is rebuilt. | 16:26 |
bcafarel | otherwiseguy: not yet, that one and psutil one I still need to dig where they come from :/ | 16:26 |
otherwiseguy | jlibosva: I'm certainly not opposed to just rebasing it onto the other race condition fix either. | 16:26 |
otherwiseguy | bcafarel++ | 16:26 |
otherwiseguy | not that CI quality could be made worse at this particular moment. :D | 16:28 |
jlibosva | otherwiseguy: rebasing sounds good :) yet another patch to get in before my "after update agent fix" | 16:28 |
jlibosva | lol | 16:28 |
openstackgerrit | Brian Haley proposed openstack/ovn-octavia-provider master: Fix gate failure https://review.opendev.org/c/openstack/ovn-octavia-provider/+/765872 | 16:28 |
openstackgerrit | Bernard Cafarelli proposed openstack/neutron master: Update requirements for recent pip failures https://review.opendev.org/c/openstack/neutron/+/766000 | 16:40 |
bcafarel | slaweq: hopefully psutil should pass with this one ^ now to look at PyYAML | 16:41 |
*** jlibosva has quit IRC | 16:42 | |
openstackgerrit | Brian Haley proposed openstack/ovn-octavia-provider master: Fix gate failure https://review.opendev.org/c/openstack/ovn-octavia-provider/+/765872 | 16:44 |
openstackgerrit | Lajos Katona proposed openstack/neutron master: lower-constraints: Bump wrapt to 1.12.0 https://review.opendev.org/c/openstack/neutron/+/765988 | 16:44 |
*** dulek has quit IRC | 16:44 | |
*** lpetrut has quit IRC | 16:48 | |
*** waleedm has quit IRC | 16:50 | |
*** ociuhandu has quit IRC | 16:51 | |
*** dulek has joined #openstack-neutron | 16:53 | |
*** hamalq has joined #openstack-neutron | 16:56 | |
*** lucasagomes has quit IRC | 17:00 | |
*** bengates has quit IRC | 17:01 | |
*** hamalq has quit IRC | 17:01 | |
*** hamalq has joined #openstack-neutron | 17:06 | |
bcafarel | haleyb: looks like we have similar changes in our WIP patches for that pip issue :) | 17:07 |
haleyb | bcafarel: yes, and i'm just waiting for the lower-constraints job to pass so i can reduce it to just what's needed | 17:08 |
haleyb | then will probably roll into another patch | 17:09 |
haleyb | what a mess | 17:09 |
*** ociuhandu has joined #openstack-neutron | 17:11 | |
*** elvira has quit IRC | 17:13 | |
openstackgerrit | Brian Haley proposed openstack/ovn-octavia-provider master: Fix gate failure https://review.opendev.org/c/openstack/ovn-octavia-provider/+/765872 | 17:18 |
*** dtantsur has joined #openstack-neutron | 17:35 | |
dtantsur | hi folks! is neutron affected by the new pip resolver? | 17:35 |
dtantsur | seeing ut failures on https://review.opendev.org/c/openstack/requirements/+/765785 | 17:35 |
bcafarel | dtantsur: yes :( fixing them in https://review.opendev.org/c/openstack/neutron/+/766000 but it is not at green state yet | 17:37 |
dtantsur | got it, thanks! | 17:38 |
* dtantsur subscribes | 17:38 | |
*** rpittau is now known as rpittau|afk | 17:43 | |
openstackgerrit | Brian Haley proposed openstack/ovn-octavia-provider master: Fix gate failure https://review.opendev.org/c/openstack/ovn-octavia-provider/+/765872 | 17:47 |
*** abdysn has quit IRC | 17:54 | |
*** lajoskatona has quit IRC | 18:00 | |
*** dtantsur is now known as dtantsur|afk | 18:05 | |
*** jangutter_ has joined #openstack-neutron | 18:14 | |
*** jangutter has quit IRC | 18:18 | |
*** jangutter_ has quit IRC | 18:22 | |
*** tesseract has quit IRC | 18:22 | |
*** jangutter has joined #openstack-neutron | 18:23 | |
*** ociuhandu_ has joined #openstack-neutron | 18:24 | |
*** ociuhandu has quit IRC | 18:27 | |
*** ociuhandu_ has quit IRC | 18:28 | |
openstackgerrit | Brian Haley proposed openstack/ovn-octavia-provider master: Fix gate failure https://review.opendev.org/c/openstack/ovn-octavia-provider/+/765872 | 18:38 |
*** jangutter_ has joined #openstack-neutron | 18:40 | |
*** jangutter_ has quit IRC | 18:41 | |
*** jangutter_ has joined #openstack-neutron | 18:42 | |
*** jangutter has quit IRC | 18:42 | |
*** dave-mccowan has joined #openstack-neutron | 18:59 | |
*** dave-mccowan has quit IRC | 19:04 | |
*** luksky has quit IRC | 19:51 | |
openstackgerrit | Bernard Cafarelli proposed openstack/neutron master: Update requirements for recent pip failures https://review.opendev.org/c/openstack/neutron/+/766000 | 19:53 |
openstackgerrit | Bernard Cafarelli proposed openstack/neutron master: Update requirements for recent pip failures https://review.opendev.org/c/openstack/neutron/+/766000 | 20:01 |
*** luksky has joined #openstack-neutron | 20:05 | |
*** jangutter has joined #openstack-neutron | 20:06 | |
*** jangutter_ has quit IRC | 20:09 | |
openstackgerrit | Brian Haley proposed openstack/ovn-octavia-provider master: Fix gate failure https://review.opendev.org/c/openstack/ovn-octavia-provider/+/765872 | 20:14 |
openstackgerrit | Bernard Cafarelli proposed openstack/neutron master: Update requirements for recent pip failures https://review.opendev.org/c/openstack/neutron/+/766000 | 20:15 |
bcafarel | last whack-a-mole round for me tonight ^ | 20:16 |
openstackgerrit | Brian Haley proposed openstack/ovn-octavia-provider master: Fix gate failure https://review.opendev.org/c/openstack/ovn-octavia-provider/+/765872 | 20:22 |
openstackgerrit | Brian Haley proposed openstack/ovn-octavia-provider master: Fix gate failure https://review.opendev.org/c/openstack/ovn-octavia-provider/+/765872 | 20:31 |
openstackgerrit | Brian Haley proposed openstack/ovn-octavia-provider master: Fix gate failure https://review.opendev.org/c/openstack/ovn-octavia-provider/+/765872 | 20:40 |
openstackgerrit | Slawek Kaplonski proposed openstack/neutron master: Switch to new engine facade for IpamPluggableBackend and Ipam driver https://review.opendev.org/c/openstack/neutron/+/662869 | 20:46 |
rm_work | Does python-neutronclient support mTLS (providing a client cert/key)? | 21:12 |
rm_work | it seems with HTTPClient it does not: https://github.com/openstack/python-neutronclient/blob/master/neutronclient/client.py#L419-L435 (no cert passed through) | 21:12 |
rm_work | but maybe via the SessionClient? | 21:12 |
*** gcheresh has quit IRC | 21:12 | |
rm_work | ah, looks like the answer is yes | 21:20 |
bcafarel | haleyb++ thanks for pointer I will take a look tomorrow at that enum fix in OSC | 21:24 |
haleyb | bcafarel: ack. have you found a way to test things locally? I always get success but zuul does not agree | 21:25 |
bcafarel | haleyb: yes I was running the commands from http://lists.openstack.org/pipermail/openstack-discuss/2020-December/019285.html | 21:25 |
bcafarel | iterations take time now that it goes further :( but at least it is much farther than wiating for CI | 21:26 |
haleyb | bcafarel: thanks, i'll see if that works | 21:29 |
*** tmorin has quit IRC | 21:31 | |
*** jakeyip has quit IRC | 21:33 | |
*** mattia_ has joined #openstack-neutron | 21:41 | |
*** mattia has quit IRC | 21:42 | |
*** sorrison has joined #openstack-neutron | 21:51 | |
*** ebbex has quit IRC | 21:55 | |
*** ebbex has joined #openstack-neutron | 21:55 | |
*** rcernin has joined #openstack-neutron | 21:59 | |
*** luksky has quit IRC | 22:18 | |
*** spatel has quit IRC | 22:19 | |
*** luksky has joined #openstack-neutron | 22:32 | |
openstackgerrit | Bernard Cafarelli proposed openstack/neutron master: Update requirements for recent pip failures https://review.opendev.org/c/openstack/neutron/+/766000 | 22:34 |
openstackgerrit | Brian Haley proposed openstack/ovn-octavia-provider master: Fix gate failure https://review.opendev.org/c/openstack/ovn-octavia-provider/+/765872 | 22:45 |
*** admin0 has joined #openstack-neutron | 22:45 | |
admin0 | hi all .. can you let me know what is causing this ? https://gist.githubusercontent.com/a1git/851649fa6f76ebd6e4782f3d1c707501/raw/8970bdd6d4367df81474166f9aaed35c07c52013/gistfile1.txt | 22:45 |
admin0 | this actually stops the linuxbrige-agent, without which nova cannot spawn any new instances | 22:46 |
openstackgerrit | Bernard Cafarelli proposed openstack/neutron master: Update requirements for recent pip failures https://review.opendev.org/c/openstack/neutron/+/766000 | 22:52 |
*** tkajinam has joined #openstack-neutron | 23:01 | |
*** nweinber has quit IRC | 23:32 | |
*** hoonetorg has quit IRC | 23:36 | |
*** gthiemonge has quit IRC | 23:40 | |
*** hoonetorg has joined #openstack-neutron | 23:40 | |
*** gthiemonge has joined #openstack-neutron | 23:43 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!