| opendevreview | wushiming proposed openstack/neutron master: Drop install_venv https://review.opendev.org/c/openstack/neutron/+/810333 | 02:26 |
|---|---|---|
| opendevreview | wushiming proposed openstack/neutron master: Remove references to sys.version_info https://review.opendev.org/c/openstack/neutron/+/809910 | 02:34 |
| opendevreview | wushiming proposed openstack/neutron master: Drop install_venv https://review.opendev.org/c/openstack/neutron/+/809910 | 02:35 |
| opendevreview | Merged openstack/neutron master: Rollback db session in case of error during releasing quota reservation https://review.opendev.org/c/openstack/neutron/+/809191 | 03:32 |
| opendevreview | Przemyslaw Szczerbik proposed openstack/neutron master: Add API extenstion for QoS minimum pps rule https://review.opendev.org/c/openstack/neutron/+/803045 | 06:56 |
| opendevreview | Przemyslaw Szczerbik proposed openstack/neutron master: ovs-agent: Report pkt processing info in heartbeat https://review.opendev.org/c/openstack/neutron/+/800444 | 06:56 |
| opendevreview | Przemyslaw Szczerbik proposed openstack/neutron master: Report CUSTOM_VNIC_TYPE_ traits on Neutron agent RP https://review.opendev.org/c/openstack/neutron/+/800445 | 06:56 |
| opendevreview | Przemyslaw Szczerbik proposed openstack/neutron master: Report pkt processing capacity on Neutron agent RP https://review.opendev.org/c/openstack/neutron/+/800446 | 06:57 |
| opendevreview | Przemyslaw Szczerbik proposed openstack/neutron master: Enable QoS minimum packet rate rule for OVS backend https://review.opendev.org/c/openstack/neutron/+/805391 | 06:57 |
| opendevreview | Przemyslaw Szczerbik proposed openstack/neutron master: Add port-resource-request-groups extension https://review.opendev.org/c/openstack/neutron/+/805637 | 06:58 |
| *** rpittau|afk is now known as rpittau | 07:28 | |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [WIP] Do not use privsep daemon within DHCP agents https://review.opendev.org/c/openstack/neutron/+/808026 | 08:23 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Replace "tenant_id" with "project_id" in Quota engine https://review.opendev.org/c/openstack/neutron/+/805849 | 08:27 |
| lajoskatona | slaweq: Hi, shall I disturb you with https://review.opendev.org/c/openstack/releases/+/809571 - release patch for neutron-tempest-plugin ? | 09:16 |
| slaweq | lajoskatona: sure, I will check it now | 09:19 |
| lajoskatona | slaweq: thanks, my question is if we should wait till patches like https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/809770 or https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/800758 are merged, or release and tag tempest-plugin anyway? | 09:23 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-tempest-plugin master: Add skip function for IPv6Test tests https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/783802 | 09:23 |
| slaweq | lajoskatona: commented | 09:24 |
| slaweq | I would wait for few patches there | 09:24 |
| slaweq | if that's still possible | 09:24 |
| lajoskatona | slaweq: 👍 | 09:24 |
| lajoskatona | slaweq: I check it with release team | 09:26 |
| slaweq | lajoskatona++ | 09:30 |
| lajoskatona | ralonsoh: Hi, have you seen https://bugs.launchpad.net/neutron/+bug/1944201 ? | 11:38 |
| lajoskatona | ralonsoh: an example of q-agt failure: https://533d3c4cca911dc9dc31-b3f98f16f01c15606b481299d248970c.ssl.cf2.rackcdn.com/809550/3/gate/neutron-ovs-tempest-slow/c939a4f/controller/logs/screen-q-agt.txt | 11:38 |
| ralonsoh | lajoskatona, let me check | 11:39 |
| lajoskatona | ralonsoh: I checked if new release of ovsdbapp or os-ken can be behind it, but I see no connection (both ovsdbapp and os-ken was released 19. Aug) | 11:40 |
| lajoskatona | ralonsoh: thanks | 11:40 |
| lajoskatona | ralonsoh: and in the commit history nothing that could cause datapath id error.... | 11:40 |
| ralonsoh | I've seen that before when we had two bridges with the same ID | 11:41 |
| ralonsoh | physical bridges | 11:41 |
| ralonsoh | or when the bridge was deleted | 11:41 |
| lajoskatona | But in openstack CI, with freshly built test vm-s that would be strange | 11:46 |
| ralonsoh | yeah, I know | 11:47 |
| ralonsoh | lajoskatona, how big is the kibana buffer? 1 week? | 11:58 |
| lajoskatona | ralonsoh: no idea, perhaps | 12:00 |
| opendevreview | OpenStack Release Bot proposed openstack/neutron-vpnaas-dashboard stable/xena: Update .gitreview for stable/xena https://review.opendev.org/c/openstack/neutron-vpnaas-dashboard/+/810416 | 12:28 |
| opendevreview | OpenStack Release Bot proposed openstack/neutron-vpnaas-dashboard stable/xena: Update TOX_CONSTRAINTS_FILE for stable/xena https://review.opendev.org/c/openstack/neutron-vpnaas-dashboard/+/810417 | 12:28 |
| opendevreview | OpenStack Release Bot proposed openstack/neutron-vpnaas-dashboard master: Update master for stable/xena https://review.opendev.org/c/openstack/neutron-vpnaas-dashboard/+/810418 | 12:28 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron stable/wallaby: Populate self.floating_ips_dict using "ip rule" information https://review.opendev.org/c/openstack/neutron/+/810393 | 13:05 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron stable/victoria: Populate self.floating_ips_dict using "ip rule" information https://review.opendev.org/c/openstack/neutron/+/810394 | 13:05 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron stable/ussuri: Populate self.floating_ips_dict using "ip rule" information https://review.opendev.org/c/openstack/neutron/+/810425 | 13:08 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron stable/train: Populate self.floating_ips_dict using "ip rule" information https://review.opendev.org/c/openstack/neutron/+/810395 | 13:09 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron stable/stein: Populate self.floating_ips_dict using "ip rule" information https://review.opendev.org/c/openstack/neutron/+/810396 | 13:09 |
| 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 | 13:12 |
| 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 | 13:12 |
| ralonsoh | lajoskatona, I think I found something | 13:32 |
| ralonsoh | https://github.com/openvswitch/ovs/blob/master/ofproto/connmgr.c#L605-L612 | 13:32 |
| ralonsoh | I see this message twice per bridge | 13:32 |
| ralonsoh | br-ex, br-int, in all logs (I reviewed 10-12) | 13:32 |
| ralonsoh | e.g.: https://e36beaa2ff297ebe7d5f-5944c3d62ed334b8cdf50b534c246731.ssl.cf5.rackcdn.com/805849/9/check/neutron-ovs-tempest-dvr-ha-multinode-full/f83fa96/compute1/logs/openvswitch/ovs-vswitchd_log.txt | 13:32 |
| ralonsoh | this is happening when (1) we change the OF protocols (this is the error reported in the bug) | 13:33 |
| ralonsoh | and (2) when executing SetFailModeCommand | 13:33 |
| ralonsoh | that also affects of | 13:33 |
| ralonsoh | I'll comment that in the bug and push a patch (that patch will retry those methods affecting the OF controller, that needs to be restarted) | 13:34 |
| lajoskatona | ralonsoh: thanks | 13:35 |
| lajoskatona | ralonsoh: What I can't see why this become an issue now, there was no new release of these packages (ovs python for example) | 13:36 |
| ralonsoh | so far, I've seen this happening recurrently (at least during the last 7 days) | 13:37 |
| ralonsoh | I can't go farther in kibana | 13:37 |
| otherwiseguy | ksambor: ralonsoh: lucasagomes: if you get a sec, here's a simple review https://review.opendev.org/c/openstack/neutron/+/810309 that affects functional tests when running locally with newer versions of OVN | 13:55 |
| ralonsoh | sure | 13:56 |
| ksambor | ack | 13:58 |
| lucasagomes | otherwiseguy, checking | 13:58 |
| otherwiseguy | ralonsoh: i noticed that downstream, this is similar to a patch that you did to fix our gate at some point. | 14:03 |
| ralonsoh | otherwiseguy, yes but the destroy part was already there. But not in U/S | 14:05 |
| otherwiseguy | ETOOMANYBRANCHES | 14:05 |
| ralonsoh | otherwiseguy, just one question in L276 | 14:10 |
| ralonsoh | should this be chassis_private? | 14:10 |
| ralonsoh | same as D/S 232483 | 14:10 |
| otherwiseguy | ralonsoh: because we are passing the result to add_fake_chassis, which creates an actual chassis and then a chassis private if it is supported. | 14:11 |
| otherwiseguy | or del_fake_chassis, etc. | 14:11 |
| ralonsoh | ok, in both cases that will work | 14:12 |
| otherwiseguy | lucasagomes++ ksambor++ ralonsoh++ | 14:14 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: Revert "update subport status when trunk/subport create/delete is triggerred" https://review.opendev.org/c/openstack/neutron/+/809550 | 14:15 |
| lajoskatona | ralonsoh: what if we pin ovs to something, now as I see we use v2.15.0 (8dc1733eaea866dce033b3c44853e1b09bf59fc7 ) for API jobs and devstack pins it here: https://opendev.org/openstack/devstack/src/branch/master/lib/neutron_plugins/ovs_source#L22 | 14:16 |
| ralonsoh | lajoskatona, you mean changing the default devstack version? | 14:18 |
| lajoskatona | ralonsoh: changing default ovs we use | 14:18 |
| ralonsoh | that should work fine (if not, then we need to fix whatever is broken) | 14:19 |
| bcafarel | rubasov: hi, looking at https://bugs.launchpad.net/neutron/+bug/1930414 do you think it is High or Medium? (I was thinking the later but it is still potential security) | 14:43 |
| hyang[m] | slaweq: hi, I'd like to work on https://bugs.launchpad.net/neutron/+bug/1942615 and want to hear your thoughts on my proposal in the comment. Thanks! | 14:49 |
| slaweq_ | hyang[m]: hi | 14:49 |
| slaweq_ | hyang[m]: great that You want to work on it | 14:49 |
| slaweq_ | let me check Your comment | 14:49 |
| hyang[m] | ok :) | 14:50 |
| opendevreview | Terry Wilson proposed openstack/neutron master: Support SB OVSDB connections to non-leader servers https://review.opendev.org/c/openstack/neutron/+/803268 | 14:55 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Check quota limits https://review.opendev.org/c/openstack/neutron/+/801470 | 15:24 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [WIP] Execute the quota reservation removal in an isolated DB txn https://review.opendev.org/c/openstack/neutron/+/809983 | 15:24 |
| hyang[m] | slaweq_: Thanks for your reply, just want to ensure I understand it correctly: When use shared=True, Neutron only returns the network/SG is shared to the project who's making the request (nova and horizon's service project, in our example) but does not include the resources that are shared to the user's tenant which we want, is that right? | 15:24 |
| ralonsoh | hyang[m], I think slaweq_ if off for today. The goal of this "include_shared" is to include, in the query requested (for example "get_security_groups"), the SG groups owned by this tenant PLUS the SGs shared with it (not belonging to this tenant) | 15:27 |
| hyang[m] | ralonsoh: thanks for your answer. I want to understand the difference of using `include_shared` vs `search_opts = {'tenant_id': ABC, 'shared': True}` so the latter (if supported), means it returns the SGs owned by ABC tenant plus SGs shared to the user who made the request (may not be the ABC project), is that correct? | 15:31 |
| ralonsoh | hyang[m], I remember talking about this two weeks ago in the Neutron drivers meeting and I don't remember why that doesn't work | 15:34 |
| hyang[m] | And If we use `search_opts = {'tenant_id': ABC, 'include_shared': True}` then it guarantees to return SGs owned and shared with ABC tenant, regardless of who made the request. | 15:35 |
| ralonsoh | let me check again the meeting logs | 15:35 |
| hyang[m] | kk, thanks | 15:35 |
| hyang[m] | My current understanding of the shared filed is it only applies to the scope of the requestor's context. You cannot ask Neutron to return the shared resources for another project. | 15:36 |
| *** rpittau is now known as rpittau|afk | 15:45 | |
| ralonsoh | hyang[m], https://meetings.opendev.org/meetings/networking/2021/networking.2021-09-07-14.00.log.html | 15:46 |
| ralonsoh | so yes, the "shared" flag was the proposal accepted | 15:46 |
| ralonsoh | not "all_projects" flag | 15:46 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [WIP] Do not use privsep daemon within DHCP agents https://review.opendev.org/c/openstack/neutron/+/808026 | 15:55 |
| hyang[m] | ralonsoh: the decision from the meeting looks aligned with my proposal in https://bugs.launchpad.net/neutron/+bug/1942615 too. But slaweq just pointed out using `shared=True` does not work for resources shared to specific tenants instead of public resources that are shared to all. | 15:55 |
| ralonsoh | I think both of you are saying the same... I don't know | 15:57 |
| ralonsoh | try to clarify in the LP bug what slaweq_ wants to say in c#4 | 15:58 |
| opendevreview | Merged openstack/neutron-tempest-plugin master: Bump OVN version for master jobs https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/800758 | 16:26 |
| hyang[m] | ralonsoh: thanks for your help. I replied in the bug ticket and will wait for some clarifications from slaweq_ | 16:51 |
| *** mgoddard- is now known as mgoddard | 17:13 | |
| -opendevstatus- NOTICE: Zuul has been restarted in order to address a performance regression related to event processing; any changes pushed or approved between roughly 17:00 and 18:30 UTC should be rechecked if they're not already enqueued according to the Zuul status page | 18:37 | |
| *** slaweq_ is now known as slaweq | 19:26 | |
| opendevreview | Merged openstack/neutron-lib master: Remove deprecated method registry.notify https://review.opendev.org/c/openstack/neutron-lib/+/807869 | 19:44 |
| opendevreview | Merged openstack/neutron master: [ovn] metadata functional tests don't support Chassis_Private https://review.opendev.org/c/openstack/neutron/+/810309 | 20:49 |
| opendevreview | Merged openstack/neutron master: Drop install_venv https://review.opendev.org/c/openstack/neutron/+/809910 | 20:50 |
| opendevreview | Merged openstack/neutron master: [Docs] Ovn backend now supports FIP QoS bandwdith limiting https://review.opendev.org/c/openstack/neutron/+/777571 | 20:50 |
| opendevreview | Merged openstack/neutron master: Fix _create_dvr_floating_gw_port missing payload field https://review.opendev.org/c/openstack/neutron/+/809438 | 23:06 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!