| opendevreview | OpenStack Proposal Bot proposed openstack/neutron-fwaas-dashboard master: Imported Translations from Zanata https://review.opendev.org/c/openstack/neutron-fwaas-dashboard/+/993708 | 03:26 |
|---|---|---|
| LarsErik1 | haleyb: good morning! I'll give that patch a try. I can post what we tried as well. We didn't really get copilot to write the patch, we just promted it with the haproxy config and the request/response headers we got. Then it suggested what could be the issue. | 06:46 |
| *** LarsErik1 is now known as LarsErikP | 06:48 | |
| LarsErikP | haleyb: good morning! I'll give that patch a try. I can post what we tried as well. We didn't really get copilot to write the patch, we just promted it with the haproxy config and the request/response headers we got. Then it suggested what could be the issue. | 06:49 |
| LarsErikP | sorry if that was a double post. got some network quirks :P | 06:49 |
| LarsErikP | haleyb: tested that patch now. Can confirm it is fixing my issue =) Posted on LP as well | 06:55 |
| *** elodilles_pto is now known as elodilles | 07:50 | |
| opendevreview | Bharath M V proposed openstack/neutron-tempest-plugin master: [TaaS] Add a driver option to taas test config. https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/993591 | 08:07 |
| opendevreview | Lajos Katona proposed openstack/neutron master: Use SDK for Nova notifications https://review.opendev.org/c/openstack/neutron/+/983905 | 08:33 |
| opendevreview | Bharath M V proposed openstack/neutron-tempest-plugin master: [TaaS] Add a driver option to taas test config. https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/993591 | 08:35 |
| eolivare | test_ovn_db_sync_repair | 09:41 |
| eolivare | sorry, I wanted to say that test_ovn_db_sync_repair started failing quite often since June 12: https://zuul.opendev.org/t/openstack/builds?job_name=whitebox-neutron-tempest-plugin-ovn&pipeline=periodic&skip=0 | 09:42 |
| opendevreview | Bharath M V proposed openstack/neutron-tempest-plugin master: [Tap-as-a-Service] Adding 2 Tap as a service test cases. https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/989931 | 09:44 |
| opendevreview | Bharath M V proposed openstack/neutron-tempest-plugin master: [Tap-as-a-Service] Adding 2 Tap as a service test cases. https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/989931 | 09:45 |
| opendevreview | Renjing Xiao proposed x/whitebox-neutron-tempest-plugin master: Add tempest test for multiple segments per host on routed provider networks https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/988332 | 09:56 |
| opendevreview | Bekei Park proposed openstack/ovn-bgp-agent master: Fix OVN LB VIP never being advertised (cr-lrp router key mismatch) https://review.opendev.org/c/openstack/ovn-bgp-agent/+/993732 | 11:07 |
| opendevreview | Merged openstack/neutron master: Remove UnknownNetworkType InvalidAddressRequest and IPAllocationFailed https://review.opendev.org/c/openstack/neutron/+/993144 | 11:10 |
| opendevreview | Eduardo Olivares proposed x/whitebox-neutron-tempest-plugin master: Fix node_counts string formatting TypeError in error path https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/993736 | 11:33 |
| opendevreview | Eduardo Olivares proposed x/whitebox-neutron-tempest-plugin master: Add unit tests for node discovery summary logging https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/993737 | 11:33 |
| opendevreview | Merged openstack/neutron master: Explicitly create log directories in the FrrFixture https://review.opendev.org/c/openstack/neutron/+/993199 | 11:45 |
| stephenfin | lajoskatona: Could I get your +W on https://review.opendev.org/c/openstack/neutron-lib/+/991963/ also? | 12:11 |
| stephenfin | and yes, it is a huge change 😅 Mostly mechanical though, thankfully | 12:11 |
| lajoskatona | stephenfin: I just wanted the PTL to be there also :-) | 12:22 |
| lajoskatona | just as keeper of the final rights to steer the project :-) | 12:22 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Add security-groups-default-statefulness API extension https://review.opendev.org/c/openstack/neutron/+/984356 | 12:26 |
| ralonsoh | haleyb, lajoskatona slaweq thanks for your previous +2+W ^^^ but we removed ADMIN_OR_PROJECT_MANAGER from Neutron and everything started to fail. I've just updated this... so close to be merged and so far... | 12:26 |
| zigo | Is there a problem with the release notes job? Should I do a recheck? | 12:29 |
| opendevreview | alisafari proposed openstack/ovn-octavia-provider stable/2026.1: Fix LB association to all connected routers and their networks https://review.opendev.org/c/openstack/ovn-octavia-provider/+/993749 | 12:30 |
| ralonsoh | zigo, because of this https://zuul.opendev.org/t/openstack/build/97367fc02036437b82278f1eeda5894d/logs ? | 12:35 |
| ralonsoh | try rechecking | 12:36 |
| zigo | Yeah. | 12:37 |
| zigo | Ok. | 12:37 |
| zigo | I saw the same error in another job (in Ironic). | 12:38 |
| zigo | https://zuul.opendev.org/t/openstack/build/72da013a7fa54d1aa6b7d201650ea6a9 | 12:52 |
| frickler | yes, the job definition was broken yesterday, but the error got reverted. hopefully | 13:02 |
| frickler | https://review.opendev.org/c/openstack/openstack-zuul-jobs/+/993700 | 13:03 |
| zigo | frickler: Thanks. I feared spamming the CI with useless rechecks ! :) | 13:05 |
| opendevreview | Brian Haley proposed openstack/neutron master: Fix unnecessary-pass pylint warnings https://review.opendev.org/c/openstack/neutron/+/993489 | 13:12 |
| haleyb | ralonsoh: ack, can take a look | 13:14 |
| haleyb | LarsErikP: thanks for testing, i'll take a look the the lp too | 13:14 |
| opendevreview | Bharath M V proposed openstack/neutron-tempest-plugin master: [Tap-as-a-Service] Adding 2 Tap as a service test cases. https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/989931 | 13:22 |
| haleyb | lajoskatona: at this point i think we are a collective of PTLs that are steering the project in the right direction (not off a cliff :) | 13:28 |
| haleyb | and the change lgtm | 13:28 |
| lajoskatona | haleyb: :-) | 13:34 |
| cardoe | stephenfin: will you add py.typed to neutron-lib in your series? | 13:40 |
| stephenfin | cardoe: I will, but I'd like to get a slightly more added first so we don't end up continually breaking other consumers of the types with each new release (like what's been happening with SDK lately) | 13:47 |
| cardoe | okay. | 13:48 |
| cardoe | I'm ultimately looking to reconcile the ML2 interfaces in neutron-lib against how neutron implements them and then how the tests implement them | 13:48 |
| haleyb | stephenfin: so will the neutron-lib changes require consumers to change anything? | 13:56 |
| stephenfin | Only if they're using type hints themselves and start seeing issues with their type checkers of choice. In many cases, the issues with highlight actual bugs (e.g. "this attribute is potentially None but you're assuming otherwise") though other cases might be necessary to massage type checkers | 13:59 |
| stephenfin | The change won't break anyone at runtime in any case. Purely at lint time | 13:59 |
| stephenfin | *changes | 13:59 |
| haleyb | and if i'm remembering correctly there are a few definitions in the neutron tree, or maybe it's just some tests | 13:59 |
| stephenfin | definitions? | 14:00 |
| haleyb | stephenfin: i'm maybe hallucinating with so many reviews on the brain, let me look | 14:01 |
| haleyb | stephenfin: right, so things like neutron/extensions/rbac.py, securitygroup.py and security_groups_default_rules.py which are in the same pattern as the neutron-lib code | 14:03 |
| haleyb | we have a cross-project experimental job in neutron were we can run against neutron-lib master branch, we should just do a test before we bump neutron-lib to see if anything breaks | 14:04 |
| stephenfin | gotcha. so your cross job won't break unless it happens to run mypy | 14:11 |
| stephenfin | and even if it does, I still don't think it will break, since we're annotating the variables but we haven't annotated any functions/methods that consume those variables yet | 14:12 |
| stephenfin | when we do the latter *and* when we add a `py.typed` file *and* when we run mypy against neutron itself, *then* mypy will start complaining that method foo expects one of the new typed dicts but is instead being given a generic dict | 14:13 |
| stephenfin | ...but that's a lot of conditionals :) | 14:14 |
| haleyb | stephenfin: yes it is, and thanks for working on this to help move neutron out of the stone age :) | 14:20 |
| opendevreview | Takashi Kajinami proposed openstack/neutron master: Do not fail when api workers count can't be determined https://review.opendev.org/c/openstack/neutron/+/993767 | 14:40 |
| opendevreview | Merged openstack/neutron-lib master: typing: Add types for RESOURCE_ATTRIBUTE_MAP https://review.opendev.org/c/openstack/neutron-lib/+/991963 | 15:15 |
| opendevreview | Fiorella Yanac proposed x/whitebox-neutron-tempest-plugin master: Add PVLAN live migration connectivity scenario test https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/993779 | 15:40 |
| opendevreview | Eduardo Olivares proposed openstack/neutron master: Install neutron-tempest-plugin in tempest tox venv for multinode jobs https://review.opendev.org/c/openstack/neutron/+/993781 | 15:49 |
| opendevreview | Eduardo Olivares proposed openstack/neutron master: DNM: Add multinode jobs to check pipeline for verification https://review.opendev.org/c/openstack/neutron/+/993782 | 15:49 |
| opendevreview | Eduardo Olivares proposed openstack/neutron master: DNM: Add multinode jobs to check pipeline for verification https://review.opendev.org/c/openstack/neutron/+/993782 | 15:51 |
| opendevreview | Brian Haley proposed openstack/neutron master: [metadata] cleanup response headers https://review.opendev.org/c/openstack/neutron/+/970625 | 15:54 |
| opendevreview | Fiorella Yanac proposed x/whitebox-neutron-tempest-plugin master: Add PVLAN live migration connectivity scenario test https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/993779 | 15:58 |
| opendevreview | Merged openstack/neutron-fwaas-dashboard stable/2026.1: Fix SCSS import for Horizon compatibility https://review.opendev.org/c/openstack/neutron-fwaas-dashboard/+/993390 | 16:15 |
| opendevreview | Merged openstack/neutron-lib master: typing: Add types for SUB_RESOURCE_ATTRIBUTE_MAP https://review.opendev.org/c/openstack/neutron-lib/+/991964 | 16:32 |
| opendevreview | Merged openstack/neutron-lib master: typing: Add types for ACTION_MAP https://review.opendev.org/c/openstack/neutron-lib/+/991965 | 16:37 |
| opendevreview | Miro Tomaska proposed openstack/neutron master: [DNM] Run functional test jobs https://review.opendev.org/c/openstack/neutron/+/993241 | 18:35 |
| opendevreview | Lajos Katona proposed openstack/neutron master: Use SDK for Nova notifications https://review.opendev.org/c/openstack/neutron/+/983905 | 20:08 |
| opendevreview | Ghanshyam Maan proposed openstack/neutron-tempest-plugin master: Use the new variable to enable the RBAC new defaults https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/993839 | 20:26 |
| opendevreview | Miro Tomaska proposed openstack/neutron master: Improve _is_centos9_frr85 skip function https://review.opendev.org/c/openstack/neutron/+/993854 | 22:30 |
| opendevreview | Miro Tomaska proposed openstack/neutron master: [DNM] Run functional test jobs https://review.opendev.org/c/openstack/neutron/+/993241 | 22:31 |
| opendevreview | Miro Tomaska proposed openstack/neutron master: Improve _is_centos9_frr85 skip function https://review.opendev.org/c/openstack/neutron/+/993854 | 22:43 |
| opendevreview | Miro Tomaska proposed openstack/neutron master: FrrDriver Vtysh accepts --vty_socket argument https://review.opendev.org/c/openstack/neutron/+/993398 | 22:57 |
| opendevreview | Miro Tomaska proposed openstack/neutron master: [DNM] Run functional test jobs https://review.opendev.org/c/openstack/neutron/+/993241 | 23:06 |
| opendevreview | Terry Wilson proposed openstack/neutron master: Only set the maintenance worker lock on the worker https://review.opendev.org/c/openstack/neutron/+/993860 | 23:49 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!