| opendevreview | Merged openstack/neutron stable/ussuri: Randomize segmentation ID assignation https://review.opendev.org/c/openstack/neutron/+/804999 | 00:17 |
|---|---|---|
| opendevreview | Merged openstack/neutron stable/victoria: Fix neutron_pg_drop-related startup issues https://review.opendev.org/c/openstack/neutron/+/808864 | 00:17 |
| opendevreview | Merged openstack/neutron stable/victoria: Skip DVR binding for ports with invalid OFPORT https://review.opendev.org/c/openstack/neutron/+/807255 | 01:39 |
| opendevreview | Merged openstack/neutron stable/train: Randomize segmentation ID assignation https://review.opendev.org/c/openstack/neutron/+/808160 | 02:24 |
| *** rpittau|afk is now known as rpittau | 06:41 | |
| opendevreview | Kamil Sambor proposed openstack/networking-ovn stable/train: Fix race condition retrieving logical router rows https://review.opendev.org/c/openstack/networking-ovn/+/810820 | 07:02 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-specs master: Add spec for Network cascade deletion https://review.opendev.org/c/openstack/neutron-specs/+/810822 | 07:10 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-specs master: Add spec for Network cascade deletion https://review.opendev.org/c/openstack/neutron-specs/+/810822 | 07:44 |
| opendevreview | Lajos Katona proposed openstack/neutron-tempest-plugin master: Make neutron-tempest-plugin-vpnaas job OVS based https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/809770 | 07:52 |
| lucasagomes | ralonsoh, thanks much for the review on the segmentation ID update patch for OVN | 08:17 |
| ralonsoh | lucasagomes, yw | 08:17 |
| lucasagomes | ralonsoh, just to confirm, in my patch I need to define the provider_network_attribute_updates_supported() returning the SEGMENTATION_ID field | 08:17 |
| lucasagomes | along with my changes right ? | 08:18 |
| ralonsoh | lucasagomes, I think this is the missing part | 08:18 |
| ralonsoh | if you have an env, it is easy to check | 08:18 |
| lucasagomes | I was reading the other patch but I think it won't apply for that patch specifically | 08:18 |
| lucasagomes | ralonsoh++ awesome thank u! | 08:18 |
| lucasagomes | I will add it | 08:18 |
| ralonsoh | creating a network and binding a port | 08:18 |
| lucasagomes | will do | 08:35 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Write FW OF rules belonging to a port in a single operation https://review.opendev.org/c/openstack/neutron/+/806246 | 09:08 |
| opendevreview | Lucas Alvares Gomes proposed openstack/neutron master: [OVN] Fix updating network segmentation ID https://review.opendev.org/c/openstack/neutron/+/810704 | 09:09 |
| ralonsoh | lucasagomes, ^^ checking (and testing) now | 09:09 |
| lucasagomes | ralonsoh++ thanks | 09:09 |
| opendevreview | Merged openstack/neutron master: [OVN][FT] Check UNIX socket file before using it https://review.opendev.org/c/openstack/neutron/+/807862 | 09:31 |
| opendevreview | Merged openstack/neutron master: Add retry when executing OF commands if "InvalidDatapath" https://review.opendev.org/c/openstack/neutron/+/810592 | 09:31 |
| opendevreview | Merged openstack/neutron stable/wallaby: [Functional] Wait for the initial state of ha router before test https://review.opendev.org/c/openstack/neutron/+/808497 | 09:44 |
| opendevreview | Merged openstack/neutron master: ovn: use stateless NAT rules for FIPs https://review.opendev.org/c/openstack/neutron/+/804807 | 09:44 |
| opendevreview | Merged openstack/neutron stable/victoria: [Functional] Wait for the initial state of ha router before test https://review.opendev.org/c/openstack/neutron/+/808498 | 09:49 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: [DVR] Set arp entries only for IPs from the correct subnet https://review.opendev.org/c/openstack/neutron/+/802037 | 09:51 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron stable/queens: [L3] Use processing queue for network update events https://review.opendev.org/c/openstack/neutron/+/803584 | 10:07 |
| slaweq | ralonsoh: hi, if You'll have few minutes, please check https://review.opendev.org/q/I4bcba0ae82d99fac962d758b48b1727f344ec7bb and https://review.opendev.org/q/I2efe66a7415f7a18fb85bd2536a1901e751d6203 | 10:08 |
| slaweq | thx in advance | 10:08 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron stable/wallaby: Delete SG log entries when SG is deleted https://review.opendev.org/c/openstack/neutron/+/810870 | 10:10 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron stable/victoria: Delete SG log entries when SG is deleted https://review.opendev.org/c/openstack/neutron/+/810871 | 10:11 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron stable/ussuri: Delete SG log entries when SG is deleted https://review.opendev.org/c/openstack/neutron/+/810851 | 10:14 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron stable/train: Delete SG log entries when SG is deleted https://review.opendev.org/c/openstack/neutron/+/810872 | 10:15 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-tempest-plugin master: Add logging API delete corresponding SG test https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/804238 | 10:16 |
| slaweq | ralonsoh: and also https://review.opendev.org/q/b8ef8e722af761dd394064ab70e159aa05639e56 :) | 10:20 |
| slaweq | thx | 10:20 |
| slaweq | ralonsoh: and https://review.opendev.org/q/Iba29ab0b1141e731cc811e8bee076dd5726248b5 :) | 10:21 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron stable/train: VLAN "allocate_partially_specified_segment" can return any physnet https://review.opendev.org/c/openstack/neutron/+/808192 | 10:22 |
| slaweq | ralonsoh: and https://review.opendev.org/q/I4d344f5ca6673b70b69a03503ec0f267bf0cadf7 | 10:22 |
| slaweq | sorry for bothering You but I want to clean a bit list of my opened patches :) | 10:23 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron stable/xena: Rollback db session in case of error during releasing quota reservation https://review.opendev.org/c/openstack/neutron/+/810873 | 10:23 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron stable/xena: Remove dhcp_extra_opt name after first newline character https://review.opendev.org/c/openstack/neutron/+/810874 | 10:31 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron stable/wallaby: Remove dhcp_extra_opt name after first newline character https://review.opendev.org/c/openstack/neutron/+/810875 | 10:31 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron stable/victoria: Remove dhcp_extra_opt name after first newline character https://review.opendev.org/c/openstack/neutron/+/810876 | 10:32 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron stable/ussuri: Remove dhcp_extra_opt name after first newline character https://review.opendev.org/c/openstack/neutron/+/810877 | 10:32 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron stable/train: Remove dhcp_extra_opt name after first newline character https://review.opendev.org/c/openstack/neutron/+/810878 | 10:32 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron stable/stein: Remove dhcp_extra_opt name after first newline character https://review.opendev.org/c/openstack/neutron/+/810879 | 10:32 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron stable/rocky: Remove dhcp_extra_opt name after first newline character https://review.opendev.org/c/openstack/neutron/+/810853 | 10:34 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron stable/queens: Remove dhcp_extra_opt name after first newline character https://review.opendev.org/c/openstack/neutron/+/810880 | 10:35 |
| ralonsoh | slaweq, sure, let me check | 10:35 |
| slaweq | ralonsoh: thx a lot | 10:35 |
| slaweq | and sorry for bothering You with so many patches :) | 10:36 |
| ralonsoh | hehehe np! | 10:39 |
| bcafarel | slaweq: should we get a "drop l-c" in neutron stable/xena? backports fail on that job - like https://review.opendev.org/c/openstack/neutron/+/810873 | 11:12 |
| ralonsoh | how is possible it is failing so fast? it was released some days ago | 11:14 |
| ralonsoh | btw, we have this patch for n-lib https://review.opendev.org/c/openstack/neutron-lib/+/808599 | 11:15 |
| ralonsoh | so could make sense that for neutron | 11:15 |
| bcafarel | ralonsoh: apparently latest setuptools triggered it "[stable][requirements][zuul] unpinned setuptools dependency on stable" | 11:16 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/xena: Dropping lower constraints testing (stable Xena) https://review.opendev.org/c/openstack/neutron/+/810858 | 11:17 |
| ralonsoh | bcafarel, ^^ | 11:17 |
| slaweq | bcafarel: ralonsoh I think we should drop it from stable branch | 11:17 |
| slaweq | we agreed on that some time ago | 11:17 |
| ralonsoh | slaweq, https://review.opendev.org/c/openstack/neutron/+/810858 | 11:17 |
| slaweq | to run that job only on master branch | 11:17 |
| bcafarel | sounds good, let's get these in then :) | 11:18 |
| ralonsoh | ok sorry, I need to drop more jobs | 11:18 |
| ralonsoh | one sec | 11:18 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/xena: Dropping lower constraints testing (stable Xena) https://review.opendev.org/c/openstack/neutron/+/810858 | 11:19 |
| opendevreview | Merged openstack/neutron stable/wallaby: Populate self.floating_ips_dict using "ip rule" information https://review.opendev.org/c/openstack/neutron/+/810393 | 11:54 |
| opendevreview | Merged openstack/neutron stable/victoria: Revert "[L3][HA] Retry when setting HA router GW status." https://review.opendev.org/c/openstack/neutron/+/809382 | 12:37 |
| ralonsoh | bcafarel, ^^ | 12:41 |
| ralonsoh | what is that? | 12:41 |
| ralonsoh | I didn't receive a message for those patches | 12:41 |
| ralonsoh | slaweq, ^^ | 12:42 |
| opendevreview | Merged openstack/neutron stable/queens: Add extra logs to the network update callback in L3 agent https://review.opendev.org/c/openstack/neutron/+/803583 | 12:43 |
| slaweq | ralonsoh: IIUC it caused problem for some users (see bug report) | 12:44 |
| slaweq | I don't remember it now in details tbh | 12:44 |
| ralonsoh | I'll propose a revert on the revert | 12:44 |
| ralonsoh | the problem is not in the neutron code | 12:44 |
| slaweq | yes, but I think that we still have the issue with privsep which Your patch triggered | 12:45 |
| slaweq | IIUC | 12:45 |
| opendevreview | Lajos Katona proposed openstack/tap-as-a-service master: CLI: remove leftover logs from tap-flow create https://review.opendev.org/c/openstack/tap-as-a-service/+/810895 | 12:54 |
| opendevreview | Merged openstack/neutron stable/stein: Add extra logs to the network update callback in L3 agent https://review.opendev.org/c/openstack/neutron/+/803579 | 13:02 |
| opendevreview | Merged openstack/neutron-dynamic-routing master: Fix Mako in lower-constraints https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/810751 | 13:11 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/xena: [Docs] Ovn backend now supports FIP QoS bandwdith limiting https://review.opendev.org/c/openstack/neutron/+/810882 | 13:13 |
| ralonsoh | slaweq, https://review.opendev.org/c/openstack/neutron/+/810858 if you don't mind | 13:14 |
| bcafarel | :/ | 13:17 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/wallaby: [Docs] Ovn backend now supports FIP QoS bandwdith limiting https://review.opendev.org/c/openstack/neutron/+/810902 | 13:18 |
| ralonsoh | slaweq, I can remove lower-constraints.txt file in this patch | 13:18 |
| bcafarel | ralonsoh: this was on reading original revert https://review.opendev.org/c/openstack/neutron/+/805366 776427 was more a workaround and reported issues with l3 agents getting stuck | 13:18 |
| ralonsoh | bcafarel, what they need to find in their environment is why ns_exists get stuck | 13:19 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron stable/xena: Lower-constraints removal follow up https://review.opendev.org/c/openstack/neutron/+/810903 | 13:21 |
| slaweq | ralonsoh: I just did follow up ^^ | 13:21 |
| slaweq | as your patch already had couple of +2 and I didn't wan't to block our gate | 13:21 |
| bcafarel | that reminds me, I think there was similar comment in other xena drop-lc patches, need to check them | 13:21 |
| hyang[m] | slaweq: sorry to bother you. When you get time, can you please check my latest comment for the shared SG bug https://bugs.launchpad.net/neutron/+bug/1942615/comments/7 I'd like to align the proposal with you before implementing it, thank you. | 14:44 |
| bcafarel | lajoskatona: want me to go ahead and add disable f-string in https://review.opendev.org/c/openstack/neutron-lib/+/808599 ? | 14:47 |
| bcafarel | (xena neutron-lib) | 14:47 |
| lajoskatona | bcafarel: if you have time for, it just got there again in my loop :-) | 14:47 |
| bcafarel | lajoskatona: on it :) | 14:48 |
| lajoskatona | bcafarel: thanks | 14:50 |
| slaweq | hyang[m]: sure, looking | 14:54 |
| opendevreview | Bernard Cafarelli proposed openstack/neutron-lib stable/xena: Dropping lower constraints testing (stable Xena) https://review.opendev.org/c/openstack/neutron-lib/+/808599 | 14:55 |
| hyang[m] | slaweq: tyty | 14:55 |
| slaweq | hyang[m]: I replied there | 14:59 |
| hyang[m] | slaweq: thanks a lot! So yup I will go forward with the second query approach with the 'shared' filter. | 15:01 |
| *** yonglihe_ is now known as yonglihe | 15:08 | |
| opendevreview | Merged openstack/neutron stable/train: [L3] Use processing queue for network update events https://review.opendev.org/c/openstack/neutron/+/803578 | 15:44 |
| opendevreview | Merged openstack/neutron stable/stein: [L3] Use processing queue for network update events https://review.opendev.org/c/openstack/neutron/+/803580 | 16:04 |
| opendevreview | Merged openstack/neutron stable/victoria: Make test_throttler happy https://review.opendev.org/c/openstack/neutron/+/808157 | 16:04 |
| opendevreview | Merged openstack/neutron stable/ussuri: Make test_throttler happy https://review.opendev.org/c/openstack/neutron/+/808158 | 16:05 |
| opendevreview | Merged openstack/neutron stable/train: Make test_throttler happy https://review.opendev.org/c/openstack/neutron/+/808159 | 16:05 |
| opendevreview | Merged openstack/neutron stable/victoria: VLAN "allocate_partially_specified_segment" can return any physnet https://review.opendev.org/c/openstack/neutron/+/808189 | 16:05 |
| opendevreview | Merged openstack/neutron stable/ussuri: VLAN "allocate_partially_specified_segment" can return any physnet https://review.opendev.org/c/openstack/neutron/+/808191 | 16:05 |
| opendevreview | Merged openstack/neutron stable/xena: Dropping lower constraints testing (stable Xena) https://review.opendev.org/c/openstack/neutron/+/810858 | 16:06 |
| opendevreview | Merged openstack/neutron stable/xena: Lower-constraints removal follow up https://review.opendev.org/c/openstack/neutron/+/810903 | 16:06 |
| opendevreview | Merged openstack/neutron stable/xena: [Docs] Ovn backend now supports FIP QoS bandwdith limiting https://review.opendev.org/c/openstack/neutron/+/810882 | 16:49 |
| *** tbachman is now known as Guest882 | 17:02 | |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN][Placement] Add a SB Chassis event to track changes in BW config https://review.opendev.org/c/openstack/neutron/+/801712 | 17:05 |
| *** tbachman is now known as Guest883 | 17:17 | |
| opendevreview | Merged openstack/neutron stable/xena: Rollback db session in case of error during releasing quota reservation https://review.opendev.org/c/openstack/neutron/+/810873 | 19:25 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!