| opendevreview | Miguel Lavalle proposed openstack/neutron master: [DNM] Add rate-limiting to metadata agents https://review.opendev.org/c/openstack/neutron/+/858879 | 00:08 |
|---|---|---|
| opendevreview | Miguel Lavalle proposed openstack/neutron master: [DNM] Add rate-limiting to metadata agents https://review.opendev.org/c/openstack/neutron/+/858879 | 03:11 |
| opendevreview | Luis Tomas Bolivar proposed openstack/neutron stable/zed: Ensure redirect-type=bridged not used for geneve networks https://review.opendev.org/c/openstack/neutron/+/879296 | 05:35 |
| opendevreview | Luis Tomas Bolivar proposed openstack/neutron stable/yoga: Ensure redirect-type=bridged not used for geneve networks https://review.opendev.org/c/openstack/neutron/+/879297 | 05:39 |
| opendevreview | Luis Tomas Bolivar proposed openstack/neutron stable/xena: Ensure redirect-type=bridged not used for geneve networks https://review.opendev.org/c/openstack/neutron/+/879298 | 05:40 |
| opendevreview | Luis Tomas Bolivar proposed openstack/neutron stable/wallaby: Ensure redirect-type=bridged not used for geneve networks https://review.opendev.org/c/openstack/neutron/+/879299 | 05:41 |
| slaweq | ralonsoh ykarel lajoskatona hi, please check https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/873380 when You will have time, thx in advance | 06:38 |
| ykarel | slaweq, ack btw i notice merge conflict there | 06:45 |
| ykarel | slaweq, when you get chance please check https://review.opendev.org/c/openstack/neutron/+/878759 | 06:46 |
| slaweq | ykarel 878759 approved | 06:51 |
| slaweq | ykarel where You noticed this merge conflict? | 06:51 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-lib stable/2023.1: Use proper neutron-tempest-plugin template in 2023.1 branch https://review.opendev.org/c/openstack/neutron-lib/+/879421 | 06:56 |
| ykarel | slaweq, https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/873380 | 06:57 |
| ykarel | merge conflict in ^ | 06:57 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-lib stable/yoga: Use neutron-tempest-plugin-api-yoga job in the stable/yoga branch https://review.opendev.org/c/openstack/neutron-lib/+/879422 | 06:58 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-lib stable/xena: Use neutron-tempest-plugin-api-xena job in stable/xena branch https://review.opendev.org/c/openstack/neutron-lib/+/879423 | 07:00 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-lib stable/wallaby: Add neutron-lib to required projects in neutron-tempest-plugin-api job https://review.opendev.org/c/openstack/neutron-lib/+/879424 | 07:02 |
| opendevreview | Merged openstack/neutron stable/2023.1: [stable/2023.1 only] Drop branches regex from skip level jobs https://review.opendev.org/c/openstack/neutron/+/878759 | 07:09 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [sqlalchemy-20] Enable SQLAlchemy 2.0 deprecation warnings https://review.opendev.org/c/openstack/neutron/+/874778 | 07:23 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-tempest-plugin master: [Stateless SG] Add IPv6 and DHCP related tests https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/873380 | 07:27 |
| slaweq | ykarel strange as gerrit didn't show me this conflict | 07:27 |
| slaweq | I had to click rebase on top of master and then it showed me it | 07:27 |
| slaweq | now should be fine | 07:28 |
| ykarel | hmm looks good now | 07:36 |
| *** elodilles_pto is now known as elodilles | 08:04 | |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Bump skip-level lower version to stable/zed https://review.opendev.org/c/openstack/neutron/+/878632 | 08:06 |
| sahid | o/ | 08:07 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Bump skip-level lower version to stable/zed https://review.opendev.org/c/openstack/neutron/+/878632 | 08:07 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: rbacs: filter out model that are already owned by context https://review.opendev.org/c/openstack/neutron/+/871113 | 08:16 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: rbacs: fix typo, s/cxt/ctx https://review.opendev.org/c/openstack/neutron/+/872762 | 08:16 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-lib stable/wallaby: Add neutron-lib to required projects in neutron-tempest-plugin-api job https://review.opendev.org/c/openstack/neutron-lib/+/879424 | 09:07 |
| opendevreview | Merged openstack/neutron stable/2023.1: Ensure redirect-type=bridged not used for geneve networks https://review.opendev.org/c/openstack/neutron/+/879295 | 09:54 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Mark "test_multiple_agents_for_network" as unstable https://review.opendev.org/c/openstack/neutron/+/879435 | 10:04 |
| opendevreview | Frode Nordahl proposed openstack/neutron master: Allow Multiple External Gateways https://review.opendev.org/c/openstack/neutron/+/873593 | 10:05 |
| opendevreview | Frode Nordahl proposed openstack/neutron master: Add extra router attributes for ECMP and BFD https://review.opendev.org/c/openstack/neutron/+/874797 | 10:05 |
| opendevreview | Frode Nordahl proposed openstack/neutron master: [ovn] Add end to end test for QosExtension https://review.opendev.org/c/openstack/neutron/+/877603 | 10:05 |
| 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 | 10:05 |
| 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 | 10:05 |
| opendevreview | Frode Nordahl proposed openstack/neutron master: [ovn] OVNClient._get_router_ports: Drop unused parameter https://review.opendev.org/c/openstack/neutron/+/878527 | 10:05 |
| opendevreview | Frode Nordahl proposed openstack/neutron master: [ovn] Implement support for external-gateway-multihoming extension https://review.opendev.org/c/openstack/neutron/+/874199 | 10:05 |
| opendevreview | Frode Nordahl proposed openstack/neutron master: [ovn] Honor `enable_default_route_ecmp` attribute https://review.opendev.org/c/openstack/neutron/+/878531 | 10:05 |
| 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 | 10:05 |
| opendevreview | Frode Nordahl proposed openstack/neutron master: [ovn] Add helper for retrieving LR associated with LRP https://review.opendev.org/c/openstack/neutron/+/873698 | 10:05 |
| 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 | 10:05 |
| opendevreview | Frode Nordahl proposed openstack/neutron master: [ovn] Add support for enable_default_route_bfd attribute https://review.opendev.org/c/openstack/neutron/+/878543 | 10:05 |
| opendevreview | Merged openstack/neutron stable/zed: Ensure redirect-type=bridged not used for geneve networks https://review.opendev.org/c/openstack/neutron/+/879296 | 10:14 |
| opendevreview | Merged openstack/neutron stable/yoga: Ensure redirect-type=bridged not used for geneve networks https://review.opendev.org/c/openstack/neutron/+/879297 | 10:14 |
| opendevreview | Merged openstack/neutron master: Try to optimize Mysql server mem usage on some CI jobs https://review.opendev.org/c/openstack/neutron/+/876556 | 10:14 |
| ralonsoh | lajoskatona, slaweq https://review.opendev.org/c/openstack/neutron/+/879435 | 10:18 |
| ralonsoh | ^ folks, this test is failing too often | 10:18 |
| ralonsoh | until we fix it, let's skip it | 10:18 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-lib stable/wallaby: Add neutron-lib to required projects in neutron-tempest-plugin-api job https://review.opendev.org/c/openstack/neutron-lib/+/879424 | 10:53 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-lib stable/wallaby: Don't check context.system_scope to check project scope https://review.opendev.org/c/openstack/neutron-lib/+/874397 | 10:53 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-lib stable/wallaby: Skip test_qos_dscp_create_and_update API test in stable/wallaby https://review.opendev.org/c/openstack/neutron-lib/+/879441 | 10:53 |
| opendevreview | Merged openstack/neutron-lib stable/2023.1: Use proper neutron-tempest-plugin template in 2023.1 branch https://review.opendev.org/c/openstack/neutron-lib/+/879421 | 12:12 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-lib stable/wallaby: Skip test_qos_dscp_create_and_update API test in stable/wallaby https://review.opendev.org/c/openstack/neutron-lib/+/879441 | 12:23 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-lib stable/wallaby: Add neutron-lib to required projects in neutron-tempest-plugin-api job https://review.opendev.org/c/openstack/neutron-lib/+/879424 | 12:28 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-lib stable/wallaby: Don't check context.system_scope to check project scope https://review.opendev.org/c/openstack/neutron-lib/+/874397 | 12:28 |
| opendevreview | Merged openstack/neutron master: Fix a number of configuration typos https://review.opendev.org/c/openstack/neutron/+/878180 | 12:53 |
| opendevreview | Merged openstack/neutron stable/xena: Ensure redirect-type=bridged not used for geneve networks https://review.opendev.org/c/openstack/neutron/+/879298 | 12:53 |
| opendevreview | Terry Wilson proposed openstack/ovsdbapp master: Add new function ls_get_localnet_ports https://review.opendev.org/c/openstack/ovsdbapp/+/873853 | 13:36 |
| opendevreview | Brian Haley proposed openstack/neutron master: Suppress IPv6 metadata DAD failure and delete address https://review.opendev.org/c/openstack/neutron/+/876566 | 13:42 |
| mlavalle2 | rubasov: hi. I am continuing the implementation of this https://review.opendev.org/c/openstack/neutron/+/858879. I am trying to add a functional test case of metadata access over ipv6: https://review.opendev.org/c/openstack/neutron/+/858879/19/neutron/tests/functional/agent/l3/test_metadata_proxy.py#135 | 13:56 |
| mlavalle2 | rubasov: but I continue getting this error: https://f67f2b5b2b13ee2479e6-b4b40f9bd28a71d14a98889c916e3212.ssl.cf1.rackcdn.com/858879/19/check/neutron-functional-with-uwsgi/7f597a7/testr_results.html | 13:57 |
| mlavalle2 | rubasov: any suggestions? | 13:57 |
| mlavalle2 | thanks in advance :-) | 13:57 |
| rubasov | mlavalle2: hi, looking | 13:57 |
| mlavalle2 | :-) | 13:58 |
| ralonsoh | #startmeeting networking | 14:01 |
| opendevmeet | Meeting started Tue Apr 4 14:01:58 2023 UTC and is due to finish in 60 minutes. The chair is ralonsoh. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:01 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:01 |
| opendevmeet | The meeting name has been set to 'networking' | 14:01 |
| ralonsoh | hello all | 14:02 |
| mlavalle2 | o/ | 14:02 |
| rubasov | o/ | 14:02 |
| haleyb | o/ | 14:02 |
| ykarel | o/ | 14:02 |
| ralonsoh | ping bcafarel, elvira, frickler, mlavalle, mtomaska, obondarev, sahid, slawek, tobias-urdin, ykarel, lajoskatona, jlibosva, averdagu, amotoki | 14:02 |
| bcafarel | o/ | 14:02 |
| obondarev | hi | 14:02 |
| ralonsoh | ok, we can start | 14:03 |
| ralonsoh | #topic announcements | 14:03 |
| ralonsoh | congrats everyone, this is the first week of Bobcat | 14:03 |
| ralonsoh | #link https://releases.openstack.org/bobcat/schedule.html | 14:03 |
| mlavalle2 | \o/ | 14:04 |
| ralonsoh | in 1 month, we'll have the first milestone | 14:04 |
| mlavalle2 | the inexorable march of the cycle.... | 14:04 |
| ralonsoh | I would recommend you to check the PTG summary | 14:04 |
| lajoskatona | o/ | 14:04 |
| ralonsoh | #link https://lists.openstack.org/pipermail/openstack-discuss/2023-April/033115.html | 14:05 |
| ralonsoh | and the etherpad used during the PTG | 14:05 |
| ralonsoh | #link https://etherpad.opendev.org/p/neutron-bobcat-ptg | 14:05 |
| mlavalle2 | Thanks for the summary! | 14:05 |
| obondarev | thanks for the summary ralonsoh | 14:05 |
| lajoskatona | +1 | 14:05 |
| ralonsoh | thanks folks | 14:05 |
| ralonsoh | you can also watch the video from openinfra | 14:06 |
| ralonsoh | #link https://openinfra.dev/live/#all-episodes | 14:06 |
| ralonsoh | about Antelope | 14:06 |
| ralonsoh | and last but not least | 14:06 |
| ralonsoh | #link https://etherpad.opendev.org/p/neutron-vancouver-2023 | 14:06 |
| ralonsoh | ^^ Vancouver PTG! | 14:06 |
| ralonsoh | please check the etherpad and add anything you want to discuss | 14:07 |
| ralonsoh | and that's all | 14:07 |
| ralonsoh | do you have something else? | 14:07 |
| ralonsoh | ok, let's jump to the next topic | 14:08 |
| ralonsoh | #topic bugs | 14:08 |
| ralonsoh | we have the report from amotoki | 14:08 |
| ralonsoh | #link https://lists.openstack.org/pipermail/openstack-discuss/2023-March/032995.html | 14:08 |
| ralonsoh | mlavalle2, did you send yours from last week? | 14:08 |
| mlavalle2 | it wasn't my tuen last week | 14:08 |
| ralonsoh | maybe I missed from the list | 14:08 |
| mlavalle2 | turn | 14:09 |
| ralonsoh | 2023-3-27 Miguel Lavalle (mlavalle) | 14:09 |
| ralonsoh | https://wiki.openstack.org/wiki/Network/Meetings | 14:09 |
| mlavalle2 | oops it was | 14:09 |
| mlavalle2 | I missed it | 14:09 |
| mlavalle2 | sorry | 14:09 |
| lajoskatona | :-) | 14:09 |
| ralonsoh | np | 14:09 |
| ralonsoh | let's check amotoki's list | 14:09 |
| ralonsoh | #link https://bugs.launchpad.net/neutron/+bug/2012391 | 14:10 |
| ralonsoh | I replied on c#2 | 14:10 |
| ralonsoh | if we are sending the correct information to Nova, Nova should be building the metadata correctly | 14:10 |
| ralonsoh | and I think is done by https://review.opendev.org/c/openstack/nova/+/778549/7/nova/api/openstack/compute/schemas/servers.py#b362 | 14:11 |
| ralonsoh | but that should be answered, if I'm not wrong, by Nova folks | 14:11 |
| ralonsoh | in any case, please check this bug if you have any other idea about it | 14:11 |
| * mlavalle2 will send a report for last week later today | 14:11 | |
| ralonsoh | the next one is | 14:12 |
| ralonsoh | #link https://bugs.launchpad.net/neutron/+bug/2012332 | 14:12 |
| ralonsoh | that is a RFE | 14:12 |
| ralonsoh | I'll add it to the drivers meeting agenda | 14:12 |
| ralonsoh | (but not for this week, there will be no meeting this Friday) | 14:12 |
| ralonsoh | I think (we'll decide it) this RFE can be a spec-less one | 14:13 |
| ralonsoh | but we'll talk about it next week | 14:13 |
| ralonsoh | something else I'm missing? | 14:13 |
| ralonsoh | (the rest of bugs are addressed or assigned) | 14:14 |
| mlavalle2 | as I said, I will send a report later today | 14:14 |
| mlavalle2 | for last week | 14:14 |
| ralonsoh | no problem, we'll check that next week | 14:14 |
| ralonsoh | we don't have any open (an active) spec | 14:15 |
| ralonsoh | and this is too soon to talk about community goals (neutronclient or RBAC) | 14:15 |
| ralonsoh | we can wait one week to have some progress | 14:15 |
| ralonsoh | so I'll jump to the last topic of today | 14:15 |
| ralonsoh | #on_demand | 14:15 |
| ralonsoh | sorry | 14:15 |
| ralonsoh | #topic on_demand | 14:16 |
| ralonsoh | The one I added in the list was already discussed in the PTG | 14:16 |
| ralonsoh | so I have nothing in the agenda | 14:16 |
| ralonsoh | do you? | 14:16 |
| lajoskatona | nothing from me | 14:16 |
| ralonsoh | if not, 45 minutes back for you | 14:16 |
| lajoskatona | \o/ | 14:16 |
| ralonsoh | please remember: no CI meeting, no drivers meeting | 14:16 |
| ralonsoh | this week | 14:17 |
| mlavalle2 | ahh, ok, I'll use this time for my bugs report | 14:17 |
| ralonsoh | so let's start this release with good mood and enjoy | 14:17 |
| ralonsoh | see you all | 14:17 |
| lajoskatona | Bye | 14:17 |
| sahid | ++ | 14:17 |
| ralonsoh | #endmeeting | 14:17 |
| opendevmeet | Meeting ended Tue Apr 4 14:17:47 2023 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:17 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/networking/2023/networking.2023-04-04-14.01.html | 14:17 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/networking/2023/networking.2023-04-04-14.01.txt | 14:17 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/networking/2023/networking.2023-04-04-14.01.log.html | 14:17 |
| ykarel | bye | 14:17 |
| obondarev | o/ | 14:17 |
| mlavalle2 | o/ | 14:17 |
| rubasov_ | mlavalle2: regarding the l3 v6 metadata functional test I'm not sure about the cause at first blick, but the first thing I would check is that the haproxy config actually works | 14:22 |
| haleyb | mlavalle2: is 25port75d574 the interface in the VM? | 14:26 |
| haleyb | since the address is link-local it needs a scope, just odd it's not eth0 or something similar | 14:27 |
| opendevreview | Dmitrii Shcherbakov proposed openstack/neutron master: [ovn] Add end to end test for QosExtension https://review.opendev.org/c/openstack/neutron/+/877603 | 14:28 |
| opendevreview | Dmitrii Shcherbakov proposed openstack/neutron master: [ovn] Drop use of OVN_GW_PORT_EXT_ID_KEY https://review.opendev.org/c/openstack/neutron/+/877831 | 14:28 |
| opendevreview | Dmitrii Shcherbakov proposed openstack/neutron master: [ovn] Drop use of LR OVN_GW_NETWORK_EXT_ID_KEY https://review.opendev.org/c/openstack/neutron/+/877712 | 14:28 |
| opendevreview | Dmitrii Shcherbakov proposed openstack/neutron master: [ovn] OVNClient._get_router_ports: Drop unused parameter https://review.opendev.org/c/openstack/neutron/+/878527 | 14:28 |
| opendevreview | Dmitrii Shcherbakov proposed openstack/neutron master: Allow Multiple External Gateways https://review.opendev.org/c/openstack/neutron/+/873593 | 14:28 |
| opendevreview | Dmitrii Shcherbakov proposed openstack/neutron master: Add extra router attributes for ECMP and BFD https://review.opendev.org/c/openstack/neutron/+/874797 | 14:28 |
| haleyb | or maybe that is in the test, i could just look at the code :) | 14:28 |
| opendevreview | Dmitrii Shcherbakov proposed openstack/neutron master: [ovn] Implement support for external-gateway-multihoming extension https://review.opendev.org/c/openstack/neutron/+/874199 | 14:28 |
| opendevreview | Dmitrii Shcherbakov proposed openstack/neutron master: [ovn] Honor `enable_default_route_ecmp` attribute https://review.opendev.org/c/openstack/neutron/+/878531 | 14:28 |
| opendevreview | Dmitrii Shcherbakov proposed openstack/neutron master: [ovn] Allow L3 scheduler to be aware of current transaction https://review.opendev.org/c/openstack/neutron/+/874760 | 14:28 |
| opendevreview | Dmitrii Shcherbakov proposed openstack/neutron master: [ovn] Add helper for retrieving LR associated with LRP https://review.opendev.org/c/openstack/neutron/+/873698 | 14:28 |
| opendevreview | Dmitrii Shcherbakov 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:28 |
| opendevreview | Dmitrii Shcherbakov proposed openstack/neutron master: [ovn] Add support for enable_default_route_bfd attribute https://review.opendev.org/c/openstack/neutron/+/878543 | 14:28 |
| opendevreview | Dmitrii Shcherbakov proposed openstack/neutron master: Add a gw_ports fields to the Router object https://review.opendev.org/c/openstack/neutron/+/879462 | 14:28 |
| mlavalle2 | haleyb: port75d574 is the interface in the VM namespace. the 25 comes from a note that rubasov_ wrote some time ago https://review.opendev.org/c/openstack/nova/+/744492/2/doc/source/user/metadata.rst | 14:31 |
| mlavalle2 | and thanks for looking :-) | 14:31 |
| haleyb | hmm, that '25' looks like a typo, the scope ID is typically an interface name, unless I'm misunderstanding | 14:33 |
| mlavalle2 | haleyb: weel, rubasov_ has this: https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/750355/9/neutron_tempest_plugin/scenario/test_metadata.py#133 | 14:34 |
| mlavalle2 | and it actually works | 14:34 |
| mlavalle2 | haleyb: so it is in the current tempest code: https://opendev.org/openstack/neutron-tempest-plugin/src/branch/master/neutron_tempest_plugin/scenario/test_metadata.py#L143 | 14:35 |
| rubasov_ | yes, the scope id should be the interface name | 14:35 |
| rubasov_ | the url looked fine to me | 14:36 |
| mlavalle2 | rubasov_: with the '25', right? | 14:36 |
| haleyb | i hate to ask if that test is run with the conditionals there | 14:36 |
| rubasov_ | but I know have seen various curl variants in my life (the real one, another coming from busybox) that behaved differently with v6 LL addresses | 14:36 |
| rubasov_ | so a double check with your actual curl can't hurt | 14:37 |
| mlavalle2 | haleyb, haleyb: at leat in tempest it runs fine with the '25'. this is a fresh execution form last night https://591231bf28089b0a7015-86415e0a04f340ad7235be99b6135f85.ssl.cf5.rackcdn.com/858879/19/check/neutron-tempest-plugin-openvswitch/9e0d163/testr_results.html | 14:40 |
| haleyb | the curl man page has a 25 but doesn't explain it | 14:40 |
| mlavalle2 | neutron_tempest_plugin.scenario.test_metadata.MetadataTest.test_metadata_routed pass | 14:40 |
| haleyb | all i know is using the 25 with ping gives an error | 14:41 |
| haleyb | ping: fe80::1e3b:f3ff:fed6:8730%25eno1: Name or service not known | 14:41 |
| mlavalle2 | maybe I should step back a bit and just, as the next iteration, just ping in the namespace to see if I have basic connectivity | 14:42 |
| rubasov_ | I believe some curl variants require url-encoding the % sign | 14:42 |
| mlavalle2 | rubasov_: ah, I can also try that | 14:42 |
| rubasov_ | but do not remember from the top of my head which variants | 14:42 |
| haleyb | ah, so might be specific to curl. i tried curl with and without and both worked the same | 14:43 |
| rubasov_ | haleyb: I believe this was curl specific, ping does not take a url, there will be no url-encoding requirement there ever | 14:43 |
| haleyb | curl http://[fe80::1e3b:f3ff:fed6:8730%eno1] worked for me, that's just another local system, but different OSes have different versions i suppose | 14:45 |
| frickler | isn't that just urlencoded? %25 == "%" | 14:46 |
| mlavalle2 | haleyb: but I can see that you had to include the "%" in the url | 14:46 |
| mlavalle2 | so maybe that is my error. | 14:47 |
| haleyb | mlavalle2: yes, the % indicates what after is the scope id, for link-local you need | 14:47 |
| mlavalle2 | ok, I'll try that | 14:47 |
| haleyb | mlavalle2: if you have a VM running can probably try from the command line to verify | 14:48 |
| mlavalle2 | frickler: yes, you are right https://www.eso.org/~ndelmott/url_encode.html | 14:49 |
| haleyb | and i guess zone-id is what IPv6 uses, rfc 4007 shows <address>%<zone_id> | 14:51 |
| haleyb | `%' is a delimiter character to distinguish between <address> and | 14:51 |
| haleyb | <zone_id>. | 14:51 |
| haleyb | but we already knew that :) | 14:51 |
| rubasov_ | yes, when it comes to link local addresses we need to tell the kernel which interface to use | 14:53 |
| rubasov_ | and yes %25 is just url-encoded % sign | 14:53 |
| haleyb | right, either by name or index | 14:53 |
| mlavalle2 | haleyb, rubasov_: so it should look like: http://[fe80::a9fe:a9fe%portcd40b9] | 14:56 |
| mlavalle2 | right? | 14:56 |
| haleyb | +1 from me | 14:56 |
| rubasov_ | yes, that's one likely option | 14:56 |
| rubasov_ | but it's worth trying with %25 in place of the % sign too | 14:57 |
| mlavalle2 | rubasov_: ack, will try that | 14:58 |
| rubasov_ | the curl in my devtack vm I have at hand seems to work with both: curl 7.81.0 (x86_64-pc-linux-gnu) libcurl/7.81.0 OpenSSL/3.0.2 zlib/1.2.11 brotli/1.0.9 zstd/1.4.8 libidn2/2.3.2 libpsl/0.21.0 (+libidn2/2.3.2) libssh/0.9.6/openssl/zlib nghttp2/1.43.0 librtmp/2.3 OpenLDAP/2.5.13 | 15:00 |
| opendevreview | Lajos Katona proposed openstack/networking-bagpipe master: [sqlalchemy-20] Remove subtransactions=True https://review.opendev.org/c/openstack/networking-bagpipe/+/879463 | 15:01 |
| haleyb | another alternative would be to use --interface, curl --interface eth0 http://[fe80::a9fe:a9fe] | 15:03 |
| rubasov_ | the manpage wants the url-encoding: "Provide the IPv6 zone index in the URL with an escaped percentage sign and the interface name." | 15:07 |
| mlavalle2 | rubasov_: thanks! | 15:09 |
| haleyb | rubasov_: both seem to have the same outcome - using --interface sets the sin6_scope_id in the connect() call, where %zone-id uses SO_BINDTODEVICE to force the interface | 15:11 |
| haleyb | that's what strace told me :) | 15:12 |
| rubasov_ | mlavalle2: maybe it's worth putting a big sleep() in the middle of the test, and while it sleeps, checking manually if the url is formatted to what the curl in your environment expects and whether haproxy is starting and listening properly with the config it got | 15:17 |
| mlavalle2 | rubasov_: yeap, good idea | 15:22 |
| rubasov_ | mlavalle2: I'm about to leave the office for today, but let me know if I can help later | 15:28 |
| mlavalle2 | rubasov_: thanks for your help. Have a nice evening! | 15:36 |
| * haleyb is here until ~5ET | 15:37 | |
| mlavalle2 | haleyb: in an internal meeting for now. I'll do the testing a little later. Thanks! | 15:38 |
| ralonsoh | bcafarel, hi! can you check https://review.opendev.org/c/openstack/neutron-lib/+/874397 and https://review.opendev.org/c/openstack/neutron-lib/+/879441? | 16:02 |
| ralonsoh | thanks in advance | 16:02 |
| bcafarel | ralonsoh: sure, I remember talking about these this morning with slaweq | 16:08 |
| opendevreview | Frode Nordahl proposed openstack/neutron master: [ovn] Add end to end test for QosExtension https://review.opendev.org/c/openstack/neutron/+/877603 | 16:18 |
| opendevreview | Merged openstack/neutron master: Mark "test_multiple_agents_for_network" as unstable https://review.opendev.org/c/openstack/neutron/+/879435 | 16:19 |
| *** iurygregory_ is now known as iurygregory | 16:24 | |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [DNM] - [OVN] Add physical network information to LS https://review.opendev.org/c/openstack/neutron/+/879496 | 17:07 |
| opendevreview | Merged openstack/neutron-lib stable/wallaby: Skip test_qos_dscp_create_and_update API test in stable/wallaby https://review.opendev.org/c/openstack/neutron-lib/+/879441 | 17:45 |
| opendevreview | Merged openstack/neutron-lib stable/wallaby: Add neutron-lib to required projects in neutron-tempest-plugin-api job https://review.opendev.org/c/openstack/neutron-lib/+/879424 | 17:45 |
| opendevreview | Merged openstack/neutron-lib stable/wallaby: Don't check context.system_scope to check project scope https://review.opendev.org/c/openstack/neutron-lib/+/874397 | 17:45 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!