| opendevreview | Hang Yang proposed openstack/neutron master: Unify the states format for address group payloads https://review.opendev.org/c/openstack/neutron/+/815575 | 00:25 |
|---|---|---|
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: Don't setup bridge controller if it is already set https://review.opendev.org/c/openstack/neutron/+/815255 | 07:13 |
| bcafarel | slaweq: morning https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/815518 is good for reviews (train n-t-p jobs passing with it) | 07:27 |
| bcafarel | https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/815458 will be soon too for stein - filtering out a few unstable tests but I am still running rechecks to see if I need to add more | 07:27 |
| opendevreview | Slawek Kaplonski proposed openstack/os-ken master: Bugfix now multiple switches can connect with TLS https://review.opendev.org/c/openstack/os-ken/+/813833 | 07:27 |
| opendevreview | Slawek Kaplonski proposed openstack/os-ken master: Allow user to set cipher list. https://review.opendev.org/c/openstack/os-ken/+/813835 | 07:30 |
| opendevreview | Slawek Kaplonski proposed openstack/os-ken master: Add support for the MTU ND option. https://review.opendev.org/c/openstack/os-ken/+/813836 | 07:30 |
| opendevreview | Tobias Urdin proposed openstack/neutron master: Set RPC timeout in PluginReportStateAPI to report_interval https://review.opendev.org/c/openstack/neutron/+/815310 | 07:30 |
| slaweq | ralonsoh: hi | 07:36 |
| ralonsoh | slaweq, hi | 07:36 |
| slaweq | ralonsoh: can You check os-ken patches https://review.opendev.org/c/openstack/os-ken/+/813836 and below? | 07:36 |
| ralonsoh | I'm on it | 07:36 |
| slaweq | I just removed from that series https://review.opendev.org/c/openstack/os-ken/+/813834 | 07:36 |
| slaweq | but I also have another question regarding https://review.opendev.org/c/openstack/os-ken/+/813834 | 07:36 |
| slaweq | You mentioned that this issue in eventlet is fixed by https://github.com/eventlet/eventlet/commit/1a41774f26f79b9e9ba03239c80796d35f9a0630 | 07:36 |
| ralonsoh | yes | 07:36 |
| slaweq | but I don't see that patch in any eventlet version | 07:36 |
| slaweq | am I missing something? | 07:37 |
| ralonsoh | let me check | 07:37 |
| slaweq | ok, I think that this is fix https://github.com/eventlet/eventlet/pull/531 | 07:37 |
| slaweq | or not | 07:38 |
| slaweq | because there is many more pull requests merged related to that issue | 07:38 |
| ralonsoh | it is in 0.26.0 | 07:39 |
| ralonsoh | https://paste.opendev.org/show/810232/ | 07:39 |
| slaweq | ralonsoh: ok, but according to last comment https://github.com/eventlet/eventlet/issues/526#issuecomment-670198818 it seems that bump lower-constraint to 0.26.1 will be ok | 07:39 |
| slaweq | ralonsoh++ ok, thx | 07:40 |
| opendevreview | Slawek Kaplonski proposed openstack/os-ken master: Bump min eventlet version to 0.26.1 https://review.opendev.org/c/openstack/os-ken/+/815601 | 07:42 |
| slaweq | ralonsoh: ok, done ^^ | 07:43 |
| ralonsoh | +2 | 07:43 |
| ralonsoh | slaweq, https://review.opendev.org/c/openstack/os-ken/+/813217 | 07:43 |
| ralonsoh | and https://review.opendev.org/c/openstack/os-ken/+/813214 | 07:44 |
| opendevreview | Slawek Kaplonski proposed openstack/os-ken master: Add support for revised RFC8227 withdraw labels https://review.opendev.org/c/openstack/os-ken/+/813215 | 07:56 |
| opendevreview | Slawek Kaplonski proposed openstack/os-ken master: Update bridge.py https://review.opendev.org/c/openstack/os-ken/+/813216 | 07:56 |
| opendevreview | Slawek Kaplonski proposed openstack/os-ken master: updated jsonrpc.Session call to have correct arguments for latest version of ovs https://review.opendev.org/c/openstack/os-ken/+/813217 | 07:57 |
| slaweq | ralonsoh: all +W'ed and rebased to be up to date in that series :) | 07:57 |
| ralonsoh | perfect | 07:57 |
| opendevreview | Przemyslaw Szczerbik proposed openstack/neutron master: Networking guide: Add Guaranteed Minimum Packet Rate https://review.opendev.org/c/openstack/neutron/+/814361 | 08:28 |
| opendevreview | Merged openstack/neutron master: Improve "get_collection_count" calls https://review.opendev.org/c/openstack/neutron/+/807687 | 08:52 |
| opendevreview | Przemyslaw Szczerbik proposed openstack/neutron master: Networking guide: Add trunk limitation to min bandwidth https://review.opendev.org/c/openstack/neutron/+/815609 | 09:05 |
| opendevreview | Merged openstack/os-ken master: nx_action_encap and nx_action_decap classes are defined https://review.opendev.org/c/openstack/os-ken/+/813214 | 09:08 |
| opendevreview | Slawek Kaplonski proposed openstack/os-ken master: Bump min eventlet version to 0.26.1 https://review.opendev.org/c/openstack/os-ken/+/815601 | 09:22 |
| opendevreview | Slawek Kaplonski proposed openstack/os-ken master: Choose the highest TLS version https://review.opendev.org/c/openstack/os-ken/+/813830 | 09:26 |
| 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 | 09:36 |
| ralonsoh | slaweq, https://review.opendev.org/c/openstack/neutron-specs/+/798704 | 09:59 |
| ralonsoh | I +2 the spec. I don't know if more reviews are needed | 09:59 |
| slaweq | ralonsoh: thx | 10:02 |
| slaweq | I don't know too really | 10:02 |
| slaweq | maybe mlavalle, njohnston or amotoki can take a look also :) | 10:02 |
| opendevreview | Merged openstack/os-ken master: Add support for revised RFC8227 withdraw labels https://review.opendev.org/c/openstack/os-ken/+/813215 | 10:27 |
| opendevreview | Merged openstack/os-ken master: Update bridge.py https://review.opendev.org/c/openstack/os-ken/+/813216 | 10:27 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron stable/rocky: Populate self.floating_ips_dict using "ip rule" information https://review.opendev.org/c/openstack/neutron/+/810427 | 10:36 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron stable/queens: Populate self.floating_ips_dict using "ip rule" information https://review.opendev.org/c/openstack/neutron/+/810397 | 10:36 |
| opendevreview | Merged openstack/os-ken master: updated jsonrpc.Session call to have correct arguments for latest version of ovs https://review.opendev.org/c/openstack/os-ken/+/813217 | 10:36 |
| opendevreview | Lucas Alvares Gomes proposed openstack/neutron stable/ussuri: Fix OVN migration workload creation order https://review.opendev.org/c/openstack/neutron/+/815618 | 10:46 |
| opendevreview | Lucas Alvares Gomes proposed openstack/neutron stable/victoria: Fix OVN migration workload creation order https://review.opendev.org/c/openstack/neutron/+/815619 | 10:46 |
| opendevreview | Lucas Alvares Gomes proposed openstack/neutron stable/wallaby: Fix OVN migration workload creation order https://review.opendev.org/c/openstack/neutron/+/815620 | 10:47 |
| opendevreview | Lucas Alvares Gomes proposed openstack/neutron stable/xena: Fix OVN migration workload creation order https://review.opendev.org/c/openstack/neutron/+/815621 | 10:47 |
| opendevreview | Lucas Alvares Gomes proposed openstack/networking-ovn stable/train: Fix OVN migration workload creation order https://review.opendev.org/c/openstack/networking-ovn/+/815622 | 10:51 |
| tobias-urdin | ralonsoh: slaweq: testing passing now and rdy for review when you have time https://review.opendev.org/c/openstack/neutron/+/815310 | 11:09 |
| slaweq | tobias-urdin: I commented on it | 11:26 |
| opendevreview | Tobias Urdin proposed openstack/neutron master: Set RPC timeout in PluginReportStateAPI to report_interval https://review.opendev.org/c/openstack/neutron/+/815310 | 11:28 |
| tobias-urdin | slaweq: fair point, just wanted to keep a fallback but really doesn't matter since if report_interval is always defined by a default | 11:29 |
| tobias-urdin | it would have raised NoConfigOpt from oslo.config anyway i think if it wasn't there | 11:29 |
| slaweq | tobias-urdin: thx, +2'ed | 11:30 |
| tobias-urdin | thanks :) | 11:30 |
| slaweq | tobias-urdin: thx for the patch :) | 11:46 |
| slaweq | ralonsoh: can You check once again https://review.opendev.org/c/openstack/neutron/+/815255 ? I just rebased it on top of Your "sleep patch":) | 11:47 |
| ralonsoh | tobias-urdin, sure, let me check | 12:26 |
| ralonsoh | slaweq, right now | 12:26 |
| amotoki | slaweq: I will look into your spec QinQ support soon | 12:26 |
| slaweq | thx ralonsoh and amotoki :) | 12:30 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Check quota limits https://review.opendev.org/c/openstack/neutron/+/801470 | 12:33 |
| mlavalle | slaweq, ralonsoh: I will review that spec later today | 12:44 |
| slaweq | mlavalle: thx a lot | 12:44 |
| opendevreview | Oleg Bondarev proposed openstack/neutron master: Add Local IP L2 extension https://review.opendev.org/c/openstack/neutron/+/807116 | 12:48 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-tempest-plugin master: Use admin client to list ports in the test_two_vms_fips test https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/815658 | 13:00 |
| em_ | we are running a new non DVR OVN based installation. We added a provider and a self-service lan. The communication in the service-lan works without any issues. Now we do not understand how to add a floating api from the provider network to an instance | 13:11 |
| em_ | adding an interface to the actual provider lan does not work (i guess that would be the DVR mode) .. but how to do this with routing | 13:11 |
| em_ | ok, we had not router. Got it | 13:25 |
| opendevreview | Anton Vazhnetsov proposed openstack/ovsdbapp master: nb: provide lrp_get method https://review.opendev.org/c/openstack/ovsdbapp/+/815671 | 14:08 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: Remove router_info from agent's cache when processing failed https://review.opendev.org/c/openstack/neutron/+/814892 | 14:16 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: Cleanup router for which processing added router failed https://review.opendev.org/c/openstack/neutron/+/814892 | 14:18 |
| slaweq | ralonsoh: mlavalle obondarev hi, please check https://review.opendev.org/c/openstack/neutron/+/797120 and https://review.opendev.org/c/openstack/neutron/+/797121 when You will have few minutes | 14:20 |
| slaweq | thx in advance | 14:20 |
| ralonsoh | sure | 14:21 |
| mlavalle | slaweq: will do.Over the past few days I spent some time looking at neutron/plugins/ml2/drivers/openvswitch/agent/common/constants.py | 14:23 |
| mlavalle | Commit message | 14:23 |
| mlavalle | neutron/plugins/ml2/drivers/openvswitch/agent/common/constants.py, so this request is timely :-) | 14:23 |
| ralonsoh | slaweq, why this change? https://review.opendev.org/c/openstack/neutron/+/797120/11/neutron/plugins/ml2/drivers/openvswitch/agent/openflow/native/br_int.py | 14:26 |
| slaweq | ralonsoh: after liuyulong's comments, I checked it and indeed those tables like constants.INT_BR_ALL_TABLES and other for other bridges are used only in that one place for cleaning purpose | 14:32 |
| slaweq | so I agreed with his proposal to keep those tables in neutron | 14:32 |
| slaweq | but as it's used only in one place each, I don't think we need to keep them in the separate constants file | 14:33 |
| ralonsoh | ok, makes sense then | 14:33 |
| slaweq | so I defined them directly in the files where they are used | 14:33 |
| slaweq | mlavalle: ralonsoh bcafarel please also review https://review.opendev.org/q/I200acfcaaae7f056ea9a563fead9ff2de8464971 | 14:34 |
| slaweq | https://review.opendev.org/q/bf6831e0d2ad33ff751f06d568abe6cedff4d717 | 14:34 |
| slaweq | https://review.opendev.org/q/I74cd5820389c86819c6884d3d61c9b2f7907cc88 | 14:34 |
| slaweq | and also patch https://review.opendev.org/c/openstack/neutron/+/799439 | 14:34 |
| slaweq | :) | 14:34 |
| slaweq | thx in advance | 14:34 |
| * slaweq needs to make list of opened patches a bit shorter finally :) | 14:34 | |
| bcafarel | or make a dynamic url to point to the "need to review" patches :) | 14:38 |
| obondarev | slaweq: hi, I raised a question in https://review.opendev.org/c/openstack/neutron/+/797120, please check | 14:42 |
| obondarev | ralonsoh: ^^ | 14:42 |
| ralonsoh | let me check | 14:43 |
| obondarev | I think *_ALL_TABLES should still be defined near actual constants | 14:43 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: Remove functions to enable Neutron's placement integration https://review.opendev.org/c/openstack/neutron/+/815684 | 14:46 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: Remove functions which enable Neutron's QoS service https://review.opendev.org/c/openstack/neutron/+/815687 | 14:52 |
| slaweq | obondarev: thx I will check it | 14:53 |
| slaweq | obondarev: ok, but what do You think would be better approach? I see 2 opposite opinions there: yours and liuyulong's | 14:54 |
| slaweq | initially I proposed to move it to neutron-lib with everything else, but liu insisted to keep it here | 14:55 |
| 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 | 14:55 |
| obondarev | slaweq: maybe we need to ping him to check my comment :) | 14:56 |
| slaweq | obondarev: k, I will do that :) | 14:56 |
| obondarev | slaweq: ideally these tables should've be defined as a dict, but I guess at that time there were no such need | 14:57 |
| slaweq | obondarev: that can be follow-up patch if needed I guess :) | 14:59 |
| obondarev | slaweq: yeah, bug patch I guess :) | 15:01 |
| opendevreview | Merged openstack/neutron-lib master: Addition of status and name field to bgp_associations API https://review.opendev.org/c/openstack/neutron-lib/+/813557 | 15:10 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Replace "tenant_id" with "project_id" in metering service https://review.opendev.org/c/openstack/neutron/+/814807 | 15:44 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Remove SubnetDNSPublishFixedIP from Subnet query https://review.opendev.org/c/openstack/neutron/+/811999 | 15:45 |
| opendevreview | Anton Vazhnetsov proposed openstack/ovsdbapp master: nb: add support for lrp's gateway chassis API https://review.opendev.org/c/openstack/ovsdbapp/+/815723 | 18:16 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!