| opendevreview | yatin proposed openstack/neutron master: Drop retries in tests for TimeoutException https://review.opendev.org/c/openstack/neutron/+/882376 | 06:42 |
|---|---|---|
| sahid | o/ | 07:30 |
| ralonsoh | hi folks, please check https://review.opendev.org/q/project:openstack/neutron-lib+status:open | 07:46 |
| ralonsoh | next week I'll propose in the team meeting to release a new n-lib version | 07:46 |
| slaweq | ralonsoh lajoskatona bcafarel elodilles hi, I'm looking at backports https://review.opendev.org/q/I5705c490bcd36e7e2edcc62954a3ffa0ff645519 now and I'm (again) not sure if we should backport things which bumps minimal version of some dependency. IIRC we should avoid that in backports but maybe I again forgot about something | 07:46 |
| slaweq | can You check it maybe? | 07:46 |
| slaweq | froyo ^^ it's your backport :) | 07:47 |
| froyo | slaweq yeah | 07:47 |
| ralonsoh | right, same as in older versions, you should implemente the ovsdbapp methods in ovn-octavia-provider | 07:49 |
| froyo | I just be able to bump for stable/zed and 2023.1, in the older ones I need to "move" the new ovsdbapp methods to local (adding a big comment to remove when the bump could be done) | 07:49 |
| ralonsoh | https://review.opendev.org/c/openstack/ovn-octavia-provider/+/880954/2/ovn_octavia_provider/ovsdb/impl_idl_ovn.py | 07:49 |
| ralonsoh | slaweq's concern is legit | 07:49 |
| ralonsoh | you should do the same even in 2023.1 and zed | 07:50 |
| froyo | ralonsoh: so keep the bump only in the master one? | 07:50 |
| ralonsoh | yes | 07:50 |
| froyo | ack! | 07:50 |
| ralonsoh | wait for lajoskatona and bcafarel opinion on this (and elodilles of course) | 07:50 |
| froyo | sure | 07:51 |
| slaweq | ralonsoh that was my understanding and I remember we were doing things like that in the past | 07:52 |
| slaweq | but I was a bit puzzled as I saw +2 from bcafarel already there :) | 07:53 |
| slaweq | ralonsoh++ thx for confirmation :) | 07:53 |
| slaweq | froyo please update backports and ping me to review it | 07:53 |
| ltomasbo | slaweq, ralonsoh: shouldn't it be fine if it is within the upper-constraint requirements? | 07:54 |
| ltomasbo | (just a naive question) | 07:54 |
| ralonsoh | not in stable releases | 07:54 |
| ralonsoh | we should not bump requirements for new features | 07:54 |
| slaweq | it may cause problems for distros which ships it | 07:54 |
| ralonsoh | just bug fixes | 07:54 |
| ltomasbo | ok, so only if the bump is due to a bug in that library that requires fixing, got it | 07:55 |
| elodilles | i agree with all you have said :) | 08:00 |
| opendevreview | Danylo Vodopianov proposed openstack/neutron master: Openvswitch driver with Virtio-Forwarder was extended https://review.opendev.org/c/openstack/neutron/+/869510 | 08:04 |
| slaweq | thx elodilles for confirmation :) | 08:05 |
| dvo-plv | ralonsoh: Hello, Do your team have some comment to our spec file in the next bp: https://review.opendev.org/c/openstack/nova-specs/+/859290 ? | 08:06 |
| ralonsoh | dvo-plv, sorry, I didn't have time to review that yet | 08:07 |
| ralonsoh | I'll add it to the spec section in our weekly meeting | 08:07 |
| dvo-plv | Should I visit this meeting? | 08:08 |
| ralonsoh | that will be just a reminder for the team | 08:09 |
| ralonsoh | in any case, any developer in Neutron should attend it weekly | 08:09 |
| dvo-plv | okay, thank you | 08:10 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider stable/2023.1: Use ovsdbapp commands to add/del backends to ip_port_mappings https://review.opendev.org/c/openstack/ovn-octavia-provider/+/880951 | 08:18 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider stable/zed: Use ovsdbapp commands to add/del backends to ip_port_mappings https://review.opendev.org/c/openstack/ovn-octavia-provider/+/880952 | 08:25 |
| opendevreview | yatin proposed openstack/neutron master: Drop retries in tests for TimeoutException https://review.opendev.org/c/openstack/neutron/+/882376 | 08:47 |
| opendevreview | huanghailun proposed openstack/neutron master: Add allowed_address_pairs add/remove atomic operations https://review.opendev.org/c/openstack/neutron/+/880922 | 08:48 |
| opendevreview | huanghailun proposed openstack/neutron master: Add allowed_address_pairs add/remove atomic operations https://review.opendev.org/c/openstack/neutron/+/880922 | 09:50 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider stable/yoga: Use ovsdbapp commands to add/del backends to ip_port_mappings https://review.opendev.org/c/openstack/ovn-octavia-provider/+/880953 | 10:22 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider stable/wallaby: Use ovsdbapp commands to add/del backends to ip_port_mappings https://review.opendev.org/c/openstack/ovn-octavia-provider/+/880955 | 10:22 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider stable/xena: Use ovsdbapp commands to add/del backends to ip_port_mappings https://review.opendev.org/c/openstack/ovn-octavia-provider/+/880954 | 10:22 |
| opendevreview | Merged openstack/neutron-lib master: DPDK datapath type added https://review.opendev.org/c/openstack/neutron-lib/+/859573 | 10:42 |
| dvo-plv | ralonsoh: My neutron-lib patch was merged by zuul. Whom should I ping regarding new neutron-lib release? | 10:51 |
| ralonsoh | dvo-plv, did you read what I wrote in this channel? | 10:52 |
| opendevreview | Merged openstack/neutron-lib master: Remove an unused exception https://review.opendev.org/c/openstack/neutron-lib/+/882334 | 10:52 |
| dvo-plv | Soory, look liek I miss something, what do you mean ? | 10:53 |
| ralonsoh | I talked about this. I'm asking people to review the open patches in n-lib in order to release a new version asap | 10:54 |
| dvo-plv | I see, I was absent, thank you, sorry for disturbance | 10:56 |
| opendevreview | Merged openstack/neutron master: [S-RBAC] Switch to new policies by default https://review.opendev.org/c/openstack/neutron/+/879827 | 11:10 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [S-RBAC] Get QoS rule types API available for READER role https://review.opendev.org/c/openstack/neutron/+/882414 | 11:14 |
| ralonsoh | slaweq, please check ^^ | 11:14 |
| ralonsoh | btw, should we add this role constant to policies.base? There are other places using it | 11:16 |
| ralonsoh | (that could be a follow up patch) | 11:17 |
| opendevreview | Merged openstack/ovn-octavia-provider stable/2023.1: Use ovsdbapp commands to add/del backends to ip_port_mappings https://review.opendev.org/c/openstack/ovn-octavia-provider/+/880951 | 11:20 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [WIP] Fix race condition when creating two routers without HA network https://review.opendev.org/c/openstack/neutron/+/881826 | 12:59 |
| ralonsoh | slaweq, https://56c3c5b7dfb0cd1d57d7-ee26557ecac08e7f1d28df56ff28c0f0.ssl.cf2.rackcdn.com/882414/1/check/neutron-tempest-plugin-openvswitch-enforce-scope-new-defaults/fe601ba/testr_results.html | 13:05 |
| ralonsoh | I think with this patch I've surfaced new bugs | 13:06 |
| opendevreview | Merged openstack/neutron stable/xena: Avoid retrieving ports if network list is empty https://review.opendev.org/c/openstack/neutron/+/882122 | 13:06 |
| opendevreview | Merged openstack/neutron stable/zed: Avoid retrieving ports if network list is empty https://review.opendev.org/c/openstack/neutron/+/882120 | 13:27 |
| opendevreview | Merged openstack/neutron stable/wallaby: Avoid retrieving ports if network list is empty https://review.opendev.org/c/openstack/neutron/+/882123 | 13:27 |
| opendevreview | Merged openstack/ovn-octavia-provider stable/zed: Use ovsdbapp commands to add/del backends to ip_port_mappings https://review.opendev.org/c/openstack/ovn-octavia-provider/+/880952 | 13:34 |
| opendevreview | Merged openstack/ovn-octavia-provider stable/yoga: Use ovsdbapp commands to add/del backends to ip_port_mappings https://review.opendev.org/c/openstack/ovn-octavia-provider/+/880953 | 13:36 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Fix race condition when creating two routers without HA network https://review.opendev.org/c/openstack/neutron/+/881826 | 13:36 |
| mtomaska | I noticed that drivers meeting was canceled for today. I wanted to bring up LP2018009 . Not sure if this is a bug or design. Basically, context object does not hold the tag value although it does contain tags key. | 14:06 |
| mtomaska | https://bugs.launchpad.net/neutron/+bug/2018009 | 14:06 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: rbacs: clean-up to use defined constants ACCESS_* https://review.opendev.org/c/openstack/neutron/+/882445 | 14:08 |
| ralonsoh | mtomaska, what is happening with this bug> | 14:09 |
| ralonsoh | ? | 14:09 |
| slaweq | ralonsoh checking it right now | 14:12 |
| mtomaska | the OP is basically trying to use the tag value to pass information to their ML2 driver and states that context object does not hold the tag value. I can confirm this on a master branch. However, I am wondering if that is actually appropriate use of tags arg | 14:12 |
| ralonsoh | mtomaska, if you need to discuss this bug you can add it to the next neutron meeting agenda (next tuesday) or to the drivers meeting agenda | 14:12 |
| mtomaska | yep, will do. I miss it for today | 14:12 |
| ralonsoh | mtomaska, maybe (only maybe, I need to confirm) we are not printing this info | 14:12 |
| ralonsoh | so the plugin should have a "extend_dict" method | 14:13 |
| ralonsoh | to populate the tag in the network dicr | 14:13 |
| ralonsoh | but I didn't check the code yet | 14:13 |
| ralonsoh | mtomaska, actually the method "_extend_tags_dict" | 14:15 |
| ralonsoh | that should work | 14:15 |
| slaweq | ralonsoh I have no idea why it's like that. I will need to deploy devstack and check it locally | 14:16 |
| slaweq | can it be in next week? | 14:16 |
| ralonsoh | slaweq, for sure, no rush at all | 14:17 |
| ralonsoh | I'll check it too once I finish a couple of tasks | 14:17 |
| ralonsoh | that bug was opened by ccamposr | 14:17 |
| slaweq | yeah, I know | 14:17 |
| slaweq | he spoke with me about it before he opened it :) | 14:17 |
| slaweq | thx for taking care of the bug so fast | 14:18 |
| ccamposr | :) | 14:18 |
| slaweq | I added it to my todo for next week | 14:18 |
| ralonsoh | perfect! | 14:18 |
| ccamposr | I respect to my devs :D | 14:19 |
| ccamposr | in fact, I fixed it locally and I think I have to open other extra bug :( | 14:20 |
| ralonsoh | why for? | 14:20 |
| ralonsoh | well, actually I think the CI has detected new errors | 14:20 |
| ralonsoh | related to QoS, once fixed the LP bug you opened | 14:20 |
| ralonsoh | so there could be more, for sure | 14:20 |
| ccamposr | it is related to srbac | 14:20 |
| ralonsoh | mtomaska, the problem is in the OSC client | 14:24 |
| ralonsoh | we are not parsing the tags | 14:24 |
| ralonsoh | at least in the network creation command | 14:25 |
| ralonsoh | if you check "_get_attrs_network" in openstackclient.network.v2.network | 14:25 |
| ralonsoh | this parser is not extracting "tags" from parsed_args | 14:26 |
| ralonsoh | this is a legit bug but for OSC | 14:26 |
| mtomaska | thanks ralonsoh. But from general ML2 framework guidelines. Is it ok for a ml2 driver to pass information via tags? I thought that tags are just for the end user to tag an object. We can discuss in the meeting | 14:31 |
| ralonsoh | mtomaska, why not? this is another resource attribute | 14:31 |
| ralonsoh | and if you enable the service plugin, it adds this info to the resource dict | 14:32 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: rpc/handlers/dhcp: fix get_network_info when no local subnets https://review.opendev.org/c/openstack/neutron/+/880131 | 14:32 |
| ralonsoh | this should be in the information passed | 14:32 |
| ralonsoh | but the problem is bigger: now we cannot create networks with tags | 14:32 |
| ralonsoh | and I've tested only network | 14:32 |
| ralonsoh | there are 18 resources that could be tagged | 14:32 |
| ralonsoh | https://paste.opendev.org/show/bofZrp7xwcWHbP7A3M1d/ | 14:32 |
| ralonsoh | --> https://bugs.launchpad.net/neutron/+bug/2018009/comments/4 | 14:33 |
| mtomaska | Thanks ralonsoh. If that the intended use then I agree its a bug. I will get myself more familiar with that code and propose a fix | 14:41 |
| opendevreview | Merged openstack/ovn-octavia-provider stable/wallaby: Use ovsdbapp commands to add/del backends to ip_port_mappings https://review.opendev.org/c/openstack/ovn-octavia-provider/+/880955 | 14:41 |
| ralonsoh | mtomaska, perfect. Please ping me for whatever you need | 14:41 |
| opendevreview | Tobias Fischer proposed openstack/neutron master: Reduce lock contention on subnets https://review.opendev.org/c/openstack/neutron/+/881943 | 15:01 |
| sahid | 15:16 | |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-specs master: Port extension to create hardware offloaded ports https://review.opendev.org/c/openstack/neutron-specs/+/882272 | 15:21 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron-lib master: [WIP] rbacs: filter out model that are already owned by context https://review.opendev.org/c/openstack/neutron-lib/+/870557 | 15:26 |
| sahid | quick question guys, regarding network list --share | 15:28 |
| sahid | what is the expected behavior of --share | 15:29 |
| sahid | we want the list of the networks owned + networks shared | 15:29 |
| sahid | or only the list of network shared? | 15:29 |
| sahid | networks shared | 15:29 |
| ralonsoh | sahid, did you check the output? without this parameter you'll print all network | 15:34 |
| ralonsoh | with --share only the shared networks | 15:35 |
| ralonsoh | with --no-share, those not shared | 15:35 |
| ralonsoh | https://paste.opendev.org/show/bGcTBRjFzc0pSFtYhPEJ/ | 15:35 |
| sahid | hum... | 15:45 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-specs master: Port extension to create hardware offloaded ports https://review.opendev.org/c/openstack/neutron-specs/+/882272 | 15:45 |
| opendevreview | Merged openstack/ovn-octavia-provider stable/xena: Use ovsdbapp commands to add/del backends to ip_port_mappings https://review.opendev.org/c/openstack/ovn-octavia-provider/+/880954 | 16:42 |
| opendevreview | Elvira García Ruiz proposed openstack/neutron master: [OVN] Update ovn meter when neutron server reloads https://review.opendev.org/c/openstack/neutron/+/880982 | 17:05 |
| opendevreview | Merged openstack/neutron master: Reduce lock contention on subnets https://review.opendev.org/c/openstack/neutron/+/881943 | 20:12 |
| opendevreview | Merged openstack/neutron master: Drop retries in tests for TimeoutException https://review.opendev.org/c/openstack/neutron/+/882376 | 20:16 |
| opendevreview | Dmitrii Shcherbakov proposed openstack/neutron master: Allow Multiple External Gateways https://review.opendev.org/c/openstack/neutron/+/873593 | 23:12 |
| opendevreview | Dmitrii Shcherbakov proposed openstack/neutron master: Allow Multiple External Gateways https://review.opendev.org/c/openstack/neutron/+/873593 | 23:54 |
| opendevreview | Dmitrii Shcherbakov proposed openstack/neutron master: Add extra router attributes for ECMP and BFD https://review.opendev.org/c/openstack/neutron/+/874797 | 23:54 |
| opendevreview | Dmitrii Shcherbakov proposed openstack/neutron master: [ovn] Implement support for external-gateway-multihoming extension https://review.opendev.org/c/openstack/neutron/+/874199 | 23:54 |
| opendevreview | Dmitrii Shcherbakov proposed openstack/neutron master: [ovn] Honor `enable_default_route_ecmp` attribute https://review.opendev.org/c/openstack/neutron/+/878531 | 23:54 |
| opendevreview | Dmitrii Shcherbakov proposed openstack/neutron master: [ovn] Allow L3 scheduler to be aware of current transaction https://review.opendev.org/c/openstack/neutron/+/874760 | 23:54 |
| opendevreview | Dmitrii Shcherbakov proposed openstack/neutron master: [ovn] Add helper for retrieving LR associated with LRP https://review.opendev.org/c/openstack/neutron/+/873698 | 23:54 |
| opendevreview | Dmitrii Shcherbakov proposed openstack/neutron master: [ovn] Apply soft anti-affinity for LRs with multiple LRPs when scheduling https://review.opendev.org/c/openstack/neutron/+/873699 | 23:54 |
| opendevreview | Dmitrii Shcherbakov proposed openstack/neutron master: [ovn] Add support for enable_default_route_bfd attribute https://review.opendev.org/c/openstack/neutron/+/878543 | 23:54 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!