| opendevreview | liuxie proposed openstack/ovsdbapp master: Support add/remove CIDR type address in Address_Set cmds https://review.opendev.org/c/openstack/ovsdbapp/+/905044 | 03:45 |
|---|---|---|
| opendevreview | liuxie proposed openstack/ovsdbapp master: Support add/remove CIDR type address in Address_Set cmds https://review.opendev.org/c/openstack/ovsdbapp/+/905044 | 06:14 |
| ralonsoh | ykarel, hello! If you have 1 min: https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/903930 | 07:55 |
| ralonsoh | thanks! | 07:55 |
| ykarel | ralonsoh, looking | 07:55 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Forbid the subnet gateway IP deletion if a router interface is attached https://review.opendev.org/c/openstack/neutron/+/904713 | 08:00 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/2023.1: Mark "test_port_creation_and_deletion" as unstable https://review.opendev.org/c/openstack/neutron/+/905072 | 08:28 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/2023.1: [OVN] Retrieve the OVN agent extensions correctly https://review.opendev.org/c/openstack/neutron/+/903933 | 08:29 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/2023.1: [OVN] Add the bridge name and datapath type to the port VIF details https://review.opendev.org/c/openstack/neutron/+/904254 | 08:33 |
| opendevreview | Bodo Petermann proposed openstack/neutron-lib master: vpnaas: add support for AES CTR https://review.opendev.org/c/openstack/neutron-lib/+/903971 | 09:13 |
| opendevreview | liuxie proposed openstack/ovsdbapp master: Support add/remove CIDR type address in Address_Set cmds https://review.opendev.org/c/openstack/ovsdbapp/+/905044 | 09:52 |
| opendevreview | Merged openstack/neutron-tempest-plugin master: Remove duplicate setup in NetworksIpAvailabilityIPv6Test class https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/903930 | 10:32 |
| opendevreview | Thomas Goirand proposed openstack/networking-sfc master: Python 3.12: assertDictContainsSubset is removed https://review.opendev.org/c/openstack/networking-sfc/+/904208 | 10:48 |
| opendevreview | Thomas Goirand proposed openstack/neutron-lib master: Python-3.12: do not use datetime.datetime.utcnow() https://review.opendev.org/c/openstack/neutron-lib/+/903841 | 10:50 |
| seba | ralonsoh, did you want to discuss the gateway fip allocation issue today? I think you said Tuesday / Neutron Team meeting, but it's so far only on the agenda for the Neutron drivers meeting. | 10:57 |
| hamidlotfi_ | Hi there, | 10:57 |
| hamidlotfi_ | What does the value of the revision_number field in the port information section indicate? | 10:57 |
| hamidlotfi_ | It went from 1 to 500 within half an hour and also I have this message on Neutron server: | 10:57 |
| hamidlotfi_ | Successfully bumped revision number for resource 46436042-db65-4c6b-9f15-4b2f8e9ee942 (type: ports) to 500 | 10:57 |
| ralonsoh | seba, please add it to the on-demand section | 10:57 |
| seba | done | 11:00 |
| ralonsoh | hamidlotfi_, that's the number of times this port has been updated | 11:16 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Remove the "new_facade" parameter that is no longer needed https://review.opendev.org/c/openstack/neutron/+/905105 | 11:30 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Update exception conditions in ``_process_create_port`` https://review.opendev.org/c/openstack/neutron/+/905106 | 11:41 |
| opendevreview | Merged openstack/neutron stable/wallaby: Reset device namespace when adding to the namespace fails https://review.opendev.org/c/openstack/neutron/+/904827 | 11:42 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Add a start method to the OVN agent extensions https://review.opendev.org/c/openstack/neutron/+/903977 | 12:03 |
| opendevreview | Merged openstack/neutron master: Add a "port" child table "porthardwareoffloadtype" https://review.opendev.org/c/openstack/neutron/+/882832 | 13:21 |
| houtknots | Hi all, would it be possible for someone to review 900212: Change exception messages to error log messages for DNS integration. | https://review.opendev.org/c/openstack/neutron/+/900212 . It has been a while since there was any activity. | 14:00 |
| haleyb | #startmeeting networking | 14:00 |
| opendevmeet | Meeting started Tue Jan 9 14:00:55 2024 UTC and is due to finish in 60 minutes. The chair is haleyb. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:00 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:00 |
| opendevmeet | The meeting name has been set to 'networking' | 14:00 |
| haleyb | hi everyone | 14:01 |
| haleyb | Ping list: bcafarel, elvira, frickler, mlavalle, mtomaska, obondarev, slaweq, tobias-urdin, ykarel, lajoskatona, jlibosva, averdagu, amotoki | 14:01 |
| obondarev | hi | 14:01 |
| slaweq | hi | 14:01 |
| seba | hi | 14:01 |
| ralonsoh | hello | 14:01 |
| ykarel | o/ | 14:01 |
| elvira | o/ | 14:01 |
| haleyb | ralonsoh: thanks for leading meeting while i was away, is hard to get back to work :( | 14:02 |
| mtomaska1 | o/ | 14:02 |
| haleyb | #topic announcements | 14:03 |
| lajoskatona1 | o/ | 14:03 |
| haleyb | #link https://releases.openstack.org/caracal/schedule.html | 14:03 |
| haleyb | this week is Caracal-2 milestone | 14:04 |
| bcafarel | late o/ | 14:04 |
| haleyb | i saw releases were proposed yesterday for neutron-lib and ovsdbapp | 14:04 |
| haleyb | so we should keep merging features over the next month we want in the release | 14:06 |
| haleyb | C-3 is week of Feb 26 | 14:06 |
| lajoskatona1 | https://review.opendev.org/c/openstack/releases/+/904924 & https://review.opendev.org/c/openstack/releases/+/904943 are the release patches | 14:06 |
| haleyb | lajoskatona1: thanks | 14:07 |
| haleyb | any other announcements? i'm still working through my backlog and might have missed something | 14:07 |
| ralonsoh | ussuri releases patch | 14:08 |
| ralonsoh | https://review.opendev.org/c/openstack/releases/+/903294 | 14:08 |
| ralonsoh | this is still pending | 14:08 |
| lajoskatona1 | +1, I check this one also (also just back to work) | 14:09 |
| haleyb | ralonsoh: ack, thanks, will take a look after meeting. those patches without FT can be ignored, we are working on a downstream solution for them | 14:09 |
| haleyb | ok, lets move on | 14:11 |
| haleyb | #topic bugs | 14:11 |
| haleyb | elvira: https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/EVBKX5NT7WMJ6II7QOVXKFU5APEHBKY2/ | 14:11 |
| haleyb | from elvira^^ | 14:11 |
| elvira | yes! There were only two bugs, and one of them is an RFE | 14:12 |
| haleyb | ack, thanks, i will look at the RFE and add to drivers meeting | 14:13 |
| haleyb | the other is unassigned | 14:13 |
| haleyb | #link https://bugs.launchpad.net/neutron/+bug/2048198 | 14:13 |
| haleyb | neutron-lib role enforcer is writing warning messages | 14:13 |
| ralonsoh | yeah this one was reported here in IRC | 14:13 |
| ralonsoh | and I open this bug | 14:13 |
| ralonsoh | this is not an "issue", this is not affecting the neutron enforcer | 14:14 |
| ralonsoh | we should have a way to filter the rules to be loaded in the n-lib role enforcer | 14:14 |
| ralonsoh | or at least to be able to no to check the rules loaded | 14:14 |
| ralonsoh | but that requires a oslo.policy change | 14:15 |
| ralonsoh | in any case, this is not an error | 14:15 |
| ralonsoh | thanks | 14:15 |
| haleyb | ralonsoh: ok, so from reading it is harmful but not impacting the neutron enforcer | 14:16 |
| ralonsoh | no no, not harmful at all | 14:16 |
| ralonsoh | just cosmetic | 14:16 |
| haleyb | ah, ok, might just be me misunderstanding the wording | 14:18 |
| elvira | ralonsoh: do you think this fits as low hanging fruit? | 14:18 |
| ralonsoh | maybe but I don't think there is a trivial fix in neutron or oslo.policy | 14:18 |
| elvira | oh ok :( | 14:19 |
| haleyb | any other bugs to discuss? | 14:20 |
| haleyb | this weeks deputy is slaweq, next week is me | 14:20 |
| haleyb | moving on | 14:21 |
| haleyb | #topic specs | 14:21 |
| haleyb | #link https://review.opendev.org/q/project:openstack%252Fneutron-specs+status:open | 14:21 |
| haleyb | so we still have 3 un-merged specs | 14:21 |
| slaweq | I already started bug deputy this week :) | 14:22 |
| * slaweq is on different meeting in same time, sorry for delayed replies | 14:22 | |
| haleyb | slaweq: ack, thanks | 14:22 |
| ralonsoh | I swear I'll spend time this week reviweing the specs | 14:22 |
| haleyb | i'm worried there won't be time to get the actual work done, but maybe we can just focus on the OVN-IC one | 14:24 |
| ralonsoh | sure | 14:24 |
| ralonsoh | --> https://review.opendev.org/c/openstack/neutron-specs/+/891204 | 14:24 |
| haleyb | right | 14:25 |
| haleyb | ok, moving on | 14:26 |
| haleyb | #topic community_goals | 14:26 |
| haleyb | lajoskatona1: with you just getting back as well assuming no changes in horizon status? | 14:27 |
| lajoskatona1 | I have to check my horizon patch is there is any comment, I just try to kill my email and chat backlog :-) | 14:27 |
| lajoskatona1 | https://review.opendev.org/c/openstack/horizon/+/891205 | 14:28 |
| haleyb | lajoskatona1: ack, thanks! | 14:28 |
| lajoskatona1 | As I remember some integration tests failed and finally I realized that it is most probably not gate issue (quite hard to find out from logs and reproduce locally for me at least) | 14:29 |
| lajoskatona1 | that's it for this topic | 14:29 |
| haleyb | ok, glad it is progressing | 14:29 |
| haleyb | #topic on_demand | 14:30 |
| haleyb | i think the two items might be from last week, i did not have time to scrub this morning | 14:31 |
| seba | at least for the fip gateway ip allocation issue, that was put in by me today | 14:31 |
| haleyb | one was a change to python-neutronclient, which already has comments pointing in the right direction | 14:31 |
| ralonsoh | haleyb, so you agree on this, right? | 14:32 |
| ralonsoh | we should not add any new feature to neutronclient | 14:32 |
| ralonsoh | even a new binding | 14:32 |
| haleyb | ralonsoh: absolutely, as lajoskatona1 mentioned in the review it can go in the sdk code | 14:33 |
| ralonsoh | perfect then (I added this topic) | 14:33 |
| haleyb | seba: is there enough info in the comments for you to move forward? | 14:33 |
| andreykurilin | Hi folks! So should neutronclient README be updated that it does not accept any new features? | 14:33 |
| haleyb | doh, wrong comment | 14:33 |
| ralonsoh | andreykurilin, yes, I'll push a patch with this information | 14:34 |
| andreykurilin | thank you! | 14:34 |
| lajoskatona1 | +1, thanks ralonsoh | 14:34 |
| haleyb | ralonsoh: thanks | 14:34 |
| haleyb | second item was from seba | 14:35 |
| haleyb | #link https://review.opendev.org/c/openstack/neutron/+/904783 | 14:35 |
| seba | yeah, so we basically need to decide if this bug is something that is relevant for upstream / other openstack installations or just in my environment | 14:36 |
| ralonsoh | quick summary: now it is possible (for an admin) to assign a FIP with the GW IP | 14:36 |
| ralonsoh | with the default policies (define the IP is allowed only to the admin) | 14:36 |
| ralonsoh | this patch wants to prevent that | 14:37 |
| seba | and it's also a changeable policy - in our OpenStack setup we allow the user to specify the ip for the fip they want to allocate | 14:37 |
| haleyb | ralonsoh, right, we allow admin to do destructive things | 14:38 |
| ralonsoh | IMO: as commented, this is (by default) an admin privilege and should be allowed if we want to have a VM as a GW proxy or firewall or something else | 14:38 |
| seba | as I never would want an external gateway to be on the same ip as a fip (I'd rather unset the gateway of the external subnet) I'd go for completely forbidding this. | 14:38 |
| ralonsoh | this could be an extension, no loaded by default of course | 14:39 |
| ralonsoh | an API extension | 14:39 |
| seba | or could be hidden behind a feature switch | 14:40 |
| ralonsoh | no, we don't allow config driven API | 14:40 |
| seba | okay | 14:40 |
| lajoskatona1 | +1 | 14:40 |
| haleyb | seba: i would agree with ralonsoh that we don't want to change the default behavior, but it could be an extension | 14:40 |
| ralonsoh | any API behaviour is ruled by the API definitions and extensions | 14:40 |
| ralonsoh | so, as commented, this could be a shim extension (empty extension) | 14:41 |
| ralonsoh | and if loaded, then the API will react in the way you want | 14:41 |
| seba | do you have an example for an api extension that is just loaded to add extra checks like this one? | 14:41 |
| ralonsoh | I'll check that after the meeting | 14:41 |
| seba | thanks | 14:42 |
| seba | so, would such an extension be something that you'd be interested in having living in the upstream neutron codebase? | 14:42 |
| ralonsoh | yes | 14:43 |
| haleyb | we'll probably spend more time bike-shedding on the name :) | 14:43 |
| seba | ha ;) | 14:43 |
| seba | okay, sounds good | 14:43 |
| haleyb | any other topics to discuss this week? | 14:43 |
| haleyb | oh, is there a CI meeting today? | 14:44 |
| ralonsoh | yes in 15 mins | 14:44 |
| lajoskatona1 | irc or video? | 14:45 |
| ykarel | video | 14:45 |
| lajoskatona1 | thanks | 14:45 |
| haleyb | thanks for attending everyone, will give you :15 back | 14:45 |
| haleyb | #endmeeting | 14:45 |
| opendevmeet | Meeting ended Tue Jan 9 14:45:48 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:45 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/networking/2024/networking.2024-01-09-14.00.html | 14:45 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/networking/2024/networking.2024-01-09-14.00.txt | 14:45 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/networking/2024/networking.2024-01-09-14.00.log.html | 14:45 |
| ralonsoh | bye | 14:45 |
| lajoskatona1 | o/ | 14:45 |
| ralonsoh | seba, https://review.opendev.org/q/topic:%22bug/1796824%22 | 14:46 |
| mnederlof | ralonsoh, since you´re tagged on my review, can you shed some light on what needs to be done to make some progression in this change? https://review.opendev.org/c/openstack/neutron/+/896883 | 14:48 |
| mnederlof | if there is anything i can do from my side i´d love to know :) | 14:49 |
| ralonsoh | mnederlof, let me review it again today | 14:49 |
| mnederlof | appreciated, thanks :) | 14:49 |
| seba | ralonsoh, thanks. Do I understand this correctly that this would need to be activated in neutron via the service_plugins parameter, then? | 14:50 |
| ralonsoh | seba, no, this is a L3 extension, not a service plugin | 15:00 |
| ykarel | #startmeeting neutron_ci | 15:01 |
| opendevmeet | Meeting started Tue Jan 9 15:01:57 2024 UTC and is due to finish in 60 minutes. The chair is ykarel. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:01 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:01 |
| opendevmeet | The meeting name has been set to 'neutron_ci' | 15:01 |
| ralonsoh | hi | 15:02 |
| ralonsoh | link? | 15:02 |
| ykarel | ping bcafarel, lajoskatona, mlavalle, mtomaska, ralonsoh, ykarel, jlibosva, elvira | 15:02 |
| ykarel | This will be video meeting today: https://meetpad.opendev.org/neutron-ci-meetings | 15:02 |
| slaweq | o/ | 15:02 |
| lajoskatona1 | o/ | 15:02 |
| bcafarel | ah video, I thought it was IRC :( | 15:03 |
| bcafarel | ykarel: on another video meeting - stable branches are mostly OK just a few failures (but different jobs), the only suspect one is fullstack but it may be related to patch itself https://review.opendev.org/c/openstack/neutron/+/904832 | 15:04 |
| ykarel | bcafarel, thx | 15:04 |
| ykarel | #topic Actions from previous meetings | 15:04 |
| ykarel | ralonsoh to check and propose fix for handling ip route commands handling | 15:04 |
| ykarel | #action ralonsoh to push patch for pyroute command handling | 15:05 |
| ykarel | ralonsoh to send patch to drop tripleo jobs from wallaby | 15:06 |
| ykarel | #link https://review.opendev.org/c/openstack/neutron/+/904498 | 15:06 |
| ykarel | #topic Stable branches | 15:06 |
| ykarel | all good except some fullstack failures | 15:07 |
| ykarel | #topic Stadium projects | 15:07 |
| ykarel | neutron-vpnaas job timed out https://zuul.openstack.org/build/56f3aab11f7d4f98ab8f80fe73145cd5 | 15:07 |
| ykarel | #topic Rechecks | 15:09 |
| ykarel | #topic fullstack/functional | 15:09 |
| ykarel | neutron.tests.common.machine_fixtures.FakeMachineException: No ICMP reply obtained from IP address | 15:09 |
| ykarel | seen all in same xena patch https://review.opendev.org/c/openstack/neutron/+/904832, need to check if it's patch specific as the job was quite stable there, some failures also seen in other backport of the patch | 15:10 |
| ykarel | #action ralonsoh to check if failures are related to the backport | 15:13 |
| ykarel | #topic Tempest/Scenario | 15:13 |
| ykarel | test_volume_assisted_snapshot_create_delete | 15:13 |
| ykarel | https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_a6e/903796/1/gate/tempest-integrated-networking/a6e37d8/testr_results.html | 15:13 |
| ykarel | #action slaweq to check with cirros guy to see if this is some knonw issue | 15:15 |
| ykarel | #topic Periodic | 15:15 |
| ykarel | all green | 15:16 |
| ykarel | #topic Grafana | 15:16 |
| ykarel | https://grafana.opendev.org/d/f913631585/neutron-failure-rate | 15:16 |
| ykarel | #topic On Demand | 15:19 |
| ykarel | #endmeeting | 15:19 |
| opendevmeet | Meeting ended Tue Jan 9 15:19:42 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:19 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/neutron_ci/2024/neutron_ci.2024-01-09-15.01.html | 15:19 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/neutron_ci/2024/neutron_ci.2024-01-09-15.01.txt | 15:19 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/neutron_ci/2024/neutron_ci.2024-01-09-15.01.log.html | 15:19 |
| opendevreview | Michal Nasiadka proposed openstack/neutron master: haproxy: Add support for configuring syslog https://review.opendev.org/c/openstack/neutron/+/884407 | 15:21 |
| opendevreview | Michal Nasiadka proposed openstack/neutron master: haproxy: Add support for configuring syslog https://review.opendev.org/c/openstack/neutron/+/884407 | 15:22 |
| seba | ralonsoh, how would you turn that behavior on and off? like for the example you have given me, how would I turn on/off l3-port-ip-change-not-allowed? | 15:25 |
| opendevreview | Bence Romsics proposed openstack/neutron master: WIP Set trunk parent port as access port in ovs to avoid loop https://review.opendev.org/c/openstack/neutron/+/905125 | 15:26 |
| opendevreview | Michal Nasiadka proposed openstack/neutron master: haproxy: Add support for configuring syslog https://review.opendev.org/c/openstack/neutron/+/884407 | 15:26 |
| ralonsoh | seba, by setting this extension in the configuration, in the l3 extension list | 15:29 |
| ralonsoh | if the extension is defined, then the new behaviour applies | 15:29 |
| seba | and that is a configuration option? I'm only aware of the supported_extension_aliases list that the l3 driver has | 15:31 |
| opendevreview | Michal Nasiadka proposed openstack/neutron master: haproxy: Add support for configuring syslog https://review.opendev.org/c/openstack/neutron/+/884407 | 15:31 |
| ralonsoh | seba, not this one. Yours should be checked during the L3RouterPlugin load | 15:36 |
| ralonsoh | but I don't know where this extension should be defined... | 15:37 |
| ralonsoh | we have the l3 agent extensions | 15:37 |
| ralonsoh | but this this one belongs to the API | 15:37 |
| ralonsoh | yeah, I don't see better place than neutron.agent.l3.extensions | 15:39 |
| seba | okay, thanks | 15:43 |
| opendevreview | Michal Nasiadka proposed openstack/neutron master: haproxy: Add support for configuring syslog https://review.opendev.org/c/openstack/neutron/+/884407 | 15:46 |
| opendevreview | Bodo Petermann proposed openstack/neutron-lib master: vpnaas: add support for AES CTR https://review.opendev.org/c/openstack/neutron-lib/+/903971 | 15:58 |
| mnederlof | thank you for the review ralonsoh, i´ve replied your concerns | 16:05 |
| ralonsoh | mnederlof, updated | 16:14 |
| mnederlof | thanks, i´ll start working on that tomorrow :) | 16:25 |
| opendevreview | Lajos Katona proposed openstack/neutron-lib master: Tap mirror API def and API ref https://review.opendev.org/c/openstack/neutron-lib/+/885354 | 17:07 |
| opendevreview | Merged openstack/neutron stable/zed: Add sleep before checking if ovs port is in the namespace https://review.opendev.org/c/openstack/neutron/+/904830 | 18:05 |
| opendevreview | Merged openstack/neutron stable/yoga: Add sleep before checking if ovs port is in the namespace https://review.opendev.org/c/openstack/neutron/+/904831 | 18:05 |
| opendevreview | Merged openstack/neutron-tempest-plugin master: Add VNaaS OVN job to the project https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/889294 | 18:40 |
| opendevreview | Merged openstack/neutron stable/2023.1: Mark "test_port_creation_and_deletion" as unstable https://review.opendev.org/c/openstack/neutron/+/905072 | 18:40 |
| opendevreview | Jakub Libosvar proposed openstack/ovn-bgp-agent master: devstack: Install vrf kernel module if needed https://review.opendev.org/c/openstack/ovn-bgp-agent/+/905144 | 19:38 |
| opendevreview | Brian Haley proposed openstack/neutron master: Disallow subnet cidr of :: without PD https://review.opendev.org/c/openstack/neutron/+/905148 | 21:05 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!