| opendevreview | Merged openstack/neutron-fwaas stable/2026.1: Pass router_id to the _cleanup_nflog_process method https://review.opendev.org/c/openstack/neutron-fwaas/+/982257 | 00:58 |
|---|---|---|
| opendevreview | Brian Haley proposed openstack/neutron master: Remove update_router_static_routes() https://review.opendev.org/c/openstack/neutron/+/984256 | 03:27 |
| opendevreview | Brian Haley proposed openstack/neutron master: Update jobs based on testing runtime for 2026.2 https://review.opendev.org/c/openstack/neutron/+/985322 | 06:13 |
| ralonsoh | haleyb, hello! nice to see you now | 06:47 |
| ralonsoh | bcafarel, hello! if you have 1 minute, please check https://review.opendev.org/q/topic:%22bug/2148211%22 | 07:06 |
| ralonsoh | thanks! | 07:06 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-lib master: Add "l3-agent-scheduler-ha-chassis-priority" API extension definition https://review.opendev.org/c/openstack/neutron-lib/+/982781 | 07:08 |
| bcafarel | ah it was on my list to ping to make sure it was identical to master (after my nit on assisted-by), nice it's already updated | 07:49 |
| ralonsoh | yeah, that was changed in the stable patches too | 07:50 |
| bcafarel | and done :) | 07:50 |
| ralonsoh | thanks! | 07:50 |
| ralonsoh | haleyb, lajoskatona slaweq hello! Please review comment https://review.opendev.org/c/openstack/releases/+/984722/comments/c9132a77_a862bd6c | 08:37 |
| opendevreview | Merged openstack/neutron stable/2026.1: Fix AttributeError on DeletedChassis missing other_config https://review.opendev.org/c/openstack/neutron/+/984530 | 09:41 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Pass ``external_ids`` in the ``address_set_add`` command https://review.opendev.org/c/openstack/neutron/+/977923 | 09:47 |
| opendevreview | Merged openstack/ovsdbapp master: Change signature of ``lsp_detach_mirror`` method https://review.opendev.org/c/openstack/ovsdbapp/+/984691 | 09:59 |
| opendevreview | Merged openstack/neutron stable/2025.2: Fix AttributeError on DeletedChassis missing other_config https://review.opendev.org/c/openstack/neutron/+/984531 | 09:59 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: WIP == Add missing Router ID and AZ hits in the HCG.external_ids https://review.opendev.org/c/openstack/neutron/+/985363 | 11:34 |
| opendevreview | Bence Romsics proposed openstack/neutron master: Reject multicast addresses in AAP https://review.opendev.org/c/openstack/neutron/+/983201 | 12:57 |
| haleyb | ralonsoh: i went to sleep right after i sent that patch | 13:32 |
| haleyb | and i can't tell if there is some new pep8 failures this morning :( | 13:33 |
| ralonsoh | yes, there are | 13:58 |
| haleyb | sigh | 14:03 |
| opendevreview | Brian Haley proposed openstack/neutron master: Start using datetime.UTC alias https://review.opendev.org/c/openstack/neutron/+/985428 | 14:32 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Add missing Router ID and AZ hits in the HCG.external_ids https://review.opendev.org/c/openstack/neutron/+/985363 | 14:48 |
| opendevreview | Merged openstack/os-vif master: Bump min version of pyroute2 https://review.opendev.org/c/openstack/os-vif/+/984934 | 15:56 |
| haleyb | elodilles: were you going to push a change to zuul-jobs for arm64? i had dropped a comment in the merged change about it | 16:07 |
| opendevreview | Brian Haley proposed openstack/neutron master: Upgrade to tox 4.28.0 https://review.opendev.org/c/openstack/neutron/+/985512 | 16:50 |
| opendevreview | Elvira García Ruiz proposed openstack/neutron master: Add PVLAN service plugin https://review.opendev.org/c/openstack/neutron/+/978798 | 17:57 |
| opendevreview | Elvira García Ruiz proposed openstack/neutron master: [WIP] PVLAN OVN Driver https://review.opendev.org/c/openstack/neutron/+/985150 | 17:57 |
| isaacvicente[m] | Hi guys, is the CI fine? I’m afraid I’m overusing the rechecks. A few days ago there was an issue with the post_failure status, but it seems to be okay now. Is there anything else I should know? | 18:21 |
| opendevreview | Merged openstack/neutron stable/2025.1: Fix AttributeError on DeletedChassis missing other_config https://review.opendev.org/c/openstack/neutron/+/984532 | 18:31 |
| opendevreview | Merged openstack/neutron stable/2024.2: Fix AttributeError on DeletedChassis missing other_config https://review.opendev.org/c/openstack/neutron/+/984533 | 18:31 |
| opendevreview | Merged openstack/neutron unmaintained/2024.1: Fix AttributeError on DeletedChassis missing other_config https://review.opendev.org/c/openstack/neutron/+/984534 | 18:31 |
| haleyb | isaacvicente[m]: it seems better. before rechecking you should make sure the test failure is not related to the patch. And you should never just do "recheck" but give a reason, for example "recheck bug 12345" | 18:32 |
| haleyb | you can always look at another recent patch for reference if CI is healthy | 18:32 |
| haleyb | . | 18:33 |
| isaacvicente[m] | thanks for the explanation haleyb, adding the reason for the recheck is very helpful, for me and the maintainers | 18:49 |
| haleyb | any reason is fine, it just reinforces you've looked at the failure | 19:04 |
| opendevreview | Brian Haley proposed openstack/neutron master: Update jobs based on testing runtime for 2026.2 https://review.opendev.org/c/openstack/neutron/+/985322 | 19:31 |
| elodilles | haleyb: i somehow remembered (maybe false memory? :S) that we are rather removing arm64 jobs than adding new ones. so i did not plan to update the arm64 job templates. but if you know that it is needed then i can prepare a patch for that | 19:35 |
| haleyb | elodilles: i did push an update just a minute ago. it technically doesn't add a new job, just changes the lowest to py311 | 19:36 |
| haleyb | i have no idea if we are removing them | 19:36 |
| elodilles | haleyb: and the other thing is what kind of nodes are those jobs running on, do they have the right python versions? | 19:45 |
| elodilles | nah, i was probably overthinking it, i see that there are py313 + py314 arm64 jobs as well, so probably your patch will do | 19:48 |
| elodilles | thanks for pushing it haleyb o/ | 19:48 |
| haleyb | elodilles: https://review.opendev.org/c/openstack/neutron/+/985322 is the neutron patch, i might have to fold-in a fix for the pep8 failures | 19:59 |
| elodilles | haleyb: ah, so py311-arm64 job is indeed passing. cool \o/ | 20:21 |
| opendevreview | Brian Haley proposed openstack/neutron master: Update jobs based on testing runtime for 2026.2 https://review.opendev.org/c/openstack/neutron/+/985322 | 21:50 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!