*** Yumeng has quit IRC | 00:05 | |
*** raildo__ has joined #openstack-neutron | 00:05 | |
*** raildo_ has quit IRC | 00:08 | |
*** JamesBenson has quit IRC | 00:08 | |
*** JamesBenson has joined #openstack-neutron | 00:09 | |
*** hoonetorg has quit IRC | 00:29 | |
*** raildo__ has quit IRC | 00:31 | |
*** raildo__ has joined #openstack-neutron | 00:32 | |
*** hoonetorg has joined #openstack-neutron | 00:41 | |
*** sapd1_x has quit IRC | 00:48 | |
openstackgerrit | lkk proposed openstack/neutron-tempest-plugin master: nagative cases for port https://review.opendev.org/749190 | 00:53 |
---|---|---|
*** lifeless has joined #openstack-neutron | 01:02 | |
*** zzzeek has quit IRC | 01:11 | |
*** zzzeek has joined #openstack-neutron | 01:14 | |
openstackgerrit | Merged openstack/neutron master: [OVN] Gap: Add QoS minimum bandwidth allocation in Placement API https://review.opendev.org/749217 | 01:27 |
*** Yumeng has joined #openstack-neutron | 01:32 | |
*** JamesBenson has quit IRC | 01:35 | |
*** raildo_ has joined #openstack-neutron | 01:52 | |
*** zhanglong has joined #openstack-neutron | 01:54 | |
*** raildo__ has quit IRC | 01:54 | |
openstackgerrit | Merged openstack/neutron master: Squash alchemy Liberty branch https://review.opendev.org/741676 | 01:59 |
openstackgerrit | Merged openstack/neutron master: Don't directly access the in-memory DB for NB_Global row https://review.opendev.org/747465 | 01:59 |
*** ccamposr__ has joined #openstack-neutron | 02:12 | |
*** zhanglong has quit IRC | 02:12 | |
*** zhanglong has joined #openstack-neutron | 02:14 | |
*** ccamposr has quit IRC | 02:16 | |
*** armax has quit IRC | 02:17 | |
*** zhanglong has quit IRC | 02:18 | |
openstackgerrit | chen lianghong proposed openstack/neutron master: Show cmd detail when execute linux command failed https://review.opendev.org/749076 | 02:39 |
*** ijw has quit IRC | 02:47 | |
*** ijw has joined #openstack-neutron | 02:48 | |
openstackgerrit | lkk proposed openstack/neutron-tempest-plugin master: cases associated with the port https://review.opendev.org/749182 | 02:56 |
*** JamesBenson has joined #openstack-neutron | 03:05 | |
*** JamesBen_ has joined #openstack-neutron | 03:09 | |
*** JamesBenson has quit IRC | 03:10 | |
openstackgerrit | XiaoYu Zhu proposed openstack/neutron-specs master: L3 router support ecmp https://review.opendev.org/729532 | 03:15 |
openstackgerrit | zhanghao proposed openstack/neutron master: Rename networking-ovn-metadata-agent to neutron-ovn-metadata-agent https://review.opendev.org/746640 | 03:17 |
*** sapd1_x has joined #openstack-neutron | 03:22 | |
*** lajoskatona has joined #openstack-neutron | 04:00 | |
*** dave-mccowan has quit IRC | 04:09 | |
openstackgerrit | Deepak proposed openstack/neutron master: Add SR-IOV ML2 driver support for VLAN trunking. https://review.opendev.org/665467 | 04:16 |
*** hjensas has joined #openstack-neutron | 04:37 | |
*** JamesBen_ has quit IRC | 04:39 | |
*** TheJulia has quit IRC | 04:39 | |
*** TheJulia has joined #openstack-neutron | 04:40 | |
*** osmanlicilegi has quit IRC | 04:40 | |
*** osmanlicilegi has joined #openstack-neutron | 04:40 | |
*** osmanlicilegi has quit IRC | 04:40 | |
*** osmanlicilegi has joined #openstack-neutron | 04:41 | |
*** ccamposr__ has quit IRC | 04:47 | |
*** ccamposr__ has joined #openstack-neutron | 04:47 | |
*** hjensas has quit IRC | 04:58 | |
*** JamesBenson has joined #openstack-neutron | 05:00 | |
*** ratailor has joined #openstack-neutron | 05:27 | |
openstackgerrit | Lajos Katona proposed openstack/networking-odl master: Migrate legacy grenade job to be native Zuul v3 https://review.opendev.org/725647 | 05:27 |
*** hjensas has joined #openstack-neutron | 05:27 | |
*** hjensas has quit IRC | 05:28 | |
*** sapd1_x has quit IRC | 05:29 | |
*** JamesBenson has quit IRC | 05:32 | |
*** JamesBenson has joined #openstack-neutron | 05:33 | |
*** hjensas has joined #openstack-neutron | 05:36 | |
*** JamesBenson has quit IRC | 05:38 | |
openstackgerrit | lkk proposed openstack/neutron-tempest-plugin master: nagative cases for port https://review.opendev.org/749190 | 05:40 |
openstackgerrit | Lajos Katona proposed openstack/neutron master: fullstack: Use smaller subnets to avoid clashes https://review.opendev.org/749041 | 05:52 |
openstackgerrit | Sam Morrison proposed openstack/networking-midonet master: Use the networking-l2gw project new location https://review.opendev.org/738046 | 05:54 |
openstackgerrit | Sam Morrison proposed openstack/networking-midonet master: Return a 404 when network doesn't exist when creating a floating IP https://review.opendev.org/749641 | 05:54 |
*** icey has quit IRC | 05:59 | |
*** raildo_ has quit IRC | 06:11 | |
*** icey has joined #openstack-neutron | 06:28 | |
*** icey has quit IRC | 06:29 | |
*** icey has joined #openstack-neutron | 06:29 | |
*** gcheresh has quit IRC | 06:31 | |
*** manuvakery has joined #openstack-neutron | 06:33 | |
*** terdei has quit IRC | 06:34 | |
*** icey has quit IRC | 06:35 | |
*** icey has joined #openstack-neutron | 06:35 | |
*** JamesBenson has joined #openstack-neutron | 06:38 | |
openstackgerrit | Merged openstack/neutron master: Move pylint disable comment to an effective place https://review.opendev.org/749320 | 06:41 |
openstackgerrit | Sam Morrison proposed openstack/networking-midonet master: Return a 404 when network doesn't exist when creating a floating IP https://review.opendev.org/749641 | 06:43 |
openstackgerrit | Sam Morrison proposed openstack/networking-midonet master: [WIP] Remove tap-as-a-service and l2gateway integrations https://review.opendev.org/749653 | 06:43 |
*** JamesBenson has quit IRC | 06:43 | |
*** gcheresh has joined #openstack-neutron | 06:46 | |
*** gibi has joined #openstack-neutron | 06:56 | |
slaweq | bcafarel: hi, can You check my comment in https://review.opendev.org/#/c/739962/ ? | 06:59 |
slaweq | does it makes sense for You or should I change my comment there? | 06:59 |
slaweq | amotoki: hi | 07:00 |
slaweq | amotoki: we should prepare releases of our non-client libs today, right? | 07:01 |
*** bengates has joined #openstack-neutron | 07:05 | |
*** JamesBenson has joined #openstack-neutron | 07:06 | |
openstackgerrit | Bernard Cafarelli proposed openstack/os-ken master: Fix sockaddr lib with python 3 https://review.opendev.org/739962 | 07:11 |
*** ralonsoh has joined #openstack-neutron | 07:11 | |
bcafarel | slaweq: ^ made sense and fixed :) | 07:11 |
*** sapd1_x has joined #openstack-neutron | 07:13 | |
*** ccamposr has joined #openstack-neutron | 07:17 | |
*** tesseract has joined #openstack-neutron | 07:19 | |
*** yoctozepto2 has joined #openstack-neutron | 07:20 | |
*** ccamposr__ has quit IRC | 07:20 | |
*** yoctozepto has quit IRC | 07:22 | |
*** yoctozepto2 is now known as yoctozepto | 07:22 | |
*** JamesBenson has quit IRC | 07:24 | |
slaweq | bcafarel: thx | 07:32 |
*** zhanglong has joined #openstack-neutron | 07:32 | |
slaweq | lajoskatona: ralonsoh: hi, I just opened https://bugs.launchpad.net/bgpvpn/+bug/1894056 as it seems we have broken gate in networking-bgpvpn | 07:34 |
openstack | Launchpad bug 1894056 in networking-bgpvpn "DB migration broken" [Undecided,Confirmed] | 07:34 |
slaweq | also ykarel from rdo pinged me about the same issue in their job now | 07:34 |
slaweq | ralonsoh: can You take a look if that is related to Your patch https://review.opendev.org/#/c/741676/ ? | 07:35 |
ralonsoh | slaweq, let me check | 07:35 |
slaweq | ralonsoh: thx | 07:35 |
ralonsoh | slaweq, could be yes | 07:36 |
ralonsoh | slaweq, but I can't find this revision ID | 07:38 |
ralonsoh | in Neutron | 07:38 |
slaweq | this one mentioned in bug is here http://codesearch.openstack.org/?q=d4d4d7f03b21&i=nope&files=&repos= | 07:38 |
slaweq | so actually both bgpvpn and bagpipe are broken | 07:38 |
slaweq | ykarel got issue with revision '3600132c6147' which is in networking-bgpvpn | 07:39 |
slaweq | see here: https://logserver.rdoproject.org/ci.centos.org/weirdo-generic-puppet-openstack-scenario004/13113/weirdo-project/puppet.txt.gz | 07:39 |
slaweq | so basically I'm affraid it can be that we broke all stadium projects which has got own db migration scripts from old versions | 07:39 |
ralonsoh | slaweq, yes but the point is that this revision number belongs only to this project | 07:39 |
*** tosky has joined #openstack-neutron | 07:40 | |
ralonsoh | slaweq, could be if they are pointing to mitaka | 07:40 |
ralonsoh | sorry, Liberty | 07:40 |
slaweq | it is pointing to liberty | 07:40 |
ralonsoh | I know... | 07:40 |
slaweq | and it's last in liberty | 07:40 |
slaweq | next one is in pike | 07:41 |
slaweq | so it can't find "down key" which is in liberty as that was squashed | 07:41 |
slaweq | correct? | 07:41 |
ralonsoh | slaweq, but that's the point | 07:41 |
ralonsoh | d4d4d7f03b21 does not exist in Neutron | 07:41 |
ralonsoh | I can't find it | 07:42 |
slaweq | it exists in networking-bagpipe | 07:42 |
ralonsoh | d4d4d7f03b21 is only a project ID revision | 07:42 |
openstackgerrit | Sam Morrison proposed openstack/networking-midonet master: [WIP] Remove tap-as-a-service and l2gateway integrations https://review.opendev.org/749653 | 07:42 |
ralonsoh | slaweq, and now cli.EXPAND is Mitaka | 07:42 |
ralonsoh | that means they are creating the new tables in Mitaka base | 07:43 |
*** zhanglong has quit IRC | 07:50 | |
openstackgerrit | lkk proposed openstack/neutron-tempest-plugin master: cases associated with the port https://review.opendev.org/749182 | 07:54 |
*** zhanglong has joined #openstack-neutron | 07:56 | |
*** jpena|off is now known as jpena | 07:57 | |
*** ccamposr__ has joined #openstack-neutron | 08:02 | |
*** ccamposr has quit IRC | 08:03 | |
*** ccamposr has joined #openstack-neutron | 08:05 | |
*** rubasov has quit IRC | 08:08 | |
*** ccamposr__ has quit IRC | 08:08 | |
*** lucasagomes has joined #openstack-neutron | 08:09 | |
*** ratailor has quit IRC | 08:12 | |
*** Qianbiao has joined #openstack-neutron | 08:13 | |
yoctozepto | kolla ansible is also unhappy about your current breakage :-) | 08:19 |
openstackgerrit | Sam Morrison proposed openstack/networking-midonet master: [WIP] Remove tap-as-a-service and l2gateway integrations https://review.opendev.org/749653 | 08:19 |
yoctozepto | it's vpnaas for us | 08:21 |
slaweq | yoctozepto: hi, are You talking about this issue with db migration? | 08:27 |
frickler | ralonsoh: did you use a tool to squash those migrations or did you do it manually? if there's a tool I might like to use it on some other project, too | 08:31 |
*** tmorin has joined #openstack-neutron | 08:32 | |
*** ebbex has quit IRC | 08:37 | |
yoctozepto | slaweq: hi, SÅ‚awek, yes, I commented on it in launchpad too | 08:38 |
openstackgerrit | Lucas Alvares Gomes proposed openstack/neutron-tempest-plugin master: NOT REVIEW: Test OVN devstack module https://review.opendev.org/749503 | 08:39 |
yoctozepto | do note it's a different key id (and a different project of course) | 08:44 |
yoctozepto | dynamic routing somehow passes, but vpnaas does not | 08:44 |
*** ebbex has joined #openstack-neutron | 08:44 | |
lajoskatona | slaweq: sorry I was "out-of-office" for 1 hour, I check it | 08:46 |
slaweq | yoctozepto: all projects which don't have any db migration scripts from liberty or older releases will pass | 08:47 |
slaweq | yoctozepto: that's probably why dynamic routing passed | 08:48 |
openstackgerrit | Sam Morrison proposed openstack/networking-midonet master: [WIP] Remove tap-as-a-service and l2gateway integrations https://review.opendev.org/749653 | 08:49 |
*** priteau has joined #openstack-neutron | 08:49 | |
*** zhanglong has quit IRC | 08:51 | |
slaweq | lajoskatona: ralonsoh can You check https://review.opendev.org/#/c/739962/ ? | 08:53 |
slaweq | and that one https://review.opendev.org/#/c/749537/ :) | 08:54 |
slaweq | thx in advance | 08:54 |
lajoskatona | slaweq: ok | 08:56 |
ralonsoh | slaweq, sure | 08:57 |
yoctozepto | slaweq: ack, makes sense | 08:58 |
*** sapd1_x has quit IRC | 08:59 | |
slaweq | yoctozepto: ralonsoh will propose fixes for stadium projects soon | 08:59 |
*** sapd1_x has joined #openstack-neutron | 08:59 | |
slaweq | if not, we will propose revert of this patch which broke it | 08:59 |
*** dtantsur|afk is now known as dtantsur | 09:06 | |
*** jlibosva has joined #openstack-neutron | 09:06 | |
yoctozepto | slaweq: thanks; vpnaas is not a mandatory dep in kolla so we'll see about disabling its now-forced bootstrap as well | 09:08 |
openstackgerrit | Mamduh proposed openstack/os-vif master: Refactor code of linux_net to more cleaner and increase performace https://review.opendev.org/746673 | 09:14 |
openstackgerrit | Merged openstack/os-ken master: Fix sockaddr lib with python 3 https://review.opendev.org/739962 | 09:17 |
openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/networking-bagpipe master: [WIP] Move initial DB migration to Mitaka https://review.opendev.org/749687 | 09:27 |
*** sapd1_x has quit IRC | 09:36 | |
openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/networking-bgpvpn master: [WIP] Move initial DB migration to Mitaka https://review.opendev.org/749696 | 09:58 |
*** rcernin has quit IRC | 10:01 | |
*** rcernin has joined #openstack-neutron | 10:06 | |
*** hjensas is now known as hjensas|afk | 10:11 | |
*** rcernin has quit IRC | 10:11 | |
*** rcernin has joined #openstack-neutron | 10:19 | |
ralonsoh | slaweq, both projects use the same CI job neutron-tempest-plugin-bgpvpn-bagpipe | 10:21 |
ralonsoh | and this CI job needs both# | 10:21 |
ralonsoh | we need to upgrade both at the same time | 10:21 |
slaweq | ralonsoh: yes, but as yoctozepto mentioned, the same problem is also with neutron-vpnaas | 10:22 |
*** rcernin has quit IRC | 10:24 | |
ralonsoh | slaweq, then we need to revert the neutron patch | 10:27 |
slaweq | ralonsoh: if You say so, I'm ok with that quick solution too :?) | 10:28 |
slaweq | * :) | 10:28 |
lajoskatona | slaweq, ralonsoh: reverting means that the whole "simplification" of migration tree must be rethinked later considering all projects that "consume" neutron-db-manage and neutron migration tree? | 10:30 |
*** rcernin has joined #openstack-neutron | 10:31 | |
ralonsoh | lajoskatona, for now, only the revert | 10:32 |
*** priteau has quit IRC | 10:33 | |
lajoskatona | ralonsoh: ok | 10:34 |
*** rcernin has quit IRC | 10:36 | |
*** priteau has joined #openstack-neutron | 10:49 | |
*** priteau has quit IRC | 10:54 | |
*** JamesBenson has joined #openstack-neutron | 11:00 | |
*** priteau has joined #openstack-neutron | 11:03 | |
*** priteau has quit IRC | 11:11 | |
*** hjensas|afk is now known as hjensas | 11:15 | |
*** jangutter has joined #openstack-neutron | 11:16 | |
*** jangutter_ has quit IRC | 11:20 | |
*** Yumeng has quit IRC | 11:22 | |
zigo | ralonsoh: Hi there! I believe I addressed all your concerns in https://review.opendev.org/#/c/669395/, plus fixed the English mistakes reported on my doc addition, could you review the patch again? | 11:28 |
ralonsoh | zigo, https://review.opendev.org/#/c/669395/25/neutron/objects/subnet.py@515 | 11:30 |
ralonsoh | I don't see any DB test there | 11:30 |
zigo | ralonsoh: Right, though I'm not sure what you'd be expecting for such test, or how to write it. | 11:31 |
zigo | Isn't the test I added doing what you asked? | 11:31 |
zigo | Or, could you point at example test code, so I can look how it's made? | 11:32 |
ralonsoh | zigo, for example, when adding new methods to an OVO interface, you should implement a SqlTestCase test case to cover it | 11:36 |
ralonsoh | that will create the needed registers in the DB | 11:37 |
ralonsoh | related to this OVO | 11:37 |
zigo | OVO ? | 11:37 |
ralonsoh | oslo versioned object | 11:37 |
ralonsoh | all classes under neutron.objects | 11:37 |
ralonsoh | example: https://review.opendev.org/#/c/721679 | 11:37 |
*** jpena is now known as jpena|lunch | 11:38 | |
*** lpetrut has joined #openstack-neutron | 11:47 | |
amotoki | slaweq: sorry for late. we need releases for libs. I will check the status right now | 11:48 |
slaweq | amotoki: no problem | 11:48 |
slaweq | amotoki: I prepared etherpad https://etherpad.opendev.org/p/neutron-victoria-lib-patches with patches which we should have merged | 11:49 |
amotoki | slaweq: thanks | 11:49 |
slaweq | now I think that everything (or almost everything) is merged already | 11:49 |
slaweq | but please check once again, maybe I missed something :) | 11:49 |
*** priteau has joined #openstack-neutron | 12:01 | |
*** JamesBenson has quit IRC | 12:04 | |
*** raildo has joined #openstack-neutron | 12:04 | |
openstackgerrit | Lajos Katona proposed openstack/neutron-lib master: New exception QosPlacementAllocationConflict https://review.opendev.org/748279 | 12:14 |
openstackgerrit | Slawek Kaplonski proposed openstack/neutron master: Convert neutron-grenade-ovn job to be zuulv3 https://review.opendev.org/729591 | 12:15 |
*** JamesBenson has joined #openstack-neutron | 12:19 | |
*** JamesBenson has quit IRC | 12:23 | |
*** jangutter_ has joined #openstack-neutron | 12:35 | |
*** jlibosva1 has joined #openstack-neutron | 12:36 | |
*** jlibosva is now known as Guest55146 | 12:36 | |
*** jlibosva1 is now known as jlibosva | 12:36 | |
*** jangutter has quit IRC | 12:39 | |
*** Guest55146 has quit IRC | 12:39 | |
*** jpena|lunch is now known as jpena | 12:40 | |
haleyb | slaweq: will this be the last release of neutron-lib for victoria cycle? | 12:42 |
*** jlibosva1 has joined #openstack-neutron | 12:45 | |
*** jlibosva is now known as Guest25548 | 12:45 | |
*** jlibosva1 is now known as jlibosva | 12:45 | |
*** Guest25548 has quit IRC | 12:47 | |
slaweq | haleyb: yes | 12:49 |
slaweq | it's final release for non-client libs this week | 12:49 |
*** ralonsoh has quit IRC | 12:51 | |
*** ralonsoh has joined #openstack-neutron | 12:52 | |
haleyb | slaweq: i'm wondering whether we should land https://review.opendev.org/#/c/730736/ - i was going to try and get the related changes merged too, but that one is a lib constant, although i suppose we don't need it since we can always define it locally in neutron | 12:52 |
haleyb | oh actually we can't as we need that in -lib | 12:53 |
*** nweinber has joined #openstack-neutron | 12:54 | |
haleyb | rm_work: are you back? | 12:54 |
openstackgerrit | Brian Haley proposed openstack/neutron-lib master: Add `access_as_readonly` sharing action https://review.opendev.org/730736 | 12:56 |
*** Inflatador has joined #openstack-neutron | 12:58 | |
slaweq | haleyb: but I don't think rm_work is going to implement that in V | 12:58 |
slaweq | so probably this can wait for next cycle also | 12:58 |
*** rcernin has joined #openstack-neutron | 12:59 | |
haleyb | slaweq: ok, i was going to propose taking the neutron changes over to get them done | 12:59 |
openstackgerrit | Lajos Katona proposed openstack/neutron master: Allow replacing the QoS policy of bound port https://review.opendev.org/747774 | 12:59 |
*** rcernin has quit IRC | 13:04 | |
amotoki | slaweq: haleyb: where we should define constants is always a question. In case of https://review.opendev.org/#/c/730736/, it sounds a kind of refactoring. I think we can define them in a neutron patch and if needed we can move them to -lib. | 13:09 |
*** dave-mccowan has joined #openstack-neutron | 13:10 | |
amotoki | slaweq: It looks like we've merged necessary patches in non-client libs. I will prepare release patches for neutron-lib, ovsdbapp and os-ken. | 13:10 |
haleyb | amotoki: yes, would agree except for change in https://review.opendev.org/#/c/730736/4/neutron_lib/db/model_query.py - but it can wait | 13:10 |
slaweq | amotoki: thx | 13:11 |
*** Inflatador has quit IRC | 13:14 | |
*** Inflatador has joined #openstack-neutron | 13:18 | |
*** sapd1_x has joined #openstack-neutron | 13:19 | |
*** armax has joined #openstack-neutron | 13:20 | |
*** jangutter has joined #openstack-neutron | 13:21 | |
*** jangutter_ has quit IRC | 13:23 | |
*** JamesBenson has joined #openstack-neutron | 13:25 | |
*** jangutter has quit IRC | 13:26 | |
*** jangutter has joined #openstack-neutron | 13:27 | |
lajoskatona | slaweq, amotoki: Hi, regarding https://review.opendev.org/#/q/status:open+project:openstack/neutron-lib+branch:master+topic:bug/1882804 ,do you see possibilty to include it to neutron-lib release? | 13:28 |
*** jangutter has quit IRC | 13:29 | |
*** jangutter has joined #openstack-neutron | 13:30 | |
amotoki | lajoskatona: do you think a neutron patch consuming it lands in victoria? I just wonder whether we need to wait them, but I am not so sure. | 13:31 |
lajoskatona | amotoki: I think it is doable | 13:33 |
lajoskatona | amotoki: the problem was to design to keep neutron - placement consistency as much as possible | 13:33 |
lajoskatona | amotoki: but I think last discussions (see https://bugs.launchpad.net/neutron/+bug/1882804 from comment #10) finally got us to the result, how to do this | 13:34 |
openstack | Launchpad bug 1882804 in neutron "RFE: allow replacing the QoS policy of bound port" [Wishlist,Confirmed] - Assigned to Lajos Katona (lajos-katona) | 13:34 |
slaweq | lajoskatona: amotoki please keep in mind that next week is feature freeze week for Victoria | 13:36 |
amotoki | slaweq: lajoskatona: yeah, I am not sure about the status of the qos policy stuff and whether we can land it. IMHO neutron-lib patch should not be a blocker, but we need to judge it depending on the status of neutron side. | 13:38 |
amotoki | slaweq: lajoskatona: I would like to have feedbacks from folks familiar with this effort. | 13:39 |
lajoskatona | amotoki, slaweq: true, I had in mind that there's an extra week for feature freeze.... | 13:40 |
*** spatel has joined #openstack-neutron | 13:40 | |
*** sapd1_x has quit IRC | 13:40 | |
*** spatel has quit IRC | 13:44 | |
slaweq | lajoskatona: according to https://releases.openstack.org/victoria/schedule.html it's next week | 13:45 |
lajoskatona | slaweq: yes, I just checked | 13:46 |
slaweq | lajoskatona: I just +2 one of those patches | 13:47 |
slaweq | lajoskatona: in https://review.opendev.org/#/c/741452/5 ralonsoh had some comments, please address them and I think we can merge both today hopefully | 13:47 |
ralonsoh | just minor comments | 13:48 |
slaweq | lajoskatona: amotoki I just added both to the list in https://etherpad.opendev.org/p/neutron-victoria-lib-patches | 13:48 |
openstackgerrit | Lajos Katona proposed openstack/neutron-lib master: Add placement client methods for allocations https://review.opendev.org/741452 | 13:48 |
lajoskatona | slaweq: thanks | 13:49 |
*** Luzi has joined #openstack-neutron | 13:50 | |
gmann | lajoskatona: can you remove the depends-on from these two - https://review.opendev.org/#/c/749579/ https://review.opendev.org/#/c/749580/ | 13:51 |
gmann | you are good to merge those after that | 13:51 |
rm_work | @haleyb: back on Tuesday | 14:09 |
rm_work | @haleyb: I mean, would be nice to get the lib stuff in early, would make it less of a pain to deal with | 14:10 |
haleyb | rm_work: ah, was going to ask about the rbac work in neutron, and if we needed it this cycle | 14:10 |
rm_work | That's what blocked me | 14:10 |
rm_work | Because lib is hard to test alongside main repo | 14:10 |
rm_work | I can't work on it much this cycle but if you could, it SHOULDN'T be that much work? I think | 14:10 |
haleyb | rm_work: i know some people, and could restructure that to get it in, the -lib window is closing | 14:11 |
rm_work | And it'd help Octavia immensely | 14:11 |
rm_work | Well, maybe getting that in early next cycle is fine | 14:11 |
rm_work | If you've got higher priority stuff now | 14:11 |
haleyb | rm_work: it's up to you, but think the neutron-lib tag review is going out soon | 14:11 |
rm_work | Seems like it might be simple to get in | 14:13 |
rm_work | It's not a big change | 14:13 |
rm_work | And shouldn't impact anything | 14:13 |
rm_work | If you think you have the cycles... I worry about stealing review cycles though too | 14:14 |
rm_work | Honestly since the biggest blocker I had was the fact that I couldn't get test with the lib change pending, I really should have just pushed to get it merged earlier | 14:14 |
haleyb | rm_work: it's about to pass zuul, so i'll try and move it along. and i can refactor the other reviews to get the code in, i'd get rid of the "test with neutron-lib" code | 14:14 |
rm_work | I don't think the other reviews are close to *done* though | 14:15 |
rm_work | I got blocked by this lib change and the way neutron did testing with lib | 14:15 |
haleyb | rm_work: ack, and at this point only the -lib change is important | 14:16 |
rm_work | But yeah if you have cycles | 14:16 |
rm_work | Appreciated | 14:16 |
rm_work | Would love to finally knock this issue out | 14:16 |
slaweq | rm_work: but since some time we have job neutron-tempest-with-neutron-lib-master in our check queue so You should be able to test Your neutron changes with changes from neutron-lib, no? | 14:17 |
rm_work | Yeah maybe, I haven't been able to get back to it... Have been pretty much 100% focused on another higher priority issue for the past few months | 14:18 |
haleyb | rm_work, slaweq: i would remove that "with neutron lib" change from the dependency for this series, it can be done outside if needed | 14:19 |
lajoskatona | gmann: that was my feeling as well to remove depends-on, but wanted to be on the safe side :-) | 14:19 |
haleyb | slaweq: oh, i mis-understood, we can test with neutron-lib stuff now? cool | 14:20 |
gmann | lajoskatona: i have added these two as depends-on on project config last patch https://review.opendev.org/#/c/738040/ | 14:20 |
openstackgerrit | Lajos Katona proposed openstack/neutron-lib master: Add placement client methods for allocations https://review.opendev.org/741452 | 14:24 |
openstackgerrit | Brian Haley proposed openstack/neutron-lib master: Start running pylint for pep8 tests https://review.opendev.org/746552 | 14:24 |
lajoskatona | gmann: done | 14:25 |
gmann | lajoskatona: thanks | 14:25 |
*** jangutter_ has joined #openstack-neutron | 14:25 | |
lajoskatona | gmann: is it safe to merge these? (https://review.opendev.org/749579 & https://review.opendev.org/749580 ) | 14:25 |
gmann | lajoskatona: only user of it left is https://review.opendev.org/#/c/738087/ and https://review.opendev.org/#/c/738046/ | 14:28 |
*** jangutter has quit IRC | 14:28 | |
gmann | lajoskatona: so they can work on fixing with the new loaction and patches are up for them | 14:28 |
gmann | lajoskatona: i will say we can merge it | 14:29 |
lajoskatona | gmann: good, it was my feeling yesterday, as both midonet and nsx has lack of maintainers recently | 14:32 |
gmann | yeah | 14:32 |
gmann | we should not block our migration on those | 14:33 |
*** ralonsoh has quit IRC | 14:41 | |
*** ralonsoh has joined #openstack-neutron | 14:42 | |
*** manuvakery has quit IRC | 14:47 | |
*** lpetrut has quit IRC | 14:50 | |
*** manuvakery has joined #openstack-neutron | 14:59 | |
openstackgerrit | Merged openstack/neutron stable/ussuri: Don't raise FileNotFoundError during disabling keepalived https://review.opendev.org/748958 | 15:01 |
openstackgerrit | Merged openstack/neutron stable/train: Don't raise FileNotFoundError during disabling keepalived https://review.opendev.org/748959 | 15:01 |
*** jangutter has joined #openstack-neutron | 15:03 | |
*** lpetrut has joined #openstack-neutron | 15:03 | |
*** Qianbiao has quit IRC | 15:04 | |
*** ccamposr__ has joined #openstack-neutron | 15:04 | |
*** jangutter_ has quit IRC | 15:06 | |
*** ccamposr has quit IRC | 15:07 | |
*** jangutter_ has joined #openstack-neutron | 15:07 | |
*** dosaboy has quit IRC | 15:10 | |
*** jangutter has quit IRC | 15:11 | |
openstackgerrit | zhoulinhui proposed openstack/os-ken master: [part1] Remove all usage of six library https://review.opendev.org/749762 | 15:17 |
openstackgerrit | zhoulinhui proposed openstack/os-ken master: [part2] Remove all usage of six library https://review.opendev.org/749763 | 15:20 |
*** mlavalle has joined #openstack-neutron | 15:20 | |
openstackgerrit | zhoulinhui proposed openstack/os-ken master: [part3] Remove all usage of six library https://review.opendev.org/749764 | 15:23 |
*** iurygregory has quit IRC | 15:24 | |
openstackgerrit | zhoulinhui proposed openstack/os-ken master: [part4] Remove all usage of six library https://review.opendev.org/749765 | 15:25 |
*** Luzi has quit IRC | 15:28 | |
*** dtantsur is now known as dtantsur|afk | 15:31 | |
openstackgerrit | Flavio Fernandes proposed openstack/neutron master: [ovn]: port forwarding -- functional tests for maintenance task https://review.opendev.org/741731 | 15:37 |
openstackgerrit | gugug proposed openstack/os-ken master: [part1] Remove all usage of six library https://review.opendev.org/749762 | 15:37 |
openstackgerrit | gugug proposed openstack/os-ken master: [part1] Remove all usage of six library https://review.opendev.org/749762 | 15:39 |
*** rubasov has joined #openstack-neutron | 15:41 | |
*** iurygregory has joined #openstack-neutron | 15:42 | |
openstackgerrit | gugug proposed openstack/os-ken master: [part2] Remove all usage of six library https://review.opendev.org/749763 | 15:50 |
openstackgerrit | gugug proposed openstack/os-ken master: [part2] Remove all usage of six library https://review.opendev.org/749763 | 15:57 |
openstackgerrit | gugug proposed openstack/os-ken master: [part1] Remove all usage of six library https://review.opendev.org/749762 | 15:58 |
*** Inflatador has quit IRC | 16:03 | |
*** dayou has quit IRC | 16:04 | |
*** tmorin has quit IRC | 16:07 | |
*** jlibosva has quit IRC | 16:08 | |
*** bengates has quit IRC | 16:09 | |
*** bengates has joined #openstack-neutron | 16:10 | |
*** lucasagomes has quit IRC | 16:14 | |
*** bengates has quit IRC | 16:15 | |
openstackgerrit | Brian Haley proposed openstack/neutron-lib master: Start running pylint for pep8 tests https://review.opendev.org/746552 | 16:18 |
amotoki | slaweq: I proposed a release patch https://review.opendev.org/#/c/749778/ with -W. As discussed above, we are waiting for two neutron-lib changes. feel free to update it. | 16:22 |
*** priteau has quit IRC | 16:25 | |
*** adrianc has quit IRC | 16:25 | |
*** adrianc has joined #openstack-neutron | 16:27 | |
*** dayou has joined #openstack-neutron | 16:28 | |
*** ijw has quit IRC | 16:39 | |
*** ijw_ has joined #openstack-neutron | 16:39 | |
*** erbarr has joined #openstack-neutron | 16:52 | |
openstackgerrit | Brian Haley proposed openstack/neutron-lib master: Start running pylint for pep8 tests https://review.opendev.org/746552 | 16:52 |
TheJulia | Greetings folks from the land of Ironic. erbarr has reported to us that his 3rd party driver CI is failing integration testing starting yesterday morning. In digging through the logs we can see the notification being generated in the neutron service at 15:14:41 (req-bcbed87d-1d5d-43fc-abd3-fb6ed0e7245b) in | 16:56 |
TheJulia | https://stash.dellemc-community.org/logs/dellemc-poweredge-R730-R630-tempest-dsvm-ironic-idrac/2020-09-02:21:02/743868/5//logs/screen-q-svc.txt but when we look at the dhcp service log we don't see any activity corresponding. Is this a known issue? is there someplace specific that we should be looking because it looks like the jobs are failing because the machines are unable to actually get a dhcp address | 16:56 |
TheJulia | due to the port update not going through. | 16:56 |
*** lpetrut has quit IRC | 16:57 | |
*** jpena is now known as jpena|off | 17:03 | |
openstackgerrit | Mohammed Naser proposed openstack/neutron-vpnaas master: Solve usage when rootwrap is not used https://review.opendev.org/749796 | 17:03 |
openstackgerrit | Brian Haley proposed openstack/neutron-lib master: Start running pylint for pep8 tests https://review.opendev.org/746552 | 17:09 |
*** gcheresh has quit IRC | 17:12 | |
*** manuvakery has quit IRC | 17:17 | |
*** dayou has quit IRC | 17:18 | |
openstackgerrit | Mohammed Naser proposed openstack/neutron-vpnaas master: Solve usage when rootwrap is not used https://review.opendev.org/749796 | 17:25 |
*** gregwork has quit IRC | 17:29 | |
*** dayou has joined #openstack-neutron | 17:31 | |
openstackgerrit | Mohammed Naser proposed openstack/neutron-vpnaas master: Solve usage when rootwrap is not used https://review.opendev.org/749796 | 17:35 |
*** gcheresh has joined #openstack-neutron | 17:37 | |
openstackgerrit | Mohammed Naser proposed openstack/neutron-vpnaas stable/ussuri: Solve usage when rootwrap is not used https://review.opendev.org/749805 | 17:38 |
openstackgerrit | Mohammed Naser proposed openstack/neutron-vpnaas stable/train: Solve usage when rootwrap is not used https://review.opendev.org/749806 | 17:40 |
*** tesseract has quit IRC | 18:21 | |
openstackgerrit | Mamduh proposed openstack/os-vif master: Refactor code of linux_net to more cleaner and increase performace https://review.opendev.org/746673 | 18:22 |
*** priteau has joined #openstack-neutron | 18:24 | |
*** priteau has quit IRC | 18:47 | |
*** ralonsoh has quit IRC | 19:30 | |
*** tosky has quit IRC | 19:45 | |
*** lajoskatona has quit IRC | 19:48 | |
slaweq | amotoki: thx a lot | 19:57 |
slaweq | I will update it tomorrow morning | 19:57 |
*** gcheresh has quit IRC | 19:58 | |
openstackgerrit | Slawek Kaplonski proposed openstack/neutron master: Convert neutron-grenade-ovn job to be zuulv3 https://review.opendev.org/729591 | 20:03 |
*** nweinber has quit IRC | 20:04 | |
*** zzzeek has quit IRC | 20:08 | |
openstackgerrit | Slawek Kaplonski proposed openstack/neutron-lib master: Add placement client methods for allocations https://review.opendev.org/741452 | 20:09 |
openstackgerrit | Slawek Kaplonski proposed openstack/neutron-lib master: New exception QosPlacementAllocationConflict https://review.opendev.org/748279 | 20:10 |
slaweq | haleyb: I just addressed Your comment in https://review.opendev.org/#/c/741452/ - please check if that is ok for You now | 20:11 |
slaweq | mlavalle: njohnston: can You also take a look at https://review.opendev.org/#/c/741452/ ? I would like to get it merged today so tomorrow morning we can update release patch and make final neutron-lib release for victoria | 20:12 |
slaweq | thx in advance for help :) | 20:12 |
slaweq | and with that - good night for today :) | 20:12 |
haleyb | slaweq: missing space in relnote, want me to edit? | 20:12 |
*** zzzeek has joined #openstack-neutron | 20:12 | |
slaweq | haleyb: yes, please :) | 20:12 |
openstackgerrit | Brian Haley proposed openstack/neutron-lib master: Add placement client methods for allocations https://review.opendev.org/741452 | 20:13 |
haleyb | done | 20:13 |
slaweq | haleyb: thx a lot | 20:13 |
slaweq | good night :) | 20:13 |
openstackgerrit | Slawek Kaplonski proposed openstack/neutron-lib master: New exception QosPlacementAllocationConflict https://review.opendev.org/748279 | 20:13 |
*** slaweq has quit IRC | 20:20 | |
*** slaweq has joined #openstack-neutron | 20:25 | |
*** JamesBenson has quit IRC | 20:51 | |
*** JamesBenson has joined #openstack-neutron | 21:08 | |
*** raildo has quit IRC | 21:16 | |
openstackgerrit | Brian Haley proposed openstack/neutron-lib master: Start running pylint for pep8 tests https://review.opendev.org/746552 | 21:17 |
openstackgerrit | Flavio Fernandes proposed openstack/networking-ovn stable/train: [DNM] [ovn] PLEASE IGNORE. This will not be merged in stable branches https://review.opendev.org/749832 | 21:27 |
*** adrianc has quit IRC | 21:35 | |
*** adrianc has joined #openstack-neutron | 21:35 | |
openstackgerrit | Flavio Fernandes proposed openstack/networking-ovn stable/train: The general usage of verify() is if you are doing something like: https://review.opendev.org/749837 | 21:49 |
openstackgerrit | Flavio Fernandes proposed openstack/networking-ovn stable/train: Remove some unnecessary usages of verify() https://review.opendev.org/749837 | 21:51 |
openstackgerrit | Flavio Fernandes proposed openstack/neutron stable/ussuri: Remove some unnecessary usages of verify() https://review.opendev.org/747293 | 21:58 |
openstackgerrit | Flavio Fernandes proposed openstack/neutron stable/train: [DNM] [ovn] PLEASE IGNORE. This will not be merged in stable branches https://review.opendev.org/749841 | 22:18 |
*** ijw_ has quit IRC | 22:25 | |
openstackgerrit | Flavio Fernandes proposed openstack/networking-ovn stable/train: Remove some unnecessary usages of verify() https://review.opendev.org/749837 | 22:25 |
*** ijw has joined #openstack-neutron | 22:25 | |
openstackgerrit | Flavio Fernandes proposed openstack/neutron stable/train: [DNM] [ovn] PLEASE IGNORE. This will not be merged in stable branches https://review.opendev.org/749843 | 22:27 |
openstackgerrit | Merged openstack/neutron master: Rename networking-ovn-metadata-agent to neutron-ovn-metadata-agent https://review.opendev.org/746640 | 22:32 |
openstackgerrit | Merged openstack/neutron master: Use MagicMock for Connection object https://review.opendev.org/749537 | 22:32 |
openstackgerrit | Flavio Fernandes proposed openstack/neutron stable/train: [DNM] [ovn] PLEASE IGNORE. This will not be merged in stable branches https://review.opendev.org/749844 | 22:34 |
*** devfaz has quit IRC | 22:36 | |
*** devfaz has joined #openstack-neutron | 22:37 | |
openstackgerrit | Flavio Fernandes proposed openstack/networking-ovn stable/train: [DNM] [ovn] PLEASE IGNORE. This will not be merged in stable branches https://review.opendev.org/749832 | 22:50 |
*** hjensas has quit IRC | 22:56 | |
*** mlavalle has quit IRC | 23:01 | |
*** zzzeek has quit IRC | 23:05 | |
*** zzzeek has joined #openstack-neutron | 23:06 | |
openstackgerrit | Flavio Fernandes proposed openstack/neutron master: [ovn]: port forwarding -- move documentation to better place https://review.opendev.org/749846 | 23:08 |
openstackgerrit | Merged openstack/neutron stable/stein: Don't raise FileNotFoundError during disabling keepalived https://review.opendev.org/748960 | 23:19 |
*** rcernin has joined #openstack-neutron | 23:24 | |
*** ijw has quit IRC | 23:25 | |
*** ijw has joined #openstack-neutron | 23:25 | |
*** rcernin has quit IRC | 23:31 | |
*** rcernin has joined #openstack-neutron | 23:31 | |
*** zzzeek has quit IRC | 23:39 | |
*** zzzeek has joined #openstack-neutron | 23:40 | |
openstackgerrit | Merged openstack/neutron stable/train: Fix deletion of subnet_id from pd_subnets https://review.opendev.org/747845 | 23:44 |
*** jamesden_ has joined #openstack-neutron | 23:53 | |
*** jamesdenton has quit IRC | 23:56 | |
*** zzzeek has quit IRC | 23:57 | |
*** zzzeek has joined #openstack-neutron | 23:59 | |
*** sapd1_x has joined #openstack-neutron | 23:59 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!