| opendevreview | yatin proposed openstack/neutron master: Revert "[fips jobs] blacklist AttachInterfacesTestJSON tempest test" https://review.opendev.org/c/openstack/neutron/+/848351 | 05:39 |
|---|---|---|
| *** froyo_ is now known as froyo | 07:01 | |
| sahid | morning qll, i think we qre good for this easy one, if someone avail to ack it https://review.opendev.org/c/openstack/neutron/+/846275 | 07:59 |
| sahid | the ci is green for the next patch of it | 07:59 |
| frickler | sahid: the CI needs to pass for every single patch, otherwise it cannot be merged | 08:12 |
| frickler | ralonsoh: good morning, by chance I tested network segment create with enforce_scope enabled and it doesn't work. do you know if that is still expected or should I create a bug? | 08:13 |
| ralonsoh | frickler, I'm checking this now, there is a bug already created in Neutron | 08:14 |
| ralonsoh | https://bugs.launchpad.net/neutron/+bug/1980346 | 08:14 |
| frickler | ralonsoh: that's a different issue, not related to system scope | 08:16 |
| ralonsoh | you can comment on this bug | 08:17 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [WIP][DNM] Do not add again an VETH pair to its namespace https://review.opendev.org/c/openstack/neutron/+/848393 | 08:43 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/wallaby: [OVN] Sync QoS policies https://review.opendev.org/c/openstack/neutron/+/846032 | 08:45 |
| sahid | frickler: the ci is green for the patch after this one, it means that the ci should have been green for that patch | 08:48 |
| sahid | there are dependences between patches of that serie | 08:48 |
| frickler | sahid: I cannot tell why the CI is failing, it may be unrelated to your patch. still if it does fail, the patch cannot be merged. | 08:53 |
| ralonsoh | frickler, from https://f65a6916a4100af9048d-92ea954b18df8a732092e221995032d4.ssl.cf1.rackcdn.com/848242/1/gate/osc-functional-devstack/57e3bfb/testr_results.html | 08:54 |
| ralonsoh | what I see is an error in the proxy server, not an error in Neutron | 08:54 |
| sahid | btw ralonsoh would you hqve a second look to https://review.opendev.org/c/openstack/neutron/+/839478/11 ? | 08:59 |
| frickler | ralonsoh: IIUC the proxy error is because q-svc takes too long to respond on the network delete call | 09:00 |
| ralonsoh | yes, maybe this afternoon | 09:00 |
| ralonsoh | frickler, yeah, I'm checking this now | 09:00 |
| ralonsoh | it takes 50 secs and fails | 09:00 |
| ralonsoh | frickler, and I think I saw the error. It is related to the new sqlalchemy version (and the fact that we are handling DB objects outside contexts) | 09:03 |
| ralonsoh | I'll push a patch right now | 09:04 |
| frickler | ralonsoh: ah, I saw those warnings but wasn't sure if they are only cosmetical. since I'm also seeing those in neutron-dynamic-routing testing. guess I'll need to have a deeper look at the latter then, too | 09:08 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [sqlalchemy-20] Add the needed DB context decorators in ``ml2.plugin`` https://review.opendev.org/c/openstack/neutron/+/848396 | 09:14 |
| ralonsoh | lajoskatona, slaweq ^^ if you have 1 min | 09:14 |
| ralonsoh | frickler, ^ can you check that as a depends-on in your CI? | 09:15 |
| frickler | ralonsoh: will do, thx | 09:19 |
| frickler | ralonsoh: a quick local test looks pretty good, no more long delays on creation. do you maybe also want to add a test for this in neutron-tempest-plugin? I looked but couldn't find anything there | 09:23 |
| ralonsoh | frickler, this is a race condition check | 09:24 |
| ralonsoh | and is difficult to simulate | 09:24 |
| slaweq | ralonsoh: done | 09:25 |
| ralonsoh | slaweq, thanks! | 09:25 |
| ralonsoh | wait | 09:25 |
| ralonsoh | you are on PTO! | 09:26 |
| ralonsoh | stop working and get drunk | 09:26 |
| frickler | ralonsoh: in my local test, a simple network segment create/delete cycle was showing issues already. might be helpful if that is checked in neutron CI and doesn't rely on OSC testing | 09:28 |
| ralonsoh | ok, I'll add this test to tempest | 09:29 |
| opendevreview | Elvira García Ruiz proposed openstack/neutron master: Revert "Temporary blacklist test_reboot_server_hard in OVN IPv6 job" https://review.opendev.org/c/openstack/neutron/+/848353 | 09:40 |
| lajoskatona | ralonsoh: regarding https://review.opendev.org/c/openstack/neutron/+/846288 , testing with pyroute2 master, do we gain anything if we test with latest tagged/released pyroute2? | 09:41 |
| ralonsoh | nope | 09:41 |
| ralonsoh | we are already doing this | 09:41 |
| ralonsoh | I'll ask svinota how to do this in our CIs | 09:42 |
| ralonsoh | I'll paste in your patch the link | 09:42 |
| ralonsoh | for now, we can hold your patch until we have a proper way to do it | 09:42 |
| ralonsoh | agree> | 09:42 |
| lajoskatona | ralonsoh: thanks, I can also ask | 09:42 |
| ralonsoh | ? | 09:42 |
| ralonsoh | perfect | 09:42 |
| ralonsoh | https://github.com/svinota/pyroute2/issues/939 | 09:47 |
| lajoskatona | ralonsoh: 👍 | 09:49 |
| opendevreview | Elvira García Ruiz proposed openstack/neutron master: Revert "Temporary blacklist test_reboot_server_hard in OVN IPv6 job" https://review.opendev.org/c/openstack/neutron/+/848353 | 09:52 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [sqlalchemy-20] Remove unnecessary DB context https://review.opendev.org/c/openstack/neutron/+/848401 | 10:07 |
| opendevreview | Bernard Cafarelli proposed openstack/networking-sfc master: Drop lower-constraints.txt and its testing https://review.opendev.org/c/openstack/networking-sfc/+/840096 | 10:16 |
| ralonsoh | lajoskatona, if you have 1 min (3 easy reviews) | 10:42 |
| ralonsoh | https://review.opendev.org/c/openstack/neutron/+/848146 | 10:42 |
| ralonsoh | https://review.opendev.org/c/openstack/neutron/+/848156 | 10:42 |
| ralonsoh | https://review.opendev.org/c/openstack/neutron/+/848159 | 10:42 |
| lajoskatona | ralonsoh: checking | 10:55 |
| ralonsoh | thanks a lot | 10:55 |
| sahid | btw ralonsoh would you hqve a second look to https://review.opendev.org/c/openstack/neutron/+/839478/11 ? | 10:56 |
| ralonsoh | sahid, yes, I told you this morning I'll check it this afternoon | 10:57 |
| ralonsoh | but I didn't have a single second for this today | 10:57 |
| sahid | I did not catch that | 10:58 |
| sahid | thank i appreciate | 10:58 |
| sahid | let me know if I can help on making reviews for some patches that you may have ini your queue | 10:59 |
| opendevreview | Merged openstack/neutron stable/yoga: ovn: revert to stateful dnat_and_snat https://review.opendev.org/c/openstack/neutron/+/848283 | 11:02 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/xena: [OVN][Placement] Make the Placement reporter compatible with OVN https://review.opendev.org/c/openstack/neutron/+/848430 | 11:17 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/wallaby: [OVN][Placement] Make the Placement reporter compatible with OVN https://review.opendev.org/c/openstack/neutron/+/848431 | 11:17 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/victoria: [OVN][Placement] Make the Placement reporter compatible with OVN https://review.opendev.org/c/openstack/neutron/+/848432 | 11:18 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/ussuri: [OVN][Placement] Make the Placement reporter compatible with OVN https://review.opendev.org/c/openstack/neutron/+/848433 | 11:18 |
| opendevreview | Merged openstack/neutron master: Mellanox_eth.img url expires, remove the mellanox_eth.img node https://review.opendev.org/c/openstack/neutron/+/848074 | 11:19 |
| opendevreview | Merged openstack/neutron stable/yoga: migration: Remove patch port between br-int and br-tun https://review.opendev.org/c/openstack/neutron/+/848276 | 12:13 |
| opendevreview | Merged openstack/neutron stable/xena: migration: Remove patch port between br-int and br-tun https://review.opendev.org/c/openstack/neutron/+/848277 | 12:13 |
| opendevreview | Merged openstack/neutron stable/wallaby: migration: Remove patch port between br-int and br-tun https://review.opendev.org/c/openstack/neutron/+/848278 | 12:13 |
| opendevreview | Mamatisa Nurmatov proposed openstack/neutron-lib master: Remove unused placement constants https://review.opendev.org/c/openstack/neutron-lib/+/848445 | 13:22 |
| opendevreview | Merged openstack/neutron master: Replace "Inspector.from_engine()" with "sqlalchemy.inspect()" https://review.opendev.org/c/openstack/neutron/+/848159 | 13:48 |
| opendevreview | Merged openstack/neutron master: [sqlalchemy-20] Add the needed DB context decorators in ``ml2.plugin`` https://review.opendev.org/c/openstack/neutron/+/848396 | 13:48 |
| lajoskatona | #startmeeting neutron_drivers | 14:00 |
| opendevmeet | Meeting started Fri Jul 1 14:00:33 2022 UTC and is due to finish in 60 minutes. The chair is lajoskatona. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:00 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:00 |
| opendevmeet | The meeting name has been set to 'neutron_drivers' | 14:00 |
| lajoskatona | Hi | 14:00 |
| mlavalle | o/o/ | 14:00 |
| ralonsoh | hello | 14:00 |
| lajoskatona | let's see if we will have enough people today | 14:01 |
| mlavalle | slaweq is off on PTO | 14:02 |
| lajoskatona | mlavalle: yes, but this morning ralonsoh had to send him home from the office :-) | 14:03 |
| mlavalle | too bad | 14:04 |
| ralonsoh | (he's now having fun, for sure) | 14:04 |
| mlavalle | ralonsoh: I promise you'll never have to do that with me | 14:04 |
| haleyb | hi | 14:05 |
| lajoskatona | mlavalle: +1 | 14:05 |
| lajoskatona | ok, we have 4 drivers, which seems few | 14:07 |
| lajoskatona | so I think we have to postpone today's discussion to next week | 14:07 |
| lajoskatona | Do you have anything to discuss in the on demand agenda? | 14:08 |
| mlavalle | bot me | 14:08 |
| mlavalle | not me | 14:08 |
| ralonsoh | nothing from me | 14:09 |
| haleyb | not from me either | 14:09 |
| lajoskatona | let's close the meeting for today then, thanks for coming | 14:09 |
| lajoskatona | #endmeeting | 14:09 |
| opendevmeet | Meeting ended Fri Jul 1 14:09:44 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:09 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/neutron_drivers/2022/neutron_drivers.2022-07-01-14.00.html | 14:09 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/neutron_drivers/2022/neutron_drivers.2022-07-01-14.00.txt | 14:09 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/neutron_drivers/2022/neutron_drivers.2022-07-01-14.00.log.html | 14:09 |
| ralonsoh | have a nice weekend | 14:09 |
| mlavalle | you too! | 14:09 |
| lajoskatona | Have a nice weekend | 14:09 |
| mlavalle | o/ | 14:09 |
| haleyb | +2 +A | 14:10 |
| bcafarel | lajoskatona: as you have some freed time :) can you check https://review.opendev.org/c/openstack/networking-sfc/+/840096 ? | 14:10 |
| bcafarel | it was W+1 I just rebased to fix conflict | 14:10 |
| lajoskatona | bcafarel: done | 14:19 |
| bcafarel | lajoskatona: thanks! | 14:21 |
| *** dasm|off is now known as dasm | 14:26 | |
| opendevreview | Lajos Katona proposed openstack/tap-as-a-service master: Py3: Remove six https://review.opendev.org/c/openstack/tap-as-a-service/+/846624 | 14:50 |
| opendevreview | Elvira García Ruiz proposed openstack/neutron master: Remove duplicated Zuul job definition https://review.opendev.org/c/openstack/neutron/+/848461 | 15:11 |
| opendevreview | Merged openstack/neutron master: [FT] Ensure "update_virtual_port_host" has been called https://review.opendev.org/c/openstack/neutron/+/848156 | 15:24 |
| opendevreview | Merged openstack/neutron master: [FT] Run all .*MySQL and .*PostgreSQL tests in an isolated thread https://review.opendev.org/c/openstack/neutron/+/848146 | 16:02 |
| opendevreview | Merged openstack/networking-sfc master: Drop lower-constraints.txt and its testing https://review.opendev.org/c/openstack/networking-sfc/+/840096 | 16:06 |
| *** tbachman_ is now known as tbachman | 18:59 | |
| opendevreview | Merged openstack/neutron master: ovn: use requested-chassis list format for live migration https://review.opendev.org/c/openstack/neutron/+/828455 | 20:23 |
| *** dasm is now known as dasm|off | 21:07 | |
| *** dasm|off is now known as dasm | 21:09 | |
| *** dasm is now known as Guest3965 | 21:10 | |
| *** Guest3965 is now known as dasm | 21:12 | |
| *** dasm is now known as dasm|off | 21:13 | |
| *** dasm|off is now known as dasm | 21:13 | |
| *** dasm is now known as dasm|off | 21:13 | |
| opendevreview | Brian Haley proposed openstack/neutron master: Fix some pylint indentation warnings https://review.opendev.org/c/openstack/neutron/+/848475 | 21:54 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!