| opendevreview | Merged openstack/neutron master: conf: Add missing ``min``/``max`` bounds to integer config options https://review.opendev.org/c/openstack/neutron/+/995507 | 01:23 |
|---|---|---|
| opendevreview | Brian Haley proposed openstack/neutron master: Fix broad-exception-raised pylint warnings https://review.opendev.org/c/openstack/neutron/+/996576 | 01:41 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: db: Remove DVR csnat_l3_agent_bindings dead migration code https://review.opendev.org/c/openstack/neutron/+/996247 | 01:58 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: db: Remove Embrane LBaaS dead migration code https://review.opendev.org/c/openstack/neutron/+/996248 | 01:58 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: db: Remove portbindingports dead migration code https://review.opendev.org/c/openstack/neutron/+/996249 | 01:59 |
| opendevreview | Merged openstack/neutron-tempest-plugin master: [hotplug ipv6] Handle DAD stuck in tentative state https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/996223 | 02:03 |
| opendevreview | JayGue Lee proposed openstack/neutron master: Fix SNAT ambiguity for routers with multiple gateway ports https://review.opendev.org/c/openstack/neutron/+/990232 | 02:14 |
| opendevreview | Takashi Kajinami proposed openstack/neutron-lib master: Remove testing of [oslo_policy] enforce_scope = False https://review.opendev.org/c/openstack/neutron-lib/+/996581 | 02:28 |
| opendevreview | Merged openstack/neutron master: Extend timeout for getting advertised routes after restart https://review.opendev.org/c/openstack/neutron/+/996343 | 02:38 |
| opendevreview | JayGue Lee proposed openstack/neutron master: ovn: Skip HA_Chassis_Group for networks without external ports https://review.opendev.org/c/openstack/neutron/+/996584 | 03:35 |
| opendevreview | Merged openstack/neutron master: conf: Add missing ``min`` bounds to timeout and interval options https://review.opendev.org/c/openstack/neutron/+/995508 | 05:29 |
| opendevreview | Marcin Wilk proposed openstack/neutron master: Denote highest priority chassis after crash and BFD failover https://review.opendev.org/c/openstack/neutron/+/989565 | 05:30 |
| opendevreview | Merged openstack/neutron master: conf: Add missing ``min`` bounds to count and size options https://review.opendev.org/c/openstack/neutron/+/995509 | 05:37 |
| opendevreview | Eduardo Olivares proposed openstack/neutron-tempest-plugin master: Add EVPN scenario test https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/994308 | 05:49 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: ovn: Fix ``UpdateLRouterCommand`` ignoring ``if_exists`` https://review.opendev.org/c/openstack/neutron/+/995955 | 06:01 |
| opendevreview | JayGue Lee proposed openstack/neutron master: ovn: Skip HA_Chassis_Group for networks without external ports https://review.opendev.org/c/openstack/neutron/+/996584 | 06:24 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: ovn: Fix ChassisAgentTypeChangeEvent ignoring events attribute https://review.opendev.org/c/openstack/neutron/+/995936 | 06:44 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-lib master: Remove unused exception classes https://review.opendev.org/c/openstack/neutron-lib/+/996454 | 06:47 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-lib master: Remove unused ``DuplicateMeteringRuleInPost`` and ``InvalidPTRZoneConfiguration`` https://review.opendev.org/c/openstack/neutron-lib/+/996455 | 06:47 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-lib master: Remove unused firewall v2 exception classes https://review.opendev.org/c/openstack/neutron-lib/+/996456 | 06:48 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-lib master: Remove unused VPN exception classes https://review.opendev.org/c/openstack/neutron-lib/+/996457 | 06:48 |
| opendevreview | Dr. Jens Harbott proposed openstack/neutron unmaintained/2024.1: Fix cross-project access to router conntrack helpers https://review.opendev.org/c/openstack/neutron/+/996230 | 06:58 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Add agentic coding guidance and docs https://review.opendev.org/c/openstack/neutron/+/996068 | 07:09 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: ai: Add code-review agent guide https://review.opendev.org/c/openstack/neutron/+/996106 | 07:09 |
| opendevreview | Takashi Kajinami proposed openstack/neutron-lib master: Remove testing of [oslo_policy] enforce_scope = False https://review.opendev.org/c/openstack/neutron-lib/+/996581 | 07:27 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: ai: Add Launchpad bug triage agent https://review.opendev.org/c/openstack/neutron/+/996231 | 07:47 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: ai: Add git commit message formatting agent https://review.opendev.org/c/openstack/neutron/+/996233 | 07:51 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/2026.1: ovn: Fix HAChassisGroupRouterEvent setting options.chassis on physnet routers https://review.opendev.org/c/openstack/neutron/+/996611 | 07:52 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/2025.2: ovn: Fix HAChassisGroupRouterEvent setting options.chassis on physnet routers https://review.opendev.org/c/openstack/neutron/+/996612 | 07:52 |
| *** elodilles_pto is now known as elodilles | 07:52 | |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/2025.1: ovn: Fix HAChassisGroupRouterEvent setting options.chassis on physnet routers https://review.opendev.org/c/openstack/neutron/+/996613 | 07:56 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: db: Remove VMware/NSX dead migration code https://review.opendev.org/c/openstack/neutron/+/996239 | 08:11 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: db: Remove Brocade dead migration code https://review.opendev.org/c/openstack/neutron/+/996240 | 08:11 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: db: Remove Nuage dead migration code https://review.opendev.org/c/openstack/neutron/+/996241 | 08:12 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: db: Remove NEC dead migration code https://review.opendev.org/c/openstack/neutron/+/996242 | 08:12 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: db: Remove Metaplugin and BigSwitch dead migration code https://review.opendev.org/c/openstack/neutron/+/996243 | 08:12 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: db: Remove OVS monolithic plugin dead migration code https://review.opendev.org/c/openstack/neutron/+/996244 | 08:12 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: db: Remove LinuxBridge monolithic plugin dead migration code https://review.opendev.org/c/openstack/neutron/+/996245 | 08:13 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: db: Remove IP availability ranges dead migration code https://review.opendev.org/c/openstack/neutron/+/996246 | 08:15 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: db: Remove IP availability ranges dead migration code https://review.opendev.org/c/openstack/neutron/+/996246 | 08:16 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: db: Remove DVR csnat_l3_agent_bindings dead migration code https://review.opendev.org/c/openstack/neutron/+/996247 | 08:17 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: db: Remove Embrane LBaaS dead migration code https://review.opendev.org/c/openstack/neutron/+/996248 | 08:17 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: db: Remove portbindingports dead migration code https://review.opendev.org/c/openstack/neutron/+/996249 | 08:17 |
| opendevreview | Tristan proposed openstack/ovn-bgp-agent master: Make ARP/NDP IP offset configurable https://review.opendev.org/c/openstack/ovn-bgp-agent/+/996620 | 09:15 |
| opendevreview | Adam Harwell proposed openstack/neutron master: Add ASGI worker count support to wsgi_utils https://review.opendev.org/c/openstack/neutron/+/995162 | 09:34 |
| opendevreview | Merged openstack/neutron master: db: Remove ``cisco_init_ops`` dead migration code https://review.opendev.org/c/openstack/neutron/+/996236 | 09:50 |
| opendevreview | Merged openstack/neutron master: db: Remove Cisco ML2 Nexus dead migration code https://review.opendev.org/c/openstack/neutron/+/996238 | 09:50 |
| opendevreview | Tristan proposed openstack/ovn-bgp-agent master: Make ARP/NDP IP offset configurable https://review.opendev.org/c/openstack/ovn-bgp-agent/+/996620 | 09:54 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-lib master: Remove unused VPN exception classes https://review.opendev.org/c/openstack/neutron-lib/+/996457 | 10:09 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Add agentic coding guidance and docs https://review.opendev.org/c/openstack/neutron/+/996068 | 10:15 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: OVN maintenance log ``start_message`` should be a string https://review.opendev.org/c/openstack/neutron/+/995965 | 10:19 |
| opendevreview | Michal Nasiadka proposed openstack/neutron master: Fix load_db_migration_drivers to work with callable plugins https://review.opendev.org/c/openstack/neutron/+/996632 | 11:43 |
| opendevreview | Michal Nasiadka proposed openstack/neutron master: Fix load_db_migration_drivers to work with callable plugins https://review.opendev.org/c/openstack/neutron/+/996632 | 11:45 |
| opendevreview | Takashi Kajinami proposed openstack/neutron-tempest-plugin master: Skip plugin tests for neutron-lib tests update https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/996637 | 12:23 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: quota: Fix reservation under-counting in ``get_total_reservations_map()`` https://review.opendev.org/c/openstack/neutron/+/996266 | 13:23 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: quota: Fix reservation under-counting in ``get_total_reservations_map()`` https://review.opendev.org/c/openstack/neutron/+/996266 | 14:01 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-tempest-plugin master: [FWaaS] Add basic scenario test for FW attached to L2 ports https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/995695 | 14:25 |
| opendevreview | Merged openstack/neutron master: OVN maintenance log ``start_message`` should be a string https://review.opendev.org/c/openstack/neutron/+/995965 | 14:35 |
| opendevreview | Merged openstack/neutron master: doc: Remove Linux bridge mechanism driver references https://review.opendev.org/c/openstack/neutron/+/996063 | 14:35 |
| opendevreview | Merged openstack/neutron-tempest-plugin master: Skip plugin tests for neutron-lib tests update https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/996637 | 14:46 |
| opendevreview | Merged openstack/neutron-lib master: adjust required Python to 3.11 and newer https://review.opendev.org/c/openstack/neutron-lib/+/996360 | 14:47 |
| winiciusallan[m] | hey neutron o/ have we done any change on the agent listing API? | 14:48 |
| winiciusallan[m] | we've seen some jobs failing on gophercloud since Jun 3rd | 14:48 |
| slaweq | what are the errors you have there? Do you have any links? | 14:49 |
| winiciusallan[m] | yup, one sec | 14:49 |
| winiciusallan[m] | in the failing step you can see the error https://github.com/gophercloud/gophercloud/actions/runs/28680796509/job/85063702391?pr=3831 | 14:49 |
| winiciusallan[m] | the listing call is returning 0 instead of 1 or more | 14:50 |
| winiciusallan[m] | Failure in l3_scheduling_test.go, line 47: The first value "0" is lesser than the second value "1" | 14:50 |
| haleyb | i do see the usage of tenant_id there, which is Ok for now, but is being deprecated, and probably not the issue | 14:53 |
| slaweq | is this running on ML2/OVN backend? If yes, there is no L3 agents in such case | 14:53 |
| haleyb | what exactly is l3_scheduling_test.go, line 47 doing? | 14:53 |
| slaweq | haleyb: the line is here https://github.com/gophercloud/gophercloud/blob/main/internal/acceptance/openstack/networking/v2/extensions/layer3/l3_scheduling_test.go#L47 I guess | 14:54 |
| ralonsoh | It requires extension `l3_agent_scheduler`, so ML2/OVS | 14:55 |
| winiciusallan[m] | IIRC it is running with ML2/OVN | 14:56 |
| winiciusallan[m] | https://github.com/gophercloud/gophercloud/blob/e83bbbc01ec2a121f45f368e301fa27a1270c606/.github/workflows/functional-networking.yaml#L63 | 14:57 |
| winiciusallan[m] | in the e2e file we don't define the ML2 plugin, so I believe it uses the devstack default | 14:57 |
| slaweq | I see it should be skipped if there is no l3_agent_scheduler extension: https://github.com/gophercloud/gophercloud/blob/904024a283b8b81510f11d3995f202774a4aaf7d/internal/acceptance/openstack/networking/v2/extensions/layer3/l3_scheduling_test.go#L22 | 14:57 |
| slaweq | so the question now is why this extension is reported as active in the ML2/OVN setup | 14:57 |
| slaweq | IMHO it shouldn't be | 14:57 |
| slaweq | but I would need to check it and I'm about to leave know | 14:58 |
| slaweq | I can take a look at it tomorrow morning my time | 14:58 |
| slaweq | winiciusallan: please open LP bug for it and if possible please share there link to all of the logs from that failed job | 14:58 |
| winiciusallan[m] | alright | 14:59 |
| winiciusallan[m] | I'll take a look at the neutron logs to see whether the extension is active | 14:59 |
| haleyb | The l3-agent-scheduler-ha-priority extension merged around that time | 14:59 |
| haleyb | June 2nd/3rd | 15:00 |
| slaweq | haleyb: and this can be root cause of the issue | 15:00 |
| slaweq | I mean the explanation why the test will need to be updated now maybe | 15:00 |
| haleyb | sure, let's get a bug so someone can work on it | 15:01 |
| opendevreview | Marcin Wilk proposed openstack/neutron master: Denote highest priority chassis after crash and BFD failover https://review.opendev.org/c/openstack/neutron/+/989565 | 15:03 |
| ralonsoh | most problably that could be caused by https://review.opendev.org/c/openstack/neutron/+/982792. | 15:04 |
| ralonsoh | The env is using ML2/OVN, as reported in the devstack deployment logs | 15:04 |
| ralonsoh | I don't know, to be honest, why is expecting this extension `l3_agent_scheduler`, that should not be enabled with OVN | 15:05 |
| ralonsoh | OVN didn't report before the agent scheduling | 15:05 |
| ralonsoh | now it does | 15:05 |
| winiciusallan[m] | https://bugs.launchpad.net/neutron/+bug/2160241 | 15:05 |
| ralonsoh | winiciusallan[m], your test needs to create a router with a external network GW | 15:06 |
| ralonsoh | that will provide the hostings agent information | 15:06 |
| winiciusallan[m] | may I ask what has changed to need the external network gateway? | 15:07 |
| ralonsoh | winiciusallan[m], you were using an API that was not available for ML2/OVN | 15:07 |
| ralonsoh | I don't know how it was working | 15:07 |
| ralonsoh | now ML2/OVN provide information about the chassis binding --> agent host binding | 15:08 |
| ralonsoh | if you don't have an external network connected to a router, the router (the GW logical_router_port) won't exist and won't be connected to any chassis==host | 15:09 |
| winiciusallan[m] | that makes sense | 15:11 |
| winiciusallan[m] | I'll modify the tests to see how it goes | 15:11 |
| winiciusallan[m] | ralonsoh: yeah... it worked | 15:25 |
| winiciusallan[m] | many thanks | 15:27 |
| winiciusallan[m] | I'll push to see if the CI is happy | 15:27 |
| opendevreview | Merged openstack/neutron master: ovn: Skip LSP host info update for router ports https://review.opendev.org/c/openstack/neutron/+/996084 | 15:27 |
| ralonsoh | nice to read that | 15:32 |
| opendevreview | Merged openstack/neutron-lib master: hacking: Remove N524 ``check_no_contextlib_nested`` https://review.opendev.org/c/openstack/neutron-lib/+/996480 | 15:50 |
| opendevreview | Merged openstack/neutron-lib master: Remove unused ``DBAPIContextManagerFixture`` https://review.opendev.org/c/openstack/neutron-lib/+/996453 | 15:50 |
| opendevreview | Merged openstack/neutron master: conf: Add missing ``min`` bounds to HA conntrackd options https://review.opendev.org/c/openstack/neutron/+/995510 | 16:02 |
| opendevreview | Michal Nasiadka proposed openstack/neutron master: Fix load_db_migration_drivers https://review.opendev.org/c/openstack/neutron/+/996632 | 16:52 |
| opendevreview | Michal Nasiadka proposed openstack/neutron master: Fix load_db_migration_drivers https://review.opendev.org/c/openstack/neutron/+/996632 | 16:53 |
| opendevreview | Michal Nasiadka proposed openstack/neutron master: Fix load_db_migration_drivers https://review.opendev.org/c/openstack/neutron/+/996632 | 17:05 |
| opendevreview | Merged openstack/neutron master: db: Remove VMware/NSX dead migration code https://review.opendev.org/c/openstack/neutron/+/996239 | 17:13 |
| opendevreview | Merged openstack/neutron master: db: Remove Brocade dead migration code https://review.opendev.org/c/openstack/neutron/+/996240 | 17:18 |
| opendevreview | Merged openstack/neutron master: ovn: Fix ``UpdateLRouterCommand`` ignoring ``if_exists`` https://review.opendev.org/c/openstack/neutron/+/995955 | 17:18 |
| opendevreview | Merged openstack/neutron master: ovn: Fix ChassisAgentTypeChangeEvent ignoring events attribute https://review.opendev.org/c/openstack/neutron/+/995936 | 17:18 |
| opendevreview | Merged openstack/neutron-lib master: Remove testing of [oslo_policy] enforce_scope = False https://review.opendev.org/c/openstack/neutron-lib/+/996581 | 18:02 |
| -Guest12850- NOTICE: The Gerrit service on review.opendev.org will be offline briefly one hour from now, at 21:00 UTC, while we rename a project. | 20:01 | |
| opendevreview | Merged openstack/neutron master: db: Remove Nuage dead migration code https://review.opendev.org/c/openstack/neutron/+/996241 | 20:54 |
| opendevreview | Merged openstack/neutron master: db: Remove NEC dead migration code https://review.opendev.org/c/openstack/neutron/+/996242 | 20:54 |
| opendevreview | Merged openstack/neutron master: db: Remove Metaplugin and BigSwitch dead migration code https://review.opendev.org/c/openstack/neutron/+/996243 | 20:55 |
| -Guest12850- NOTICE: The Gerrit service on review.opendev.org is going offline momentarily at 21:00 UTC while we rename a project, but will return within a few minutes. | 20:59 | |
| *** avanzaghi13 is now known as avanzaghi1 | 22:37 | |
| opendevreview | Merged openstack/neutron-lib master: [FWaaS] Set 'is_filter' for the attributes which allows filtering https://review.opendev.org/c/openstack/neutron-lib/+/995265 | 23:26 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!