*** tbachman has joined #openstack-neutron | 00:06 | |
*** dayou_ has joined #openstack-neutron | 00:09 | |
*** slaweq has joined #openstack-neutron | 00:11 | |
*** slaweq has quit IRC | 00:16 | |
*** yamamoto has joined #openstack-neutron | 00:24 | |
abaindur | kevinbenton: you around? had questions about this new resource cache used in ovs agent | 00:25 |
---|---|---|
openstackgerrit | Merged openstack/networking-sfc master: use setup_extension in unit tests https://review.openstack.org/595333 | 00:26 |
abaindur | we had a port binding failed. The port was plugged/created and OVS agent discovered the port, in treat_devices_added_or_updated. But get_device_details determined that "if not port_obj.binding_levels:" and therefore the port was not bound | 00:27 |
abaindur | after this is pretty much did no further processing on the port... and the vif plugging timed out and it was deleted | 00:27 |
abaindur | I noticed new in pike, there is no RPC call made to query the device details, instead it retrieved it from some cache: | 00:28 |
abaindur | port_obj = self.remote_resource_cache.get_resource_by_id | 00:28 |
*** longkb has joined #openstack-neutron | 00:29 | |
*** yamamoto has quit IRC | 00:34 | |
*** yamahata__ has joined #openstack-neutron | 00:38 | |
*** isq_ has quit IRC | 00:39 | |
*** mriedem_away has quit IRC | 00:40 | |
*** ruijing1 has quit IRC | 00:42 | |
*** cshen has joined #openstack-neutron | 00:43 | |
*** cshen has quit IRC | 00:48 | |
*** toshiiw has joined #openstack-neutron | 01:03 | |
*** imacdonn has quit IRC | 01:15 | |
*** imacdonn has joined #openstack-neutron | 01:16 | |
*** yamahata has quit IRC | 01:42 | |
*** iyamahat_ has quit IRC | 01:42 | |
*** Dinesh_Bhor has joined #openstack-neutron | 01:50 | |
*** masakif has joined #openstack-neutron | 01:54 | |
*** hongbin has joined #openstack-neutron | 01:58 | |
*** lujinluo has quit IRC | 02:03 | |
*** slaweq has joined #openstack-neutron | 02:11 | |
*** slaweq has quit IRC | 02:16 | |
*** Dinesh_Bhor has quit IRC | 02:26 | |
openstackgerrit | LIU Yulong proposed openstack/neutron master: Install centralized floating IP nat rules to all ha nodes https://review.openstack.org/604094 | 02:26 |
*** errantekarmico has quit IRC | 02:34 | |
*** yamamoto has joined #openstack-neutron | 02:36 | |
*** Dinesh_Bhor has joined #openstack-neutron | 02:38 | |
*** cshen has joined #openstack-neutron | 02:44 | |
*** hoonetorg has quit IRC | 02:48 | |
*** cshen has quit IRC | 02:49 | |
*** gvrangan has joined #openstack-neutron | 02:50 | |
*** yamamoto has quit IRC | 02:52 | |
*** Bhujay has joined #openstack-neutron | 03:01 | |
*** hoonetorg has joined #openstack-neutron | 03:01 | |
*** errantekarmico has joined #openstack-neutron | 03:10 | |
*** abaindur has quit IRC | 03:25 | |
*** gcb_ has joined #openstack-neutron | 03:27 | |
*** moshele has joined #openstack-neutron | 03:42 | |
openstackgerrit | Kim Bao Long proposed openstack/neutron master: Introduce L3 agent extension for SNAT logging https://review.openstack.org/577086 | 03:47 |
*** rcernin has quit IRC | 03:48 | |
*** rcernin has joined #openstack-neutron | 03:49 | |
*** carl_baldwin has quit IRC | 03:56 | |
*** edmondsw has quit IRC | 03:56 | |
*** Bhujay has quit IRC | 03:58 | |
*** moshele has quit IRC | 03:58 | |
*** Dinesh_Bhor has quit IRC | 03:58 | |
*** gvrangan has quit IRC | 04:06 | |
*** ramishra has joined #openstack-neutron | 04:07 | |
*** sambetts|afk has quit IRC | 04:08 | |
*** sambetts_ has joined #openstack-neutron | 04:10 | |
*** slaweq has joined #openstack-neutron | 04:11 | |
*** armax has quit IRC | 04:12 | |
*** slaweq has quit IRC | 04:15 | |
openstackgerrit | Nguyen Phuong An proposed openstack/neutron-specs master: [WIP] First draft of L7 filtering extension https://review.openstack.org/600714 | 04:26 |
*** Dinesh_Bhor has joined #openstack-neutron | 04:34 | |
*** cshen has joined #openstack-neutron | 04:43 | |
*** hongbin has quit IRC | 04:45 | |
*** cshen has quit IRC | 04:47 | |
*** yamamoto has joined #openstack-neutron | 04:52 | |
*** armax has joined #openstack-neutron | 05:06 | |
*** yamamoto has quit IRC | 05:08 | |
*** ratailor has joined #openstack-neutron | 05:11 | |
*** slaweq has joined #openstack-neutron | 05:11 | |
*** Dinesh_Bhor has quit IRC | 05:12 | |
*** slaweq has quit IRC | 05:15 | |
*** Dinesh_Bhor has joined #openstack-neutron | 05:16 | |
*** armax has quit IRC | 05:17 | |
*** Bhujay has joined #openstack-neutron | 05:18 | |
openstackgerrit | Swaminathan Vasudevan proposed openstack/neutron master: Fix connection between 2 dvr routers https://review.openstack.org/597567 | 05:21 |
*** hoangcx has quit IRC | 05:21 | |
*** longkb has quit IRC | 05:21 | |
*** hoangcx has joined #openstack-neutron | 05:21 | |
*** longkb has joined #openstack-neutron | 05:21 | |
*** yamamoto has joined #openstack-neutron | 05:29 | |
*** hoangcx has quit IRC | 05:32 | |
*** hoangcx has joined #openstack-neutron | 05:32 | |
*** iyamahat has joined #openstack-neutron | 05:34 | |
*** ijw has joined #openstack-neutron | 05:42 | |
*** ijw has quit IRC | 05:46 | |
*** sridharg has joined #openstack-neutron | 05:47 | |
*** abaindur has joined #openstack-neutron | 05:48 | |
*** masakif has quit IRC | 05:49 | |
*** yamahata has joined #openstack-neutron | 05:53 | |
*** abaindur has quit IRC | 05:54 | |
*** lajoskatona has joined #openstack-neutron | 05:58 | |
*** abaindur has joined #openstack-neutron | 06:02 | |
*** janki has joined #openstack-neutron | 06:04 | |
*** ijw has joined #openstack-neutron | 06:07 | |
*** armax has joined #openstack-neutron | 06:10 | |
*** nplanel__ has joined #openstack-neutron | 06:10 | |
*** ijw has quit IRC | 06:12 | |
*** gkadam has joined #openstack-neutron | 06:13 | |
*** gkadam has quit IRC | 06:13 | |
*** ksambor has joined #openstack-neutron | 06:23 | |
*** cshen has joined #openstack-neutron | 06:27 | |
*** aojea has joined #openstack-neutron | 06:28 | |
*** slaweq has joined #openstack-neutron | 06:34 | |
*** armax has quit IRC | 06:40 | |
*** ijw has joined #openstack-neutron | 06:43 | |
*** ijw has quit IRC | 06:48 | |
*** sambetts_ has quit IRC | 06:53 | |
*** sambetts_ has joined #openstack-neutron | 06:55 | |
*** gvrangan has joined #openstack-neutron | 06:57 | |
*** rcernin has quit IRC | 07:02 | |
*** tssurya has joined #openstack-neutron | 07:04 | |
*** ijw has joined #openstack-neutron | 07:07 | |
openstackgerrit | Merged openstack/neutron master: Use templates for cover and lower-constraints https://review.openstack.org/600762 | 07:11 |
*** ijw has quit IRC | 07:12 | |
*** mosulica has joined #openstack-neutron | 07:15 | |
openstackgerrit | Merged openstack/neutron master: Use Tempest slow job to run all slow tests https://review.openstack.org/583847 | 07:17 |
*** yamamoto has quit IRC | 07:19 | |
*** gkadam has joined #openstack-neutron | 07:19 | |
*** longkb has quit IRC | 07:22 | |
*** mattgo has joined #openstack-neutron | 07:23 | |
*** jlibosva has joined #openstack-neutron | 07:23 | |
*** longkb has joined #openstack-neutron | 07:24 | |
*** lajoskatona has quit IRC | 07:28 | |
openstackgerrit | Slawek Kaplonski proposed openstack/neutron-tempest-plugin master: WIP Migrate dvr multinode scenario to zuulv3 syntax https://review.openstack.org/578796 | 07:32 |
*** rkukura has quit IRC | 07:34 | |
*** rkukura has joined #openstack-neutron | 07:34 | |
*** lajoskatona has joined #openstack-neutron | 07:43 | |
*** jpena|off is now known as jpena | 07:48 | |
*** ralonsoh has joined #openstack-neutron | 07:50 | |
*** janki has quit IRC | 07:53 | |
openstackgerrit | LIU Yulong proposed openstack/neutron master: Install centralized floating IP nat rules to all ha nodes https://review.openstack.org/604094 | 07:57 |
*** yamamoto has joined #openstack-neutron | 07:58 | |
*** masakif has joined #openstack-neutron | 08:00 | |
*** yamamoto has quit IRC | 08:04 | |
*** yamamoto has joined #openstack-neutron | 08:04 | |
*** dims has quit IRC | 08:11 | |
*** dims has joined #openstack-neutron | 08:12 | |
*** tssurya has quit IRC | 08:16 | |
*** gcb_ has quit IRC | 08:18 | |
*** abaindur has quit IRC | 08:23 | |
*** cshen has quit IRC | 08:24 | |
*** s10 has joined #openstack-neutron | 08:33 | |
openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add abstract OVSDB API https://review.openstack.org/476612 | 08:35 |
*** Dinesh_Bhor has quit IRC | 08:37 | |
openstackgerrit | Merged openstack/ovsdbapp master: Make nested transaction thread aware https://review.openstack.org/604029 | 08:44 |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ovsdbapp master: Move ovsdbapp jobs to its tree https://review.openstack.org/604102 | 08:45 |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ovsdbapp master: Move ovsdbapp jobs to its tree https://review.openstack.org/604102 | 08:46 |
*** Dinesh_Bhor has joined #openstack-neutron | 08:55 | |
lucasagomes | dalvarez, https://review.openstack.org/#/c/604307/ | 08:57 |
jlibosva | lucasagomes: I see you did backports of da awesome fix, does it work with the old eventlet? I plan to test it today | 08:58 |
*** ramishra has quit IRC | 08:59 | |
*** sapd1 has quit IRC | 09:00 | |
*** sapd1_ has joined #openstack-neutron | 09:01 | |
dalvarez | lucasagomes: ack! thanks | 09:01 |
dalvarez | jlibosva: i didn't | 09:01 |
dalvarez | did i? | 09:01 |
*** masakif has quit IRC | 09:02 | |
dalvarez | jlibosva: lucasagomes why not 0.12.2? | 09:02 |
dalvarez | jlibosva: lucasagomes i'd love to have it in rocky... | 09:02 |
jlibosva | dalvarez: if it works with the rocky eventlet | 09:02 |
dalvarez | jlibosva: yes... if not it'd be awesome to come up with somethin | 09:03 |
*** ramishra has joined #openstack-neutron | 09:06 | |
*** lajoskatona has quit IRC | 09:06 | |
*** hoonetorg has quit IRC | 09:08 | |
*** salmankhan has joined #openstack-neutron | 09:10 | |
lucasagomes | dalvarez, well we can do a 0.12.2 | 09:10 |
lucasagomes | dalvarez, since it was a new stein I thought bough bumping it to 0.13.0 | 09:10 |
lucasagomes | dalvarez, want to change it ? | 09:10 |
dalvarez | lucasagomes: nono, sounds good | 09:11 |
*** salmankhan has quit IRC | 09:13 | |
*** tssurya has joined #openstack-neutron | 09:15 | |
*** Dinesh_Bhor has quit IRC | 09:16 | |
*** toshiiw has quit IRC | 09:16 | |
lucasagomes | dalvarez, that patch for releases will push it to pypi btw | 09:16 |
*** salmankhan has joined #openstack-neutron | 09:17 | |
dalvarez | oh nice didnt know that | 09:18 |
dalvarez | lucasagomes++ | 09:18 |
lucasagomes | o/ palmeiras | 09:18 |
openstackgerrit | Merged openstack/neutron master: Parse dhcp leases file in a more robust way https://review.openstack.org/595235 | 09:20 |
openstackgerrit | Jakub Libosvar proposed openstack/ovsdbapp master: ut: Use monkey_patch when running with eventlet https://review.openstack.org/604310 | 09:21 |
jlibosva | lucasagomes: dalvarez ^^ | 09:21 |
dalvarez | yeah we need ittt | 09:21 |
jlibosva | dalvarez: lucasagomes the only thing I worry there is that when running the test on a system with eventlet, it will monkey patch somewhere in the middle of test suite. So some tests with run with patched stuff while others won't | 09:23 |
jlibosva | so maybe, the monkey_patch should go somewhere higher to the base class? | 09:23 |
dalvarez | jlibosva: __init__ there? | 09:23 |
jlibosva | if available | 09:23 |
jlibosva | dalvarez: where? | 09:23 |
dalvarez | jlibosva: i mean | 09:23 |
dalvarez | jlibosva: creating __init__.py | 09:24 |
dalvarez | with the monkey patch | 09:24 |
jlibosva | dalvarez: how would that help? | 09:24 |
jlibosva | dalvarez: the monkey patching would remain global even if it's in the __init__, wouldn't it? | 09:25 |
dalvarez | jlibosva: yes this is what we want so that some tests dont run with patched stuff and others don't | 09:25 |
dalvarez | no? | 09:25 |
*** hoonetorg has joined #openstack-neutron | 09:25 | |
dalvarez | no? | 09:25 |
dalvarez | jlibosva: no? | 09:25 |
jlibosva | dalvarez: I probably miss something. If you patch it in the __init__ or setUp, from that point on all other modules will be patched, won't they? | 09:26 |
dalvarez | lucasagomes: why abandoned the backport? | 09:26 |
dalvarez | jlibosva: yes | 09:26 |
lucasagomes | dalvarez, thought wasn't needed | 09:26 |
dalvarez | lucasagomes: yeah restoring | 09:26 |
jlibosva | dalvarez: we'd need something like "unpatch" | 09:26 |
lucasagomes | dalvarez, ok | 09:26 |
dalvarez | jlibosva: why? we want it to be patched all the time no? | 09:26 |
jlibosva | or run all tests with eventlet, if it's present - that should be fine thing to do I guess | 09:26 |
*** lajoskatona has joined #openstack-neutron | 09:27 | |
dalvarez | jlibosva: yeah im ok with that | 09:28 |
dalvarez | if present | 09:28 |
dalvarez | jlibosva: but im ok with this approach better | 09:29 |
dalvarez | jlibosva: as this is actually testing the greenthread stuff so this is where we only require the monkey patching | 09:29 |
dalvarez | the rest shouldnt mess up wth eventlet | 09:29 |
dalvarez | plus it's not a dep | 09:30 |
*** sambetts_ is now known as sambetts | 09:31 | |
openstackgerrit | Jakub Libosvar proposed openstack/ovsdbapp master: ut: Use monkey_patch when running with eventlet https://review.openstack.org/604313 | 09:31 |
jlibosva | dalvarez: ^^ | 09:31 |
dalvarez | nono | 09:32 |
dalvarez | :P | 09:32 |
dalvarez | jlibosva: do we want that better? | 09:32 |
*** cshen has joined #openstack-neutron | 09:32 | |
jlibosva | dalvarez: how? | 09:32 |
dalvarez | jlibosva: just run that particular test patching | 09:32 |
jlibosva | dalvarez: how? | 09:32 |
dalvarez | ok no way to unpatch right? | 09:32 |
jlibosva | dalvarez: no | 09:32 |
dalvarez | ok | 09:32 |
dalvarez | damn | 09:32 |
dalvarez | ok | 09:32 |
*** lajoskatona has quit IRC | 09:33 | |
openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add abstract OVSDB API https://review.openstack.org/476612 | 09:33 |
dalvarez | jlibosva: question | 09:35 |
dalvarez | jlibosva: what if we have a test_with_eventlet.py there | 09:35 |
dalvarez | which patches on setUp.py | 09:35 |
dalvarez | next tests in other files will run with patched stuff? | 09:35 |
jlibosva | dalvarez: yes | 09:35 |
dalvarez | jlibosva: i dont like that | 09:35 |
dalvarez | so all or nothing | 09:35 |
jlibosva | dalvarez: we'd need to make sure no other tests are run after the monkey patch | 09:35 |
jlibosva | that's exactly what the eventlet best practices say | 09:36 |
dalvarez | jlibosva: not possible right? or perhaps change the in tox | 09:36 |
dalvarez | the tox file | 09:36 |
dalvarez | to run a separate suite | 09:36 |
jlibosva | "When using eventlet.monkey_patch, do it first or not at all." | 09:36 |
dalvarez | right | 09:37 |
dalvarez | jlibosva: i left a comment in gerrit | 09:37 |
*** cshen has quit IRC | 09:38 | |
jlibosva | dalvarez: you mean blacklisting the "race" UT and run it only in another tox target? | 09:39 |
lucasagomes | jlibosva, dalvarez for the test sake do we need to run with it with eventlet ? | 09:44 |
jlibosva | lucasagomes: that was the cause of the ovn failures | 09:46 |
jlibosva | lucasagomes: on the gate it runs without eventlet | 09:46 |
jlibosva | we could make it conditional, if you want to test with eventlet | 09:47 |
jlibosva | via env variable | 09:47 |
*** hoangcx has quit IRC | 09:48 | |
*** lajoskatona has joined #openstack-neutron | 09:48 | |
openstackgerrit | Lajos Katona proposed openstack/neutron master: supported_vnic_type configurable for sriov and ovs https://review.openstack.org/601600 | 09:49 |
*** lajoskatona has quit IRC | 09:53 | |
dalvarez | jlibosva: my concern is mainly due to debugging issues | 09:53 |
dalvarez | but... | 09:53 |
*** tssurya has quit IRC | 09:54 | |
jlibosva | dalvarez: yeah, so you'd need to say explicitly that you want to monkey patch the modules | 09:54 |
*** cshen has joined #openstack-neutron | 09:54 | |
dalvarez | yeah | 09:55 |
dalvarez | cool | 09:55 |
dalvarez | jlibosva: so two jobs in the gate would make sense? | 09:55 |
jlibosva | dalvarez: I'd defer from multiplying jobs | 09:56 |
jlibosva | dalvarez: just for local testing - or we could just remove the eventlet thingy :-/ | 09:56 |
*** Dinesh_Bhor has joined #openstack-neutron | 09:56 | |
dalvarez | jlibosva: ok then i'd write some warning if the import succeeds | 09:56 |
dalvarez | or remove it yeah... idk | 09:57 |
jlibosva | dalvarez: as functional tests don't run with eventlet too. but ovsdbapp does both with neutron and ovn | 09:57 |
dalvarez | jlibosva: ok i have sum idea | 09:57 |
dalvarez | ah no we are moving the ovsdbapp tests to their tree | 09:57 |
dalvarez | jlibosva: does it make more sense to move the test somewhere in ovn/neutron? | 09:57 |
jlibosva | dalvarez: I needed eventlet to test the thing we saw with OVN, to see whether it works | 09:57 |
jlibosva | dalvarez: no | 09:58 |
jlibosva | dalvarez: it's strictly ovsdbapp | 09:58 |
dalvarez | right | 09:58 |
dalvarez | i think it's ok to test how ovsdbapp behaves with greenthreads | 09:58 |
dalvarez | but should be isolated | 09:58 |
jlibosva | dalvarez: I have another dirty idea | 09:59 |
dalvarez | jlibosva: https://emptysqua.re/blog/undoing-gevents-monkey-patching/ | 09:59 |
dalvarez | perhaps we could spawn a separate process there | 09:59 |
dalvarez | patch | 09:59 |
dalvarez | run | 09:59 |
dalvarez | return | 10:00 |
jlibosva | dalvarez: weird, that site didn't work for me when I found it | 10:00 |
dalvarez | lemme try that | 10:00 |
dalvarez | that'd be awesome | 10:00 |
dalvarez | jlibosva: | 10:00 |
dalvarez | that'd be awesome | 10:00 |
dalvarez | lemme try | 10:00 |
dalvarez | jlibosva: | 10:00 |
dalvarez | jlibosva: that site says that the multiprocessing doesn't cope well with windows though | 10:01 |
openstackgerrit | Manuel Buil proposed openstack/networking-odl master: Use extras for ceilometer dependency https://review.openstack.org/602622 | 10:02 |
jlibosva | dalvarez: I wanted to use eventlet.sleep when eventlet is imported. But it doesn't work without the monkey_patching | 10:06 |
openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add abstract OVSDB API https://review.openstack.org/476612 | 10:13 |
*** gokhani has quit IRC | 10:19 | |
*** Dinesh_Bhor has quit IRC | 10:20 | |
openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add abstract OVSDB API https://review.openstack.org/476612 | 10:26 |
ajo | lucasagomes: I guess we need to backport to rocky, right: https://review.openstack.org/#/c/602632/1 . ? | 10:28 |
lucasagomes | ajo, lemme confirm cause when I backported the patch which consolidate the transaction to rocky I fixed it there | 10:29 |
ajo | lucasagomes: ah, ack if that's the case please comment on that review, thanks for checking :D | 10:29 |
ajo | lucasagomes: It just came to my mind when I received a notification email :D | 10:29 |
lucasagomes | ajo, yeah rocky is covered :D | 10:30 |
ajo | awesome lucasagomes++ | 10:30 |
lucasagomes | ajo, will comment on the review | 10:30 |
ajo | lucasagomes: I see we're going python3!, thanks for handling that too :D | 10:30 |
lucasagomes | ajo, yeah! We already have a job in the gate runinng with python3 | 10:30 |
lucasagomes | and we soon are changing it to default | 10:30 |
lucasagomes | o/ | 10:30 |
*** lajoskatona has joined #openstack-neutron | 10:32 | |
lucasagomes | commented | 10:33 |
dalvarez | jlibosva: | 10:34 |
dalvarez | lucasagomes: | 10:34 |
dalvarez | hola | 10:34 |
lucasagomes | dalvarez, o/ | 10:34 |
openstackgerrit | Daniel Alvarez proposed openstack/ovsdbapp master: ut: monkey patch threading test if eventlet is installed https://review.openstack.org/604326 | 10:34 |
dalvarez | https://review.openstack.org/#/c/604326/ | 10:34 |
dalvarez | ^ | 10:34 |
dalvarez | lucasagomes: jlibosva ^ see if this makes more sense | 10:34 |
jlibosva | dalvarez: no, you need to do the eventlet.monkey_patch() before any other imports | 10:35 |
dalvarez | jlibosva: why? | 10:35 |
dalvarez | jlibosva: im just confining the patching to time and thread modules | 10:36 |
jlibosva | dalvarez: to avoid unexpected behavior | 10:36 |
dalvarez | like what? | 10:36 |
dalvarez | it's confined to that particular test only | 10:36 |
dalvarez | i understand it's a recommendation but i'd say that this is a "controlled environment" | 10:36 |
jlibosva | dalvarez: https://specs.openstack.org/openstack/openstack-specs/specs/eventlet-best-practices.html#monkey-patching | 10:36 |
dalvarez | jlibosva: yes yes | 10:37 |
dalvarez | i understand it's best practices | 10:37 |
dalvarez | and recommended | 10:37 |
dalvarez | but this is a test :) | 10:37 |
dalvarez | so we're controlling the environment | 10:37 |
lucasagomes | dalvarez, looks like a big hack :-P | 10:43 |
openstackgerrit | Daniel Alvarez proposed openstack/ovsdbapp master: ut: monkey patch threading test if eventlet is installed https://review.openstack.org/604326 | 10:44 |
dalvarez | lucasagomes: hehe it kinda is | 10:44 |
*** tbachman has quit IRC | 10:44 | |
dalvarez | but not sure if we have other options | 10:44 |
lucasagomes | s/kinda/totally :-P | 10:44 |
dalvarez | i think it's good to test that out | 10:44 |
lucasagomes | yeah, an option (you may not link) is to make nesting txn's optional | 10:45 |
dalvarez | lucasagomes: lol so? eventlet hacks too by patching | 10:45 |
dalvarez | lucasagomes: yeah i'm ok with that too but that's a major change and everything out there using ovsdbapp may be broken | 10:46 |
dalvarez | may get* | 10:46 |
dalvarez | oh well | 10:46 |
dalvarez | if by default they are nested then it shouldnt break | 10:46 |
jlibosva | dalvarez: aha, I see. That's a great idea | 10:46 |
*** cshen has quit IRC | 10:46 | |
dalvarez | jlibosva: thanks buddy | 10:47 |
dalvarez | i mean... i understand it's a bit hacky | 10:47 |
lucasagomes | dalvarez, yeah | 10:47 |
jlibosva | dalvarez: I'm reading eventlet code to see if we're on the safe side | 10:47 |
jlibosva | dalvarez: perhaps we could avoid the monkey patching at all | 10:48 |
dalvarez | jlibosva: can we? | 10:49 |
lucasagomes | dalvarez, ajo otherwiseguy https://review.openstack.org/#/c/604102/ (see the new tempest job ovsdbapp-tempest-dsvm-networking-ovn-ovs-release is now green) | 10:50 |
jlibosva | dalvarez: I assume we can, if we use the eventlet methods instead | 10:50 |
dalvarez | lucasagomes: no tempest logs there? :S | 10:51 |
lucasagomes | dalvarez, it has | 10:51 |
dalvarez | lucasagomes: legacy-tempest-dsvm-networking-ovn-ovs-release-ovsdbapp-src | 10:51 |
dalvarez | this is red | 10:51 |
lucasagomes | dalvarez, that will be removed | 10:51 |
dalvarez | jlibosva: oh k | 10:51 |
dalvarez | ahh | 10:51 |
dalvarez | lucasagomes: cool ! | 10:51 |
lucasagomes | dalvarez, that's set in the openstack-zuul-jobs | 10:51 |
dalvarez | yeah im rechecking also my patch | 10:52 |
dalvarez | lucasagomes: we hit the port status error | 10:52 |
lucasagomes | dalvarez, but we need to merge that patch first prior to removing it from there | 10:52 |
dalvarez | but that's different from the race conditions | 10:52 |
dalvarez | got it | 10:52 |
lucasagomes | dalvarez, ack | 10:52 |
lucasagomes | dalvarez, https://review.openstack.org/#/c/604114/ | 10:52 |
lucasagomes | it's failing yet cause of the project-config patch as well (it's a ball of wax) | 10:52 |
dalvarez | lol | 10:54 |
*** annp has quit IRC | 11:00 | |
*** ijw has joined #openstack-neutron | 11:07 | |
*** ijw has quit IRC | 11:12 | |
*** longkb has quit IRC | 11:12 | |
*** cshen has joined #openstack-neutron | 11:13 | |
jlibosva | dalvarez: I made yet another patch :) | 11:21 |
openstackgerrit | Merged openstack/networking-ovn master: Default tempest jobs to Python3 https://review.openstack.org/602348 | 11:21 |
openstackgerrit | Jakub Libosvar proposed openstack/ovsdbapp master: ut: Patch get_ident for race transaction test https://review.openstack.org/604331 | 11:22 |
jlibosva | dalvarez: ^^ this one | 11:22 |
*** jpena is now known as jpena|lunch | 11:23 | |
jlibosva | dalvarez: it doesn't rely on monkey_patching, patches only the used get_ident() | 11:23 |
*** janki has joined #openstack-neutron | 11:28 | |
*** gvrangan has quit IRC | 11:30 | |
*** yamamoto has quit IRC | 11:35 | |
*** lajoskatona has quit IRC | 11:41 | |
dalvarez | jlibosva: much better | 11:44 |
dalvarez | let me abandon mine | 11:44 |
jlibosva | dalvarez: I abandon the other too of mine two | 11:46 |
dalvarez | jlibosva: i feel sad for abandoning a patch | 11:46 |
dalvarez | but i can't take care of it anymore | 11:46 |
dalvarez | i hope it has a better life from now on | 11:46 |
*** salmankhan1 has joined #openstack-neutron | 11:52 | |
*** Luzi has joined #openstack-neutron | 11:53 | |
Luzi | hi, i have a question concerning security groups. Is this the right channel to ask? | 11:54 |
*** salmankhan has quit IRC | 11:54 | |
*** salmankhan1 is now known as salmankhan | 11:54 | |
*** yamamoto has joined #openstack-neutron | 11:56 | |
*** lajoskatona has joined #openstack-neutron | 11:59 | |
Luzi | is it a desired behavior, that, when you create a rule for a security-group without any further arguments, it adds a rule, which allows ALL incoming tcp-connections? | 12:00 |
openstackgerrit | Manuel Buil proposed openstack/networking-bgpvpn master: Use extras for horizon dependency https://review.openstack.org/604335 | 12:00 |
*** jlibosva has quit IRC | 12:08 | |
*** jlibosva has joined #openstack-neutron | 12:08 | |
*** tssurya has joined #openstack-neutron | 12:10 | |
*** tbachman has joined #openstack-neutron | 12:15 | |
*** gibi is now known as giblet | 12:17 | |
*** tssurya has quit IRC | 12:21 | |
*** jpena|lunch is now known as jpena | 12:26 | |
*** boden has joined #openstack-neutron | 12:26 | |
*** tbachman has quit IRC | 12:32 | |
*** gkadam has quit IRC | 12:34 | |
*** ratailor has quit IRC | 12:34 | |
*** salmankhan1 has joined #openstack-neutron | 12:40 | |
*** salmankhan has quit IRC | 12:41 | |
*** salmankhan1 is now known as salmankhan | 12:41 | |
*** aniketh has joined #openstack-neutron | 12:45 | |
*** panda|off is now known as panda | 12:53 | |
*** tbachman has joined #openstack-neutron | 13:00 | |
*** amuller has joined #openstack-neutron | 13:00 | |
*** liuyulong has joined #openstack-neutron | 13:02 | |
*** tmorin has joined #openstack-neutron | 13:07 | |
*** ramishra has quit IRC | 13:13 | |
*** yamamoto has quit IRC | 13:18 | |
*** yamamoto has joined #openstack-neutron | 13:18 | |
*** tidwellr has joined #openstack-neutron | 13:27 | |
*** mriedem has joined #openstack-neutron | 13:29 | |
*** jistr is now known as jistr|call | 13:31 | |
*** tidwellr has quit IRC | 13:32 | |
*** s10 has quit IRC | 13:32 | |
*** sridharg has quit IRC | 13:33 | |
*** dansmith is now known as SteelyDan | 13:50 | |
*** errantekarmico has quit IRC | 13:51 | |
*** yamamoto has quit IRC | 13:52 | |
openstackgerrit | Daniel Alvarez proposed openstack/networking-ovn master: WIP: Fix ovn db sync with port groups https://review.openstack.org/599426 | 14:01 |
tmorin | hi haleyb: I'd like to finish fixing bug 1789878 ... could you have a look at stable/queens change https://review.openstack.org/602016 , please ? | 14:04 |
openstack | bug 1789878 in neutron "bgpvpn router fallback broken by change in neutron openvswitch firewall" [High,In progress] https://launchpad.net/bugs/1789878 - Assigned to Nguyen Phuong An (annp) | 14:04 |
*** tidwellr has joined #openstack-neutron | 14:05 | |
*** openstackgerrit has quit IRC | 14:07 | |
*** mchlumsky has quit IRC | 14:09 | |
*** mchlumsky has joined #openstack-neutron | 14:18 | |
*** lbragstad is now known as elbragstad | 14:24 | |
*** yamamoto has joined #openstack-neutron | 14:26 | |
*** munimeha1 has joined #openstack-neutron | 14:28 | |
*** janki has quit IRC | 14:33 | |
*** maciejjozefczyk has quit IRC | 14:34 | |
*** cshen has quit IRC | 14:38 | |
*** Bhujay has quit IRC | 14:38 | |
*** openstackgerrit has joined #openstack-neutron | 14:39 | |
openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add abstract OVSDB API https://review.openstack.org/476612 | 14:39 |
*** mchlumsky has quit IRC | 14:40 | |
*** mchlumsky has joined #openstack-neutron | 14:41 | |
*** Bhujay has joined #openstack-neutron | 14:41 | |
*** Luzi has quit IRC | 14:42 | |
*** gkadam has joined #openstack-neutron | 14:43 | |
slaweq | haleyb: hi | 14:44 |
slaweq | haleyb: I updated bug https://bugs.launchpad.net/neutron/+bug/1791989 with what I found today | 14:44 |
openstack | Launchpad bug 1791989 in neutron "grenade-dvr-multinode job fails" [High,Confirmed] - Assigned to Slawek Kaplonski (slaweq) | 14:44 |
slaweq | I really need some extra eyes to take a look on that as I'm out of ideas what can be the reason of this issue :/ | 14:45 |
* haleyb fires-up the popcorn machine before taking a look :) | 14:45 | |
*** ksambor has quit IRC | 14:45 | |
*** mlavalle has joined #openstack-neutron | 14:48 | |
slaweq | haleyb: You don't need popcorn - there is not too much there ;) | 14:48 |
*** lajoskatona has quit IRC | 14:53 | |
*** mosulica has quit IRC | 14:58 | |
*** Leo_m has joined #openstack-neutron | 15:00 | |
*** hongbin has joined #openstack-neutron | 15:00 | |
*** armax has joined #openstack-neutron | 15:02 | |
*** Leo_m has quit IRC | 15:06 | |
openstackgerrit | Thomas Morin proposed openstack/neutron master: doc: OpenVSwitch firewall internal design, improvements (+ newline fix) https://review.openstack.org/599604 | 15:07 |
*** gkadam has quit IRC | 15:10 | |
*** jistr|call is now known as jistr | 15:11 | |
*** Leo_m has joined #openstack-neutron | 15:13 | |
*** _ix has joined #openstack-neutron | 15:15 | |
slaweq | haleyb: did You read my comments for this issue with grenade job? do You any ideas what can be the reason of this problem? :) | 15:20 |
*** rpittau has quit IRC | 15:21 | |
*** bnemec is now known as beekneemech | 15:21 | |
haleyb | slaweq: i haven't yet, busy with something else | 15:21 |
*** rkukura has quit IRC | 15:24 | |
*** sapd1 has joined #openstack-neutron | 15:27 | |
jlibosva | otherwiseguy: hi, we have a followup for the yesterday beautiful patch (which got stuck in case eventlet was installed on the system) - fixed here: https://review.openstack.org/#/c/604331/ if you could have a look pretty please | 15:27 |
slaweq | haleyb: ok | 15:27 |
otherwiseguy | jlibosva: looking | 15:28 |
openstackgerrit | Hongbin Lu proposed openstack/neutron-tempest-plugin master: Add jobs for stable/rocky branch https://review.openstack.org/595403 | 15:30 |
otherwiseguy | jlibosva: I will trust that you and dalvarez know what you are doing with eventlet more than i do. :p | 15:30 |
openstackgerrit | Hongbin Lu proposed openstack/neutron master: Add propagate_uplink_status to port https://review.openstack.org/571899 | 15:31 |
jlibosva | otherwiseguy: we also sent another 3 patches if you want to have a look | 15:33 |
dalvarez | otherwiseguy: that was a bit hacky thingy :D | 15:33 |
dalvarez | hah | 15:33 |
jlibosva | otherwiseguy: butt I think this one last one is the best | 15:33 |
dalvarez | the best of all the patches ever? | 15:33 |
dalvarez | otherwiseguy: how? | 15:33 |
dalvarez | otherwiseguy: what? | 15:33 |
dalvarez | otherwiseguy: who? | 15:33 |
dalvarez | ops | 15:33 |
otherwiseguy | dalvarez: did you just have a stroke? | 15:34 |
dalvarez | LOL | 15:34 |
jlibosva | dalvarez: or smoke | 15:34 |
dalvarez | nono | 15:34 |
dalvarez | im all sane | 15:34 |
*** tbachman has quit IRC | 15:35 | |
openstackgerrit | Hongbin Lu proposed openstack/neutron-tempest-plugin master: Test propagate_uplink_status on create port https://review.openstack.org/586719 | 15:37 |
openstackgerrit | Merged openstack/ovsdbapp master: ut: Patch get_ident for race transaction test https://review.openstack.org/604331 | 15:39 |
openstackgerrit | Hongbin Lu proposed openstack/neutron master: Introduce floating IP pool resource https://review.openstack.org/555088 | 15:40 |
otherwiseguy | jlibosva: huh, well, at least the nonvoting tempest job fails really quickly allowing patches to merge in ovsdbapp in almost no time. :D | 15:44 |
jlibosva | otherwiseguy: we're lucky the zuul Q is not as insane as it was yesterday | 15:47 |
openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add native implementation OVSDB API https://review.openstack.org/482226 | 15:47 |
*** Bhujay has quit IRC | 15:48 | |
otherwiseguy | jlibosva: indeed. there were things in post for like 30+ hours. | 15:48 |
openstackgerrit | Daniel Alvarez proposed openstack/networking-ovn master: DNM: More debug traces for bug 1735154 https://review.openstack.org/603354 | 15:49 |
openstack | bug 1735154 in networking-ovn "Tempest jobs failing with SSH timeout sometimes" [High,New] https://launchpad.net/bugs/1735154 - Assigned to Daniel Alvarez (dalvarezs) | 15:49 |
*** jlibosva has quit IRC | 15:50 | |
openstackgerrit | Merged openstack/neutron master: Don't uninstall flow tables in setup_dvr_flows_on_integ_br https://review.openstack.org/572729 | 15:50 |
*** sapd1 has quit IRC | 15:51 | |
*** rkukura has joined #openstack-neutron | 15:57 | |
*** tbachman has joined #openstack-neutron | 15:59 | |
*** aojea has quit IRC | 15:59 | |
openstackgerrit | LIU Yulong proposed openstack/neutron master: Install centralized floating IP nat rules to all ha nodes https://review.openstack.org/604094 | 16:15 |
*** ralonsoh has quit IRC | 16:18 | |
*** tmorin has quit IRC | 16:20 | |
_ix | Good morning folks. I was asking about some interesting behavior yesterday that we've been wrestling with for some time under dvr_snat where guests would arbitrarily re-route their north-bound traffic through the network node fips instead of hypervisor fips. | 16:24 |
_ix | It looks like it's related to ipv6 router solicitations from the network nodes themselves. Does this sound at all plausible? | 16:24 |
*** janki has joined #openstack-neutron | 16:25 | |
_ix | To be more specific, the router solicitations were being received by the hypervisor, and fdb/show br-int displayed a corrupt port/vlan for the mac address. Instead of the correct locally defined, it was populated with a network node's port/vlan. | 16:33 |
*** salmankhan has quit IRC | 16:36 | |
*** vkmc is now known as vkmc|afk | 16:37 | |
*** yamahata has quit IRC | 16:38 | |
*** iyamahat has quit IRC | 16:39 | |
*** mattgo has quit IRC | 16:41 | |
*** Leo_m has quit IRC | 16:47 | |
*** Leo_m_ has joined #openstack-neutron | 16:48 | |
*** jpena is now known as jpena|off | 16:54 | |
*** iyamahat has joined #openstack-neutron | 16:55 | |
*** sambetts is now known as sambetts|afk | 16:56 | |
*** rkukura has quit IRC | 16:56 | |
*** hongbin has quit IRC | 17:01 | |
*** gkadam has joined #openstack-neutron | 17:03 | |
*** spsurya has quit IRC | 17:05 | |
*** maciejjozefczyk has joined #openstack-neutron | 17:12 | |
*** maciejjozefczyk has quit IRC | 17:14 | |
*** maciejjozefczyk has joined #openstack-neutron | 17:14 | |
*** yamahata has joined #openstack-neutron | 17:16 | |
*** maciejjozefczyk has quit IRC | 17:19 | |
*** manjeets has joined #openstack-neutron | 17:26 | |
openstackgerrit | Munish proposed openstack/neutron-specs master: Introduce port mirroring for sriov-vf https://review.openstack.org/574477 | 17:34 |
*** etp has quit IRC | 17:41 | |
*** cshen has joined #openstack-neutron | 17:42 | |
*** etp has joined #openstack-neutron | 17:44 | |
*** etp has quit IRC | 17:55 | |
*** etp has joined #openstack-neutron | 17:56 | |
*** janki has quit IRC | 17:57 | |
openstackgerrit | Merged openstack/os-vif master: Fix upper-constraints link in tox file https://review.openstack.org/604158 | 17:59 |
*** panda has quit IRC | 18:08 | |
*** panda has joined #openstack-neutron | 18:10 | |
manjeets | mlavalle, did drivers meeting happen today, couldn't find logs ? | 18:14 |
mlavalle | manjeets: no, since we just came back from PTG, we skipped it | 18:15 |
manjeets | will it be resumed next week ? | 18:16 |
mlavalle | manjeets: yes, of course | 18:16 |
manjeets | cool thanks :) | 18:16 |
mlavalle | manjeets: thanks for caring :-) | 18:16 |
manjeets | mlavalle, I was being mean there tbh, since its at 7 am PST, to have an extra hour of sleep in case it doesn't happen I update my alarms early | 18:18 |
*** rkukura has joined #openstack-neutron | 18:20 | |
mlavalle | manjeets: I understand. I notified the cancellation, though: http://lists.openstack.org/pipermail/openstack-dev/2018-September/134933.html | 18:20 |
manjeets | mlavalle, I have filtered subject [neutron] to access neutron emails quickly, but since this email have Neutron instead of [neutron] as subject so it landed to other folder, which I still need to catch up | 18:23 |
manjeets | thanks ! | 18:23 |
mlavalle | manjeets: I am sorry any ways if you woke up early on this ocassion :-( | 18:24 |
manjeets | mlavalle, luckily I didn't not that's why I was looking for logs and munimeha1 asked me about it as well | 18:24 |
mlavalle | manjeets: oh, great. Then we are fine | 18:25 |
openstackgerrit | Nate Johnston proposed openstack/neutron master: Make neutron-fullstack-python35 job voting https://review.openstack.org/591081 | 18:28 |
openstackgerrit | Nate Johnston proposed openstack/neutron master: Make neutron-fullstack-python36 job voting https://review.openstack.org/591081 | 18:33 |
*** cshen has quit IRC | 18:33 | |
*** cshen has joined #openstack-neutron | 18:36 | |
*** nplanel_ has joined #openstack-neutron | 18:36 | |
*** nplanel__ has quit IRC | 18:39 | |
munimeha1 | thanks manjeet and mlavelle, i was wondering the drivers meeting is happening every week, i may not have updated info. | 18:49 |
*** aniketh has quit IRC | 18:51 | |
*** graysonh has joined #openstack-neutron | 19:10 | |
*** maciejjozefczyk has joined #openstack-neutron | 19:28 | |
*** gkadam has quit IRC | 19:35 | |
*** hjensas has quit IRC | 19:36 | |
*** abaindur has joined #openstack-neutron | 19:46 | |
openstackgerrit | Matt Riedemann proposed openstack/neutron master: Add openstack/placement as a required project for neutron-grenade https://review.openstack.org/604458 | 19:49 |
*** hongbin has joined #openstack-neutron | 19:52 | |
*** slaweq has quit IRC | 19:54 | |
*** slaweq has joined #openstack-neutron | 19:57 | |
*** hjensas has joined #openstack-neutron | 20:05 | |
abaindur | kevinbenton: Had some questions about this new resourche cache mechanism in OVS agent. Possibly seeing some issues with it | 20:29 |
abaindur | the vif was plugged, OVS agent detected it as added, and requested device details | 20:29 |
abaindur | but port_obj.binding_levels was returned as None. The code seems to query some cache now, rather than making an RPC call | 20:30 |
abaindur | either way, it never re-queried the device details. It just said device was not bound on plugin, and never attempted again. VIF plugging timed out in nova and it was deleted | 20:31 |
abaindur | possible bug...? | 20:31 |
*** nplanel__ has joined #openstack-neutron | 20:42 | |
*** nplanel_ has quit IRC | 20:43 | |
*** munimeha1 has quit IRC | 20:44 | |
*** amuller has quit IRC | 20:46 | |
*** cshen has quit IRC | 20:47 | |
openstackgerrit | Hongbin Lu proposed openstack/neutron master: Introduce floating IP pool resource https://review.openstack.org/555088 | 20:59 |
*** abaindur has quit IRC | 21:02 | |
*** abaindur has joined #openstack-neutron | 21:03 | |
*** cshen has joined #openstack-neutron | 21:05 | |
*** boden has quit IRC | 21:12 | |
*** sambetts|afk has quit IRC | 21:12 | |
*** sambetts_ has joined #openstack-neutron | 21:15 | |
openstackgerrit | Nate Johnston proposed openstack/neutron master: Make neutron-fullstack-python36 job voting https://review.openstack.org/591081 | 21:27 |
*** mchlumsky has quit IRC | 21:30 | |
openstackgerrit | Hongbin Lu proposed openstack/neutron-tempest-plugin master: Test floatingip pools service plugin https://review.openstack.org/562038 | 21:35 |
*** ivve has quit IRC | 21:38 | |
*** zul has quit IRC | 22:04 | |
*** abaindur has quit IRC | 22:04 | |
openstackgerrit | Merged openstack/neutron master: Use mktemp instead of tempfile in gate_hook.sh https://review.openstack.org/603536 | 22:06 |
*** mriedem is now known as mriedem_away | 22:09 | |
*** _ix has quit IRC | 22:24 | |
*** dave-mccowan has joined #openstack-neutron | 22:26 | |
*** dave-mccowan has quit IRC | 22:31 | |
*** iyamahat_ has joined #openstack-neutron | 22:32 | |
*** cshen has quit IRC | 22:33 | |
*** cshen has joined #openstack-neutron | 22:33 | |
*** iyamahat has quit IRC | 22:35 | |
*** cshen has quit IRC | 22:38 | |
*** nplanel__ has quit IRC | 22:42 | |
*** tpsilva has quit IRC | 22:43 | |
*** hongbin has quit IRC | 22:44 | |
*** Leo_m has joined #openstack-neutron | 22:45 | |
*** Leo_m_ has quit IRC | 22:47 | |
*** tbachman has quit IRC | 22:55 | |
*** janki has joined #openstack-neutron | 23:13 | |
*** abaindur has joined #openstack-neutron | 23:21 | |
*** iyamahat_ has quit IRC | 23:23 | |
*** janki has quit IRC | 23:28 | |
*** abaindur has quit IRC | 23:29 | |
*** tbachman has joined #openstack-neutron | 23:29 | |
*** abaindur has joined #openstack-neutron | 23:30 | |
*** maciejjozefczyk has quit IRC | 23:31 | |
*** tidwellr_ has joined #openstack-neutron | 23:32 | |
*** tidwellr has quit IRC | 23:32 | |
*** tbachman has quit IRC | 23:34 | |
*** iyamahat_ has joined #openstack-neutron | 23:35 | |
*** tbachman has joined #openstack-neutron | 23:37 | |
*** iyamahat__ has joined #openstack-neutron | 23:39 | |
*** iyamahat_ has quit IRC | 23:43 | |
*** dsneddon has quit IRC | 23:58 | |
*** dsneddon has joined #openstack-neutron | 23:58 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!