| opendevreview | Merged openstack/neutron-tempest-plugin master: Test metadata query over IPv6 only network https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/925928 | 02:54 |
|---|---|---|
| slaweq | ralonsoh hi, do You want to get https://review.opendev.org/c/openstack/neutron-lib/+/926777 to be in the final 2024.2 release of the neutron-lib? | 07:01 |
| ralonsoh | slaweq, that would be perfect, for sure | 07:15 |
| ralonsoh | let me check the errors in the CI | 07:15 |
| ralonsoh | I'll push a new PS in some minutes | 07:15 |
| opendevreview | Merged openstack/os-ken master: Update python classifier in setup.cfg https://review.opendev.org/c/openstack/os-ken/+/906017 | 07:20 |
| lajoskatona | ricolin: Hi, a question regarding Heat and openstacksdk. | 07:27 |
| lajoskatona | ricolin: I started once to push patches to Heat (https://review.opendev.org/q/topic:%22neutron_openstacksdk%22 ) to migrate from neutronclient to SDK, I am thinking of dedicating some time again to this topic, do you think that I can find somebody who can check the patches and help me to have a good and future proof direction (as I am not a Heat expert) | 07:29 |
| opendevreview | Liushy proposed openstack/neutron master: [OVN] Support address group for ovn driver https://review.opendev.org/c/openstack/neutron/+/851509 | 08:28 |
| opendevreview | Liushy proposed openstack/neutron master: [OVN] Support address group for ovn driver https://review.opendev.org/c/openstack/neutron/+/851509 | 08:34 |
| opendevreview | Bodo Petermann proposed openstack/neutron-vpnaas master: VPNaaS support for OVN: documentation https://review.opendev.org/c/openstack/neutron-vpnaas/+/895651 | 09:05 |
| opendevreview | Tobias Urdin proposed openstack/neutron-dynamic-routing master: Ignore disabled routers for advertising https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/882560 | 09:07 |
| opendevreview | Tobias Urdin proposed openstack/neutron-dynamic-routing master: Ignore disabled routers for advertising https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/882560 | 09:12 |
| slaweq | yay, I think I found issue with networking-sfc :) | 09:26 |
| opendevreview | Bodo Petermann proposed openstack/neutron-vpnaas master: VPNaaS support for OVN: documentation https://review.opendev.org/c/openstack/neutron-vpnaas/+/895651 | 09:48 |
| lajoskatona | slaweq: really? sounds great | 09:53 |
| opendevreview | Slawek Kaplonski proposed openstack/networking-sfc master: Fix _delete_ports_flowrules_by_id so that it expects only one port_id https://review.opendev.org/c/openstack/networking-sfc/+/926892 | 10:28 |
| slaweq | lajoskatona ^^ this should fix the issue :) | 10:28 |
| slaweq | and ci jobs should be green I hope | 10:29 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-lib master: Add API extension ``quota-check-limit-default`` https://review.opendev.org/c/openstack/neutron-lib/+/926777 | 10:39 |
| lajoskatona | slaweq: checking | 11:34 |
| slaweq | it seems it failed again but I will need to check logs now | 12:28 |
| slaweq | locally this patch fixed the issue for me | 12:28 |
| rubasov | ovn-octavia folks: Can you please take a look at this new bug report? https://bugs.launchpad.net/neutron/+bug/2077399 | 12:45 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-tempest-plugin master: Switch networking-sfc job to be run with eventlet https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/926915 | 14:03 |
| opendevreview | Slawek Kaplonski proposed openstack/networking-sfc master: Fix _delete_ports_flowrules_by_id so that it expects only one port_id https://review.opendev.org/c/openstack/networking-sfc/+/926892 | 14:04 |
| ihrachys | haleyb: re https://review.opendev.org/c/openstack/neutron/+/926495/comments/bdec9a81_0e2731f3 how would you approach testing datapath is doing what we want it to do? (beyond just checking snat records of the expected form are in place)? | 14:32 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-tempest-plugin master: Switch networking-sfc job to be run with eventlet https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/926915 | 14:36 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: docs: Clarify fullstack local run procedure https://review.opendev.org/c/openstack/neutron/+/926921 | 14:39 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: docs: Clarify fullstack local run procedure https://review.opendev.org/c/openstack/neutron/+/926921 | 14:42 |
| opendevreview | Slawek Kaplonski proposed openstack/networking-sfc master: Fix _delete_ports_flowrules_by_id so that it expects only one port_id https://review.opendev.org/c/openstack/networking-sfc/+/926892 | 14:43 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Monkey patch the system libraries before calling them https://review.opendev.org/c/openstack/neutron/+/925376 | 14:48 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: DNM == Revert "Fix ObjectChangeHandler thread usage" https://review.opendev.org/c/openstack/neutron/+/926727 | 14:49 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: DNM == Make RPC event cast synchronous with the event https://review.opendev.org/c/openstack/neutron/+/926922 | 14:49 |
| ralonsoh | slaweq, ^^ the last one. I think this is what is happening with wsgi | 14:49 |
| ralonsoh | I'm pushing on top of that the DNM testing patch | 14:49 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: DNM == Test neutron-tempest-plugin-openvswitch (2) https://review.opendev.org/c/openstack/neutron/+/926788 | 14:50 |
| opendevreview | Terry Wilson proposed openstack/neutron stable/2023.2: Use oslo_service's SignalHandler for signals https://review.opendev.org/c/openstack/neutron/+/926924 | 15:08 |
| opendevreview | Terry Wilson proposed openstack/neutron stable/2023.2: Use oslo_service's SignalHandler for signals https://review.opendev.org/c/openstack/neutron/+/926924 | 15:08 |
| opendevreview | Terry Wilson proposed openstack/neutron stable/2023.2: Use oslo_service's SignalHandler for signals https://review.opendev.org/c/openstack/neutron/+/926924 | 15:08 |
| opendevreview | Terry Wilson proposed openstack/neutron stable/2023.1: Use oslo_service's SignalHandler for signals https://review.opendev.org/c/openstack/neutron/+/926926 | 15:09 |
| opendevreview | Terry Wilson proposed openstack/neutron stable/2023.1: Use oslo_service's SignalHandler for signals https://review.opendev.org/c/openstack/neutron/+/926926 | 15:10 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: docs: Clarify fullstack local run procedure https://review.opendev.org/c/openstack/neutron/+/926921 | 15:16 |
| opendevreview | Kyuyeong Lee proposed openstack/neutron-tempest-plugin master: Add test "test_sg_rule_quota_with_default_sg_rules" https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/926469 | 15:23 |
| haleyb | ihrachys: yeah, there was never any testing besides unit from what I can tell. I guess a scenario test would be best, only enabling for certain drivers for now | 15:28 |
| ihrachys | haleyb: curious - was ovn fullstack ever proposed? | 15:47 |
| * haleyb shrugs | 15:47 | |
| haleyb | i really don't know, i'm remembering kuba working on something "new" but don't know if that was part of it as it was years ago | 15:49 |
| ihrachys | aha! https://bugs.launchpad.net/neutron/+bug/1863113 | 15:55 |
| ihrachys | George for OVN! | 15:55 |
| opendevreview | Merged openstack/ovn-octavia-provider stable/2024.1: [stable/2024.1 only] Drop -master jobs https://review.opendev.org/c/openstack/ovn-octavia-provider/+/926726 | 16:23 |
| haleyb | ihrachys: ah yet, George, I only remembered it was a name | 16:31 |
| ihrachys | haleyb: one problem with the tempest scenario would be that we won't be able to enable it for ovn in gate because I think we don't have patched ovn there (?) | 16:31 |
| ihrachys | can still be validated locally / enabled when you guys patch ovn22.03 in ubuntu, but not right away | 16:32 |
| haleyb | ihrachys: right, we might only build from source in the experimental queue? | 16:32 |
| ihrachys | ah do we? I'd need to look into it. thought we only patch for fullstack and functional. | 16:32 |
| haleyb | neutron-ovn-tempest-full-multinode-ovs-master | 16:33 |
| haleyb | check experimental should trigger a run | 16:34 |
| haleyb | neutron-ovn-tempest-ovs-master is there too | 16:34 |
| ihrachys | coool | 16:35 |
| ihrachys | thanks for this, I will work on tempest then | 16:35 |
| opendevreview | Pierre Riteau proposed openstack/ovn-octavia-provider stable/2024.1: Maintenance task: do not change IPv4 ip_port_mappings https://review.opendev.org/c/openstack/ovn-octavia-provider/+/926713 | 17:21 |
| sean-k-mooney | hi folks o/ | 17:56 |
| sean-k-mooney | https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_5be/926532/2/check/python-novaclient-functional/5be70f2/testr_results.html | 17:56 |
| sean-k-mooney | we recently have started seeign failures in the nova client functionatl tests with booting a vm | 17:56 |
| sean-k-mooney | effectivly there are now mulitpe netorks visale in devstack | 17:56 |
| sean-k-mooney | are ye aware of anythin that recently change in neutron or devstack that might have cause this to start failing | 17:57 |
| sean-k-mooney | i know we have had that problem in the past with tempest | 17:57 |
| sean-k-mooney | we worked aroudn that in temest by changing the test to allow passign a network to use | 17:58 |
| sean-k-mooney | but this is testing the nova clinet cli which has been frozen for 2 years now | 17:58 |
| sean-k-mooney | so we really dont wnat to have to update the test code if we can avoid it but first we need to understand why its failing so if anyone know why this might be failing please let us know | 17:59 |
| sean-k-mooney | these are the legacy api tests meanign teh ones that test the oldest api microveion we supprot i.e. 2.1 | 18:01 |
| sean-k-mooney | so this is explictly testing boting with out specifying a netwrok and defaulting to the only visable network in the project | 18:02 |
| sean-k-mooney | that should be the private network created by devstack | 18:02 |
| sean-k-mooney | but im wondering if some of the recent rbac chagnes related to the external network may have made the public network visable? | 18:03 |
| sean-k-mooney | this is using the admin user/project so im also wondering if that is a factor | 18:05 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron-tempest-plugin master: WIP Implement nested snat validation test scenario https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/926940 | 18:11 |
| sean-k-mooney | https://bugs.launchpad.net/neutron/+bug/2071451 is now fixed by https://review.opendev.org/c/openstack/neutron/+/923040 but that may have been a public security bug | 18:16 |
| sean-k-mooney | depending on what flows were installed or not installed incorrectly as a result of that bug this might be a breach of tenant isolation and may be a high priority to backport to all stable release before stable/2023.1 goes to unmaintained | 18:17 |
| haleyb | sean-k-mooney: there could have been something recent? Looking quickly f22f7ae012e may have but merged at least a week ago | 18:18 |
| sean-k-mooney | haleyb: ya that was the one i was supicisous of | 18:18 |
| sean-k-mooney | haleyb: but i didnt have the time to parse https://github.com/openstack/neutron/commit/f22f7ae012e75b34051945fcac29f955861896ab yet to know one way or another | 18:19 |
| haleyb | being admin would be a factor as well, i just don't know off the top of my head | 18:20 |
| sean-k-mooney | i could test that by pinnning the comit of neutron to the one before that | 18:21 |
| sean-k-mooney | i.e. https://github.com/openstack/neutron/commit/2b6abf0fc08e0b1d5beae4f1536a7f49247146e7 | 18:21 |
| sean-k-mooney | i could hardcode that in a DNM with an override_checkout directive in the zuul job | 18:22 |
| opendevreview | Roberto Acosta proposed openstack/neutron master: [ML2/OVN] Add external_ids.neutron:is_snat key for SNAT rules https://review.opendev.org/c/openstack/neutron/+/907915 | 18:22 |
| sean-k-mooney | 18:24 | |
| frickler | looking at https://zuul.opendev.org/t/openstack/builds?job_name=python-novaclient-functional&project=openstack/python-novaclient the issue might be some weeks or even months old | 18:38 |
| sean-k-mooney | the lsat sucs form form may | 18:41 |
| sean-k-mooney | but also no patches for a while so its not clear | 18:41 |
| sean-k-mooney | frickler: rodofos patch meged on 2024-08-07 | 18:45 |
| sean-k-mooney | the first failure was 2024-07-28 | 18:46 |
| sean-k-mooney | so its not the rbac patch | 18:46 |
| opendevreview | Ghanshyam proposed openstack/neutron master: DNM debugging designate job https://review.opendev.org/c/openstack/neutron/+/926945 | 19:08 |
| sean-k-mooney | frickler: haleyb i think it might have been https://github.com/openstack/neutron/commit/dfc01beab22f1c2b977d3e399c3fcda69a72082d | 19:09 |
| sean-k-mooney | im not certen but the test might have been depending on that broken behaivor | 19:10 |
| opendevreview | sean mooney proposed openstack/neutron master: [DNM] Revert "Return both project_id when validating auto allocate network" https://review.opendev.org/c/openstack/neutron/+/926948 | 19:16 |
| haleyb | sean-k-mooney: interesting, i'll watch it | 19:31 |
| opendevreview | Merged openstack/neutron-tempest-plugin master: Switch networking-sfc job to be run with eventlet https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/926915 | 19:32 |
| sean-k-mooney | haleyb: i have a dnm to test it but i dont expect to proceed with the revert | 19:32 |
| haleyb | sean-k-mooney: understood | 19:33 |
| sean-k-mooney | haleyb: it woudl be a bug in our orgianl test design if that is the problem | 19:33 |
| sean-k-mooney | haleyb: https://review.opendev.org/c/openstack/python-novaclient/+/926950 is the test patch | 19:36 |
| sean-k-mooney | haleyb: before i drop for today i dont know if you saw my other message about https://bugs.launchpad.net/neutron/+bug/2071451 | 19:36 |
| *** melwitt is now known as jgwentworth | 19:38 | |
| *** jgwentworth is now known as melwitt | 19:38 | |
| haleyb | sean-k-mooney: no, hadn't seen that, thanks for letting me know | 19:38 |
| opendevreview | Brian Haley proposed openstack/neutron stable/2024.1: Always get local vlan from port other_config https://review.opendev.org/c/openstack/neutron/+/926953 | 19:39 |
| opendevreview | Brian Haley proposed openstack/neutron stable/2023.2: Always get local vlan from port other_config https://review.opendev.org/c/openstack/neutron/+/926954 | 19:40 |
| opendevreview | Brian Haley proposed openstack/neutron stable/2023.1: Always get local vlan from port other_config https://review.opendev.org/c/openstack/neutron/+/926955 | 19:40 |
| opendevreview | Merged openstack/neutron master: docs: Clarify fullstack local run procedure https://review.opendev.org/c/openstack/neutron/+/926921 | 19:46 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron-tempest-plugin master: WIP Implement nested snat validation test scenario https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/926940 | 20:17 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: ovn: don't log about rev number bumped when it was not bumped https://review.opendev.org/c/openstack/neutron/+/919536 | 20:41 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: style: Clarify a check for mixed stateful/stateless SGs https://review.opendev.org/c/openstack/neutron/+/924087 | 20:42 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron-lib master: Deprecate get_notifier(service=None) https://review.opendev.org/c/openstack/neutron-lib/+/911392 | 20:51 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron stable/2023.2: Revert "[OVN] Prevent Trunk creation/deletion with parent port bound" https://review.opendev.org/c/openstack/neutron/+/921607 | 20:54 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron stable/2023.1: Revert "[OVN] Prevent Trunk creation/deletion with parent port bound" https://review.opendev.org/c/openstack/neutron/+/921608 | 20:55 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron unmaintained/zed: Revert "[OVN] Prevent Trunk creation/deletion with parent port bound" https://review.opendev.org/c/openstack/neutron/+/921609 | 20:56 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron unmaintained/yoga: Revert "[OVN] Prevent Trunk creation/deletion with parent port bound" https://review.opendev.org/c/openstack/neutron/+/921610 | 20:56 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron unmaintained/xena: Revert "[OVN] Prevent Trunk creation/deletion with parent port bound" https://review.opendev.org/c/openstack/neutron/+/921612 | 20:57 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: DNM testing nested snat behavior with neutron gate https://review.opendev.org/c/openstack/neutron/+/926962 | 21:07 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: DNM testing nested snat behavior with neutron gate https://review.opendev.org/c/openstack/neutron/+/926963 | 21:09 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: DNM: test nested snat for ovn with everything enabled https://review.opendev.org/c/openstack/neutron/+/926964 | 21:12 |
| JayF | Howdy neutroninos or whatever you all go by; Would there by any opposition if we (GR-OSS) wanted to start a project to add kea support to neutron-dhcp-agent? I will do so for Ironic as well, and obviously we need the neutron agent on board for integrated cases. | 21:26 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!