| opendevreview | Miguel Lavalle proposed openstack/neutron master: [DNM] Add rate-limiting to metadata agents https://review.opendev.org/c/openstack/neutron/+/858879 | 00:34 |
|---|---|---|
| opendevreview | ZhouHeng proposed openstack/neutron master: [ovn]disable security group notifier https://review.opendev.org/c/openstack/neutron/+/873850 | 02:41 |
| opendevreview | yatin proposed openstack/neutron-tempest-plugin master: Add plugin.spec to irrelevant-files https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/878741 | 06:32 |
| opendevreview | yatin proposed openstack/neutron master: Add plugin.spec to irrelevant-files https://review.opendev.org/c/openstack/neutron/+/878742 | 06:44 |
| opendevreview | yatin proposed openstack/neutron master: Update url and package name https://review.opendev.org/c/openstack/neutron/+/863013 | 06:49 |
| opendevreview | yatin proposed openstack/neutron master: Add plugin.spec to irrelevant-files https://review.opendev.org/c/openstack/neutron/+/878742 | 06:57 |
| opendevreview | yatin proposed openstack/neutron master: Update url and package name https://review.opendev.org/c/openstack/neutron/+/863013 | 06:59 |
| fnordahl | slaweq: would you have a moment for a second review of https://review.opendev.org/c/openstack/neutron-specs/+/870030 ? I believe dmitriis responded to your comments, please let us know if that satisfies your requests | 07:43 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-tempest-plugin master: [Stateless SG] Add test to check connectivity between vms https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/874820 | 08:15 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-tempest-plugin master: [Stateless SG] Ensure replies won't work without ingress rule https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/877092 | 08:15 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-tempest-plugin master: [Stateless SG] Test if TCP packets with various conn state are allowed https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/877106 | 08:15 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-tempest-plugin master: [Stateless SG] Test fragmented traffic is allowed by stateless SG https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/877480 | 08:15 |
| slaweq | fnordahl hi, sure, I will check it today | 08:17 |
| slaweq | ralonsoh lajoskatona hi, can You check https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/877944 and patches on top of this one again? I rebased all of them | 08:17 |
| slaweq | thx in advance | 08:17 |
| ralonsoh | sure | 08:17 |
| ralonsoh | (I'll wait for the CI) | 08:18 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [DNM] Testing patch for https://review.opendev.org/c/openstack/neutron/+/878632 https://review.opendev.org/c/openstack/neutron/+/878746 | 08:19 |
| lajoskatona | slaweq: ack | 08:22 |
| ralonsoh | ykarel, hi! I've pushed https://review.opendev.org/c/openstack/neutron/+/878746/ | 09:03 |
| ralonsoh | but this is still using stable/yoga | 09:03 |
| ralonsoh | --> https://5ad53869f40e52779e3b-33d59b470f05b3a2fc5a2e384838ba54.ssl.cf1.rackcdn.com/878746/1/check/neutron-ovs-grenade-multinode-skip-level/c189920/job-output.txt | 09:04 |
| ralonsoh | do you know why? (maybe you know what is missing here) | 09:04 |
| frickler | ralonsoh: looking at _inheritance_path in https://5ad53869f40e52779e3b-33d59b470f05b3a2fc5a2e384838ba54.ssl.cf1.rackcdn.com/878746/1/check/neutron-ovs-grenade-multinode-skip-level/c189920/zuul-info/inventory.yaml it seems to be using the stable/2023.1 job definition | 09:13 |
| ralonsoh | frickler, but it should be using master, I don't understand | 09:22 |
| sahid | o/ quick question, regarding osprofiler and neutron, any idea of the current status? I have difficulty to get traces from it | 09:32 |
| ralonsoh | sahid, please specify the question. There are many references/uses of the osprofiler in Neutron | 09:40 |
| sahid | i have done some quick tests running command openstack network list --profile foobar, but traces do not look to be generated | 09:42 |
| sahid | where i can see traces generated by keystone | 09:43 |
| sahid | looks like middleware is well installed and [oprofiler] section well intruced with enabled = True | 09:43 |
| ralonsoh | sahid, and what is in api-paste.ini? | 09:45 |
| sahid | [filter:osprofiler] | 09:46 |
| sahid | paste.filter_factory = osprofiler.web:WsgiMiddleware.factory | 09:46 |
| ralonsoh | no, the neutron api | 09:46 |
| ralonsoh | composite:neutronapi_v2_0 | 09:46 |
| sahid | oh... | 09:46 |
| sahid | [app:neutronapiapp_v2_0] | 09:47 |
| sahid | paste.app_factory = neutron.api.v2.router:APIRouter.factory | 09:47 |
| ralonsoh | composite:neutronapi_v2_0 | 09:47 |
| sahid | oh yes sorry | 09:48 |
| sahid | [composite:neutronapi_v2_0] | 09:48 |
| sahid | use = call:neutron.auth:pipeline_factory | 09:48 |
| sahid | noauth = cors http_proxy_to_wsgi request_id fake_project_id catch_errors osprofiler extensions neutronapiapp_v2_0 | 09:48 |
| sahid | keystone = cors http_proxy_to_wsgi request_id catch_errors osprofiler authtoken keystonecontext extensions neutronapiapp_v2_0 | 09:48 |
| frickler | ralonsoh: this is what I see in the zuul log. maybe zuul chooses any of the possible branches randomly, then the branch matcher for stable branches would need to be updated https://paste.opendev.org/show/bJv2F8LVX3iiaicdtHrm/ | 09:49 |
| ralonsoh | frickler, yes, I've commented in the patch | 09:49 |
| ralonsoh | this is not supported in 2023.2 | 09:49 |
| ralonsoh | I'm pushing a new PS | 09:49 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Do not execute "-skip-level" jobs during 2023.2 release https://review.opendev.org/c/openstack/neutron/+/878632 | 09:50 |
| ralonsoh | sahid, you should see the profiler lines in the logs | 09:51 |
| sahid | osprofiler is enabled with connection string: jaeger+otlp://localhost:4318 - I think we are good. no worry ralonsoh I will investigate, thank you for this sanity check | 09:54 |
| frickler | ralonsoh: iiuc the job should still work, but deciding that it is not needed is certainly a valid option, too | 10:02 |
| ralonsoh | frickler, it can work but we need to modify the allowed distros in yoga | 10:02 |
| ralonsoh | because jammy is not included, or use FORCE=yes | 10:02 |
| frickler | if you want stable/yoga as base, you need to use focal, not jammy. or some other distro possibly | 10:04 |
| ralonsoh | but do we support 2023.2 in focal? | 10:04 |
| frickler | for devstack we haven't dropped it yet at least, not sure if that should happen | 10:06 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-tempest-plugin master: [Secure RBAC] Add scope enforcement enabled job for Zed branch https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/874709 | 10:06 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-tempest-plugin master: DNM Just test of the Zed job's time execution https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/876869 | 10:06 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-tempest-plugin master: DNM Just test of the Zed job's time execution https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/876869 | 10:08 |
| frickler | debian 11 would be an option that is unchanged from yoga until now | 10:09 |
| frickler | for ubuntu only 22.04 is listed by the tc https://governance.openstack.org/tc/reference/runtimes/2023.2.html | 10:09 |
| ralonsoh | that's the point, 2023.2 only supports ubuntu jammy | 10:12 |
| ralonsoh | it makes no sense to test it in focal and in a non-slurp release | 10:13 |
| ralonsoh | I think we can save CI time during 2023.2 | 10:13 |
| opendevreview | Merged openstack/neutron-tempest-plugin master: Skip stateless SG related tests for backends which don't support it https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/877944 | 10:26 |
| opendevreview | Luis Tomas Bolivar proposed openstack/neutron master: Ensure redirect-type=bridged not used for geneve networks https://review.opendev.org/c/openstack/neutron/+/878450 | 10:45 |
| ykarel | ralonsoh, as frickler said the duplicated job variants(in master and stable/2023.1 branch) seems causing it | 11:12 |
| ykarel | will push a patch to fix it | 11:13 |
| ralonsoh | ykarel, what duplicated variants? | 11:13 |
| ralonsoh | I think what is causing that is the branches regex | 11:13 |
| ralonsoh | https://review.opendev.org/c/openstack/neutron/+/878632/1..2/zuul.d/grenade.yaml#b380 | 11:13 |
| ralonsoh | we don't allow to use zed as "grenade_from_branch"# | 11:14 |
| opendevreview | yatin proposed openstack/neutron stable/2023.1: [stable/2023.1 only] Drop branches regex from skip level jobs https://review.opendev.org/c/openstack/neutron/+/878759 | 11:16 |
| ralonsoh | ykarel, but why? | 11:17 |
| ralonsoh | this is how is supposed to be | 11:17 |
| ykarel | ralonsoh, see https://zuul.openstack.org/job/neutron-ovn-grenade-multinode-skip-level | 11:17 |
| ralonsoh | this is exactly what I'm saying in https://review.opendev.org/c/openstack/neutron/+/878632/1..2/zuul.d/grenade.yaml#b380 | 11:17 |
| ralonsoh | I saw that | 11:17 |
| ralonsoh | I know | 11:17 |
| ralonsoh | and this is why we should not be testing it | 11:17 |
| ykarel | for master branch both version matches ^ | 11:17 |
| ykarel | which is wrong | 11:18 |
| ralonsoh | this is what I'm saying | 11:18 |
| ralonsoh | and is not wrong | 11:18 |
| ralonsoh | is because this is a non slurp release | 11:18 |
| ykarel | ralonsoh, grenade_from_branch is not related to branches: at all AFAIK | 11:20 |
| ykarel | i think the above stable/2023.1 patch should fix the previous issue seen(job was not running) in your patch | 11:21 |
| ralonsoh | it was in the DNM patch I sent | 11:22 |
| ralonsoh | and it was not working | 11:22 |
| ralonsoh | because we are using stable/yoga | 11:22 |
| ralonsoh | and we use jammy in this job | 11:22 |
| ykarel | yes above patch will also fix that | 11:22 |
| ralonsoh | why? if we are still using stable/yoga | 11:23 |
| ralonsoh | https://review.opendev.org/c/openstack/neutron/+/878759/1/zuul.d/grenade.yaml#368 this won't fix the issue | 11:23 |
| ykarel | your first PR https://review.opendev.org/c/openstack/neutron/+/878632/1 was good one, just we need to decide do we want to run for non slurp release | 11:24 |
| ykarel | i think we can keep running it but in periodic/experimental for non slurp and check/gate for slurp ones | 11:25 |
| ralonsoh | ykarel, no, PS1 doesn't work because is running on jammy nodes | 11:25 |
| ykarel | and https://review.opendev.org/c/openstack/neutron/+/878759 will just fix the conflicting job variants | 11:25 |
| ralonsoh | and devstack yoga doesn't have this release | 11:25 |
| ykarel | your PS1 set grenade_from_branch=stable/zed so it will work | 11:26 |
| ralonsoh | no, grenade is NOT installing zed | 11:26 |
| ykarel | just we need stable/2023.1 patch to get it work | 11:26 |
| ralonsoh | is installing yoga | 11:26 |
| ykarel | to fix ^ case we need to merge https://review.opendev.org/c/openstack/neutron/+/878759 | 11:27 |
| ykarel | let me try without merging if that works | 11:27 |
| opendevreview | yatin proposed openstack/neutron master: [DNM] Test 878759 https://review.opendev.org/c/openstack/neutron/+/878761 | 11:29 |
| ykarel | let's see if that works, after merge should work, me not sure if across branches it will work or not | 11:30 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: dnm: ci: MacvtapAgentTestCase.test_get_all_devices https://review.opendev.org/c/openstack/neutron/+/878762 | 11:30 |
| ykarel | didn't worked we need to merge that patch to fix the issue | 11:31 |
| ralonsoh | ykarel, come on | 11:31 |
| ralonsoh | please check the previous patches | 11:31 |
| ralonsoh | https://review.opendev.org/c/openstack/neutron/+/878761/1/zuul.d/grenade.yaml doesn't work | 11:31 |
| ralonsoh | we are using jammy, grenade will NOT checkout zed but yoga | 11:31 |
| ralonsoh | and devstack yoga doesn't support jammy | 11:32 |
| opendevreview | yatin proposed openstack/neutron master: [DNM] Test 878759 https://review.opendev.org/c/openstack/neutron/+/878761 | 11:32 |
| ykarel | ralonsoh, that will work, jobs running in ^ will confirm, i missed depends-on | 11:33 |
| ykarel | https://zuul.opendev.org/t/openstack/status#878761 | 11:34 |
| ykarel | not sure if you received last message | 11:37 |
| ykarel | https://zuul.opendev.org/t/openstack/status#878761 will confirm if that works | 11:37 |
| ralonsoh | 2023.2 is not a slurp version | 11:38 |
| ralonsoh | we should NOT remove the branches regez | 11:38 |
| ralonsoh | and we should not be testing -skip-level during this release | 11:39 |
| ralonsoh | that will work but this is not the point | 11:39 |
| ykarel | yes i meant that only if team decides not to run it in non slurp it's ok as it's an option | 11:42 |
| ykarel | not fixing branches in stable/2023.1 will have issues when we start running that job in next slurp so that always has to be fixed | 11:43 |
| ralonsoh | ok but then we should not remove branches and only change yoga with zed | 11:44 |
| ykarel | we need to drop branches: from stable/2023.1 neutron branch, for master branch ^ can be an option | 11:48 |
| ykarel | The test patch is working fine TARGET_BRANCH=stable/zed | 11:49 |
| ykarel | https://zuul.opendev.org/t/openstack/stream/e453b41379c943258408f8348ded25a4?logfile=console.log | 11:49 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: Ensure redirect-type=bridged not used for geneve networks https://review.opendev.org/c/openstack/neutron/+/878450 | 12:14 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-tempest-plugin master: [Secure RBAC] Add scope enforcement enabled job for Zed branch https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/874709 | 12:20 |
| slaweq | ralonsoh I was checking this stable/zed slowness with S-RBAC in https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/874709 and I added comment there. Please check it and let me know if that's ok for You :) | 12:21 |
| *** gthiemon1e is now known as gthiemonge | 12:24 | |
| ralonsoh | let me check | 12:34 |
| opendevreview | Luis Tomas Bolivar proposed openstack/neutron master: Ensure redirect-type=bridged not used for geneve networks https://review.opendev.org/c/openstack/neutron/+/878450 | 12:40 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [ovn] Add end to end test for QosExtension https://review.opendev.org/c/openstack/neutron/+/877603 | 12:52 |
| opendevreview | yatin proposed openstack/neutron stable/zed: Make path for tcpdump release-specific https://review.opendev.org/c/openstack/neutron/+/878662 | 12:53 |
| opendevreview | yatin proposed openstack/neutron master: [DNM] Test 878759 https://review.opendev.org/c/openstack/neutron/+/878761 | 12:54 |
| ralonsoh | hi folks!! 1 minute to go | 12:58 |
| ralonsoh | --> https://ptg.opendev.org/ptg.html | 12:58 |
| ralonsoh | https://www.openstack.org/ptg/rooms/juno | 12:58 |
| ralonsoh | you can download de app or use the web browser | 12:59 |
| ralonsoh | #link https://etherpad.opendev.org/p/neutron-bobcat-ptg | 13:05 |
| opendevreview | Merged openstack/neutron master: Revert "Ensure vlan network traffic is not centralized" https://review.opendev.org/c/openstack/neutron/+/878441 | 13:44 |
| opendevreview | Frode Nordahl proposed openstack/neutron master: [ovn] Add end to end test for QosExtension https://review.opendev.org/c/openstack/neutron/+/877603 | 14:13 |
| opendevreview | Frode Nordahl proposed openstack/neutron master: [ovn] Drop use of OVN_GW_PORT_EXT_ID_KEY https://review.opendev.org/c/openstack/neutron/+/877831 | 14:13 |
| opendevreview | Frode Nordahl proposed openstack/neutron master: [ovn] Drop use of LR OVN_GW_NETWORK_EXT_ID_KEY https://review.opendev.org/c/openstack/neutron/+/877712 | 14:13 |
| opendevreview | Frode Nordahl proposed openstack/neutron master: [ovn] OVNClient._get_router_ports: Drop unused parameter https://review.opendev.org/c/openstack/neutron/+/878527 | 14:13 |
| opendevreview | Frode Nordahl proposed openstack/neutron master: [ovn] Implement support for external-gateway-multihoming extension https://review.opendev.org/c/openstack/neutron/+/874199 | 14:13 |
| opendevreview | Frode Nordahl proposed openstack/neutron master: [ovn] Honor `enable_default_route_ecmp` attribute https://review.opendev.org/c/openstack/neutron/+/878531 | 14:13 |
| opendevreview | Frode Nordahl proposed openstack/neutron master: [ovn] Allow L3 scheduler to be aware of current transaction https://review.opendev.org/c/openstack/neutron/+/874760 | 14:13 |
| opendevreview | Frode Nordahl proposed openstack/neutron master: [ovn] Add helper for retrieving LR associated with LRP https://review.opendev.org/c/openstack/neutron/+/873698 | 14:13 |
| opendevreview | Frode Nordahl proposed openstack/neutron master: [ovn] Apply soft anti-affinity for LRs with multiple LRPs when scheduling https://review.opendev.org/c/openstack/neutron/+/873699 | 14:13 |
| opendevreview | Frode Nordahl proposed openstack/neutron master: WIP [ovn] Add support for enable_default_route_bfd attribute https://review.opendev.org/c/openstack/neutron/+/878543 | 14:13 |
| opendevreview | Merged openstack/neutron-tempest-plugin master: [Stateless SG] Add test to check connectivity between vms https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/874820 | 14:52 |
| opendevreview | Merged openstack/neutron-tempest-plugin master: [Stateless SG] Ensure replies won't work without ingress rule https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/877092 | 14:52 |
| opendevreview | Merged openstack/neutron-tempest-plugin master: [Stateless SG] Test if TCP packets with various conn state are allowed https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/877106 | 14:52 |
| mlavalle1 | ralonsoh: Everything Everywhere All at Once | 14:57 |
| opendevreview | Merged openstack/neutron stable/2023.1: Revert "Ensure vlan network traffic is not centralized" https://review.opendev.org/c/openstack/neutron/+/878442 | 15:31 |
| frickler | ralonsoh: matfechner is a colleague of mine and also around here. not sure if that is meant to be an operator topic or just added there because of the "topics to be discussed" header | 15:31 |
| ralonsoh | frickler, that's perfect, we can have this discussion in this time slot | 15:32 |
| opendevreview | Merged openstack/neutron stable/zed: Revert "Ensure vlan network traffic is not centralized" https://review.opendev.org/c/openstack/neutron/+/878443 | 15:42 |
| racosta | ralonsoh, do you believe we can discuss in tomorrow's operator time slot about the OVN backend? more specifically, about features not supported by Neutron and which require solutions "under the hood", like ovn-bgp-agent that creates openflow flows directly in ovs (agent design! but it couldn't be done using OVN because of Neutron) or solutions that create a whole layer to integrate OVN requirements with Neutron, like: | 16:20 |
| racosta | https://review.opendev.org/c/openstack/neutron-vpnaas/+/765353. | 16:20 |
| racosta | I know that each case is different, my point is about ovn db sync. Neutron drops everything that is not managed by CMS (and this is very good for consistency). However the backend of ovn is very flexible and there are unsupported solutions that cannot be integrated, for example ovn-ic (OVN interconnect). The ovn-kube project are using ovn-ic and integrating with OpenStack - https://github.com/kubeovn/kube-ovn/blob/maste | 16:20 |
| racosta | r/docs/OpenStackK8sInterconnection.md | 16:20 |
| racosta | Would it be possible to make the ovn db sync more flexible? create some mechanism integrated with Neutron to meet integration solutions and different network architectures that can coexist without problems? | 16:20 |
| ralonsoh | racosta, so you mean a way to extend the DB sync tool, right? | 16:21 |
| racosta | yes | 16:21 |
| ralonsoh | about the schedule, we can't use a time slot already assigned | 16:21 |
| lajoskatona | racosta: Hi, that patch is huge :-) To make review easier can't it be cut to smaller pieces? I try to find some time to review it | 16:22 |
| ralonsoh | (we have been calling for topics for 1 month) | 16:22 |
| ralonsoh | racosta, let me check if we can schedule this topic on Thursday | 16:22 |
| ralonsoh | racosta, first add it to https://etherpad.opendev.org/p/neutron-bobcat-ptg, in the "Topics" section (at the end) | 16:23 |
| ralonsoh | vmaccel, hi, this is about a new topic I see in the agenda | 16:23 |
| ralonsoh | when was that added? | 16:23 |
| racosta | No worries, It's just a concern about OVN integration, not something urgent. | 16:24 |
| ralonsoh | racosta, but is better if we track it in the PTG | 16:24 |
| ralonsoh | add this topic there and I'll schedule it for the next thursday | 16:25 |
| racosta | lajoskatona, it's not my patch, it was just an example. | 16:26 |
| lajoskatona | racosta: ahh, ok | 16:40 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Replace "tenant_id" with "project_id" in IPAM engine https://review.opendev.org/c/openstack/neutron/+/877533 | 17:09 |
| opendevreview | Luis Tomas Bolivar proposed openstack/neutron master: Ensure redirect-type=bridged not used for geneve networks https://review.opendev.org/c/openstack/neutron/+/878450 | 17:15 |
| *** sfinucan is now known as stephenfin | 17:20 | |
| opendevreview | Merged openstack/neutron stable/yoga: Revert "Ensure vlan network traffic is not centralized" https://review.opendev.org/c/openstack/neutron/+/878444 | 18:45 |
| opendevreview | Merged openstack/neutron stable/xena: Revert "Ensure vlan network traffic is not centralized" https://review.opendev.org/c/openstack/neutron/+/878445 | 18:45 |
| opendevreview | Merged openstack/neutron stable/wallaby: Revert "Ensure vlan network traffic is not centralized" https://review.opendev.org/c/openstack/neutron/+/878446 | 18:45 |
| vmaccel | ralonsoh I added it yesterday | 19:19 |
| vmaccel | It's on the nova agenda for the nova/neutron joint session, but it was never added on the neutron side | 19:20 |
| opendevreview | Miguel Lavalle proposed openstack/neutron master: [DNM] Add rate-limiting to metadata agents https://review.opendev.org/c/openstack/neutron/+/858879 | 23:41 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!