| opendevreview | OpenStack Proposal Bot proposed openstack/neutron-vpnaas-dashboard master: Imported Translations from Zanata https://review.opendev.org/c/openstack/neutron-vpnaas-dashboard/+/856919 | 04:03 |
|---|---|---|
| opendevreview | Brian Haley proposed openstack/ovsdbapp master: Add more IDL condition comparisons https://review.opendev.org/c/openstack/ovsdbapp/+/873748 | 04:40 |
| opendevreview | OpenStack Proposal Bot proposed openstack/neutron master: Imported Translations from Zanata https://review.opendev.org/c/openstack/neutron/+/876251 | 05:33 |
| opendevreview | Felix Huettner proposed openstack/neutron master: Reduce lock contention on subnets https://review.opendev.org/c/openstack/neutron/+/875938 | 07:37 |
| ralonsoh | haleyb, ping | 08:02 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [sqlalchemy-20] The Session.begin.subtransactions flag is deprecated https://review.opendev.org/c/openstack/neutron/+/874938 | 08:04 |
| opendevreview | Merged openstack/neutron master: Imported Translations from Zanata https://review.opendev.org/c/openstack/neutron/+/876251 | 08:24 |
| opendevreview | Ke Niu proposed openstack/networking-sfc master: remove unicode prefix from code https://review.opendev.org/c/openstack/networking-sfc/+/854152 | 09:18 |
| opendevreview | Lajos Katona proposed openstack/neutron master: WIP: desperate try to make DVR functional tests more stable https://review.opendev.org/c/openstack/neutron/+/873111 | 12:21 |
| opendevreview | Lajos Katona proposed openstack/neutron master: Add Lajos Katona to Client and Doc areas as lieutenant https://review.opendev.org/c/openstack/neutron/+/875918 | 12:22 |
| opendevreview | Dr. Jens Harbott proposed openstack/neutron-dynamic-routing stable/2023.1: Add a reno for the fixed address scope calculation https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/876326 | 12:32 |
| frickler | ^^ I'm not sure how this work, just noticed that we had no reno at all for the current release, which leads to really weird results. can we do with ^^ and create and rc2? or does the reno also need to go into master first? | 12:33 |
| opendevreview | Merged openstack/neutron-dynamic-routing master: Update master for stable/2023.1 https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/875837 | 12:46 |
| opendevreview | Lajos Katona proposed openstack/python-neutronclient master: OSC: Remove BGP calls to neutronclient https://review.opendev.org/c/openstack/python-neutronclient/+/868321 | 13:14 |
| opendevreview | Lajos Katona proposed openstack/python-neutronclient master: OSC: Remove BGP calls to neutronclient https://review.opendev.org/c/openstack/python-neutronclient/+/868321 | 13:16 |
| opendevreview | Dr. Jens Harbott proposed openstack/neutron-dynamic-routing master: Add a reno for the fixed address scope calculation https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/876358 | 13:33 |
| frickler | seems there are no releasenotes rendered at all for 2023.1 in 876326, so trying the master path now ^^ | 13:34 |
| opendevreview | Felix Huettner proposed openstack/neutron master: Reduce lock contention on subnets https://review.opendev.org/c/openstack/neutron/+/875938 | 13:52 |
| lajoskatona | frickler: thanks for checking, do you think it is general issue? | 14:28 |
| haleyb | ralonsoh: pong | 14:36 |
| frickler | lajoskatona: I'm not sure yet if this is something special about n-d-r or a general issue if a project has no renos created during a cycle | 14:40 |
| frickler | oh, seems one part is a different issue. neutron releasenotes for 2023.1 also list all previous relasenotes https://docs.openstack.org/releasenotes/neutron/2023.1.html | 14:42 |
| frickler | so likely something related to the change in release name style | 14:42 |
| frickler | I'll move that to #-release | 14:43 |
| frickler | they're already on it seemingly | 14:44 |
| bcafarel | oh, I only checked that the series list looked good (with 2023.1 on top), did not open it directly :/ | 14:45 |
| lajoskatona | frickler: thanks | 14:48 |
| opendevreview | Lajos Katona proposed openstack/tap-as-a-service master: OSC: Remove calls to neutronclient https://review.opendev.org/c/openstack/tap-as-a-service/+/867786 | 14:58 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron stable/zed: Make retrieval of port mac column safe https://review.opendev.org/c/openstack/neutron/+/876220 | 15:35 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron stable/yoga: Make retrieval of port mac column safe https://review.opendev.org/c/openstack/neutron/+/876221 | 15:36 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron stable/xena: Make retrieval of port mac column safe https://review.opendev.org/c/openstack/neutron/+/876222 | 15:36 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron stable/wallaby: Make retrieval of port mac column safe https://review.opendev.org/c/openstack/neutron/+/876223 | 15:36 |
| haleyb | ralonsoh: i'm back if you're around, or if anyone else had debugged the db unit tests before :) | 16:09 |
| ralonsoh | haleyb, which ones? | 16:09 |
| haleyb | ralonsoh: https://review.opendev.org/c/openstack/neutron/+/875809/2/neutron/tests/unit/db/test_db_base_plugin_v2.py#4083 - i'm trying to figure out why doing that self.network(mtu=####) call throws an error. i know it's obvious i'm stumped though | 16:11 |
| haleyb | it's on a new change to restrict network mtu | 16:11 |
| haleyb | any mtu= value fails, not just small one, so there must be a piece of test code somewhere, i.e. whatever is getting the POST | 16:12 |
| ralonsoh | let me check | 16:13 |
| opendevreview | Brian Haley proposed openstack/neutron master: Add 2023.1 release name in routed networks doc https://review.opendev.org/c/openstack/neutron/+/876407 | 16:30 |
| haleyb | ralonsoh: so i figured out part of the problem, my change in _create_network() there is causing the failure. i still haven't figured out how to create a network there that as an mtu field in the returned dict. | 16:37 |
| ralonsoh | haleyb, the problem in the UTs is that we don't load all the API extensions | 16:39 |
| ralonsoh | we have net-mtu and net-mtu-writable | 16:39 |
| ralonsoh | these are loaded when the server starts | 16:40 |
| ralonsoh | but not in the UTs | 16:40 |
| haleyb | ralonsoh: doh, i guess i forgot about that since mtu is in the Network class definition, sigh. maybe there's another way for me to test this | 16:44 |
| ralonsoh | haleyb, we can directly call to the plugin | 16:51 |
| ralonsoh | avoiding the API | 16:51 |
| ralonsoh | actually these are plugin tests | 16:51 |
| ralonsoh | something like https://paste.opendev.org/show/byPk3drzAwwGek2miipS/ | 16:51 |
| ralonsoh | ^^ this is passing | 16:51 |
| haleyb | ralonsoh: thanks! i think i was staring at that code too long to notice | 16:54 |
| haleyb | ralonsoh: i will not be in Vancouver, but owe you a beer :) | 17:02 |
| ralonsoh | thanks! | 17:02 |
| haleyb | hopefully i can get to the next in-person, but it's graduation time here and i would be in big trouble if i went | 17:03 |
| opendevreview | Corey Bryant proposed openstack/networking-bagpipe master: Drop b1 from min versions in requirements.txt https://review.opendev.org/c/openstack/networking-bagpipe/+/876421 | 19:07 |
| opendevreview | Brian Haley proposed openstack/neutron master: WIP: Change API to validate network MTU minimums https://review.opendev.org/c/openstack/neutron/+/875809 | 22:48 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!