| opendevreview | Merged openstack/neutron-specs master: BGPaaS enhancements https://review.opendev.org/c/openstack/neutron-specs/+/783791 | 00:59 |
|---|---|---|
| opendevreview | XiaoYu Zhu proposed openstack/neutron master: L3 router support ECMP https://review.opendev.org/c/openstack/neutron/+/743661 | 02:10 |
| *** rpittau|afk is now known as rpittau | 07:30 | |
| opendevreview | Lajos Katona proposed openstack/neutron master: Privsep with timout for get_link_devices call https://review.opendev.org/c/openstack/neutron/+/794994 | 07:48 |
| opendevreview | wushiming proposed openstack/networking-sfc master: Changed minversion in tox to 3.18.0 https://review.opendev.org/c/openstack/networking-sfc/+/799596 | 08:04 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Remove SG RPC "use_enhanced_rpc" check. https://review.opendev.org/c/openstack/neutron/+/799446 | 08:10 |
| opendevreview | wushiming proposed openstack/neutron-tempest-plugin master: Changed minversion in tox to 3.18.0 https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/799599 | 08:11 |
| opendevreview | Merged openstack/neutron master: Add fake_project_id middleware for noauth https://review.opendev.org/c/openstack/neutron/+/799162 | 08:21 |
| opendevreview | Merged openstack/neutron master: Change fullstack dhclient lease file to tmp folder https://review.opendev.org/c/openstack/neutron/+/799438 | 08:22 |
| opendevreview | Merged openstack/neutron stable/ussuri: [OVN] Do not fail when processing SG rule deletion https://review.opendev.org/c/openstack/neutron/+/799372 | 08:22 |
| ralonsoh | hi folks | 08:26 |
| ralonsoh | there were pending questions in https://review.opendev.org/c/openstack/neutron/+/799438/1 | 08:26 |
| ralonsoh | please, next time wait for the reply | 08:27 |
| slaweq | ralonsoh: what question was there? | 08:28 |
| ralonsoh | https://review.opendev.org/c/openstack/neutron/+/799438/1/neutron/tests/fullstack/resources/machine.py | 08:28 |
| ralonsoh | I'm replyuing right now | 08:29 |
| ralonsoh | anyway, the patch is merged | 08:29 |
| slaweq | ralonsoh: ahh, sorry, I somehow missed it | 08:29 |
| slaweq | please next time give -1 in such case - just to avoid such problems :) | 08:29 |
| slaweq | if there would be -1 from You, I would for sure not approve it | 08:30 |
| slaweq | sorry, my bad | 08:30 |
| lajoskatona | ralonsoh: thanks for headsup | 08:43 |
| opendevreview | Merged openstack/neutron-lib master: Introduce rbac-bgpvpn api extension https://review.opendev.org/c/openstack/neutron-lib/+/795423 | 09:10 |
| opendevreview | Lajos Katona proposed openstack/neutron master: Privsep with timeout for get_link_devices call https://review.opendev.org/c/openstack/neutron/+/794994 | 09:11 |
| opendevreview | Lajos Katona proposed openstack/neutron master: Revert "Use 2 dhcp agents in TestLegacyL3Agent" https://review.opendev.org/c/openstack/neutron/+/799579 | 09:12 |
| opendevreview | Lajos Katona proposed openstack/neutron master: Privsep with timeout for get_link_devices call https://review.opendev.org/c/openstack/neutron/+/794994 | 09:12 |
| sean-k-mooney | ralonsoh: lucasagomes any suggestions on how i fix os-vifs ci following the ovn default change https://review.opendev.org/c/openstack/os-vif/+/798038/7/.zuul.yaml | 09:38 |
| ralonsoh | sean-k-mooney, let me check, maybe using neutron CI job definitions | 09:39 |
| ralonsoh | let me check | 09:39 |
| ralonsoh | sean-k-mooney, OVS with iptables is working | 09:40 |
| sean-k-mooney | yes mostly although some tests failed | 09:40 |
| sean-k-mooney | its linuxbridge that is really th e issue | 09:41 |
| sean-k-mooney | testtools.matchers._impl.MismatchError: 'dvr' not in [ | 09:41 |
| sean-k-mooney | that is what broke ovs-iptables | 09:41 |
| sean-k-mooney | i turned off dvr | 09:42 |
| sean-k-mooney | that shoudl be a simple fix | 09:42 |
| sean-k-mooney | oh i think i just have a typo | 09:43 |
| ralonsoh | i think so | 09:43 |
| sean-k-mooney | https://zuul.opendev.org/t/openstack/build/9e0e009586a44631bcf7b75e58402591/log/controller/logs/devstacklog.txt#17381-17384 | 09:44 |
| ralonsoh | yes, I was reading that | 09:44 |
| sean-k-mooney | OH | 09:45 |
| sean-k-mooney | it the indentation | 09:45 |
| sean-k-mooney | https://review.opendev.org/c/openstack/os-vif/+/798038/7/.zuul.yaml#161 | 09:45 |
| sean-k-mooney | im not currently using the neutron plugin so i dont need that section currently | 09:45 |
| sean-k-mooney | but i was considring enabling it | 09:45 |
| sean-k-mooney | ill remove it for now and i can add that in a seperate patch | 09:46 |
| ralonsoh | sean-k-mooney, but I think the indentation is correct | 09:46 |
| ralonsoh | aligned to "/$NEUTRON_CORE_PLUGIN_CONF" | 09:46 |
| sean-k-mooney | test-config should be aligned with post-config | 09:46 |
| sean-k-mooney | test-config is a phase in devstack like post-config or install | 09:47 |
| ralonsoh | that's right | 09:47 |
| ralonsoh | (checking neutron tempest plugin now) | 09:47 |
| sean-k-mooney | i have the alignment right in the iptable job https://review.opendev.org/c/openstack/os-vif/+/798038/7/.zuul.yaml#87 | 09:48 |
| opendevreview | Oleg Bondarev proposed openstack/neutron-specs master: [WIP] Add Node-Local Virtual IP Spec https://review.opendev.org/c/openstack/neutron-specs/+/797798 | 09:48 |
| sean-k-mooney | i just need to tell tempest i disabled dvr or i could renable it its singel node so its basically the same as normal router anyway | 09:48 |
| ralonsoh | exactly | 09:49 |
| ralonsoh | sean-k-mooney, in Neutron, single node tests we use Q_DVR_MODE: dvr_snat | 09:50 |
| ralonsoh | (when testing dvr) | 09:51 |
| sean-k-mooney | sure i can set that | 09:51 |
| sean-k-mooney | really im just trying to test the interface pluging in os-vif so its not really relevent but that shoudl get it to pass again | 09:52 |
| sean-k-mooney | i can optimise the jobs later but they are pretty fast anyway | 09:52 |
| sean-k-mooney | ralonsoh: i was using these as a reference https://opendev.org/openstack/neutron-tempest-plugin/src/branch/master/zuul.d/master_jobs.yaml#L202-L352 | 09:53 |
| ralonsoh | yes, that's a good reference | 09:54 |
| sean-k-mooney | there are nolonger any in tree iptables or linux bridge jobs right just the ones that use the neutron-tempest-plugin | 09:54 |
| sean-k-mooney | by in tree i meen in neutron | 09:55 |
| sean-k-mooney | oh there are | 09:55 |
| sean-k-mooney | https://github.com/openstack/neutron/blob/master/zuul.d/tempest-singlenode.yaml#L49-L83 | 09:55 |
| ralonsoh | right | 09:56 |
| opendevreview | Manu B proposed openstack/neutron-specs master: BFD for BGP Dynamic Routing https://review.opendev.org/c/openstack/neutron-specs/+/785349 | 10:05 |
| opendevreview | sean mooney proposed openstack/os-vif master: update os-vif ci to account for devstack default changes https://review.opendev.org/c/openstack/os-vif/+/798038 | 10:14 |
| opendevreview | sean mooney proposed openstack/os-vif master: add configurable per port bridges https://review.opendev.org/c/openstack/os-vif/+/798055 | 10:14 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: Make configure_for_func_testing compatible with e.g. Centos https://review.opendev.org/c/openstack/neutron/+/799625 | 10:15 |
| ralonsoh | sean-k-mooney, btw, just a heads up: qos works out of the box | 10:17 |
| ralonsoh | with your os-vif patch | 10:17 |
| ralonsoh | cool! I'm updating the spec right now | 10:17 |
| sean-k-mooney | ralonsoh: it does? in that case do we need the spec at all? | 10:18 |
| sean-k-mooney | i tought the spec was mainly for the qos change | 10:18 |
| sean-k-mooney | oh | 10:18 |
| sean-k-mooney | you are also chaning ml2/ovn | 10:18 |
| ralonsoh | sean-k-mooney, yes, at least to provide this info to Neutron community | 10:19 |
| sean-k-mooney | to actully send vif plugged events when the ports are wired up | 10:19 |
| ralonsoh | of course, this is up to os-vif community to merge this patch | 10:19 |
| ralonsoh | but yes, we need to change that part | 10:19 |
| sean-k-mooney | ok so in your spec can you provide a way for nova to know that you will send "plugtime" events | 10:20 |
| ralonsoh | sean-k-mooney, but we do now | 10:20 |
| sean-k-mooney | e.g. someing in the binding details that will indicate that nova can safely wait | 10:20 |
| ralonsoh | this is why we implemented this | 10:20 |
| ralonsoh | (one sec) | 10:20 |
| sean-k-mooney | ralonsoh: no we removed the event stuff from your train? spec i think | 10:21 |
| ralonsoh | sean-k-mooney, https://github.com/openstack/neutron/blob/97618b08769636ec07b994b26af5d73bae4e5a6c/neutron/plugins/ml2/drivers/ovn/mech_driver/mech_driver.py#L796-L815 | 10:21 |
| sean-k-mooney | we just no the ml2 driver | 10:21 |
| ralonsoh | in OVN, we send the plugged event | 10:21 |
| ralonsoh | of course, when "we can" | 10:21 |
| ralonsoh | it doesn't mean this is the best place | 10:21 |
| ralonsoh | but at least is something | 10:21 |
| sean-k-mooney | ralonsoh this is not a plugtime event | 10:21 |
| sean-k-mooney | its a bind time event right | 10:22 |
| ralonsoh | I know | 10:22 |
| ralonsoh | because that'ss not possible | 10:22 |
| ralonsoh | we can't send a plug event without the port | 10:22 |
| sean-k-mooney | right so then im confused | 10:22 |
| ralonsoh | but where is the concept of bind or plug | 10:22 |
| sean-k-mooney | in nova | 10:22 |
| ralonsoh | from Neutron we only send plugged events | 10:22 |
| ralonsoh | ah ok | 10:22 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: WIP/DNM - Add FIPS enabled jobs https://review.opendev.org/c/openstack/neutron/+/797537 | 10:22 |
| sean-k-mooney | we have special code to deal with the difference between ml2/ovs and ml2/ovn | 10:22 |
| ralonsoh | so what do you need from Neutron? | 10:22 |
| sean-k-mooney | to know if we should wait or not | 10:23 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: WIP/DNM - Add FIPS enabled jobs https://review.opendev.org/c/openstack/neutron/+/797537 | 10:23 |
| ralonsoh | ah ok | 10:23 |
| sean-k-mooney | right now we use hybrid plug as an indicator | 10:23 |
| ralonsoh | I think you can use the os-vif flag | 10:23 |
| sean-k-mooney | if vif_type=ovs and hybrid_plug=true we wait in pre live migrate | 10:23 |
| ralonsoh | this is for OVN | 10:23 |
| ralonsoh | at least we are implementing that for OVN | 10:23 |
| ralonsoh | of course, that implies the code in Neutron is merged | 10:24 |
| ralonsoh | but we can add a reno in os-vif | 10:24 |
| sean-k-mooney | ralonsoh: we cant use os-vif flags for this it would have to be a nova config option and we did not wnat to add ml2 driver specific config option to nova | 10:24 |
| ralonsoh | saying that to use this os-vif flag, you need this neutron version | 10:24 |
| sean-k-mooney | we are talking about different things | 10:24 |
| sean-k-mooney | if i understand in your spec there were really only two changes | 10:25 |
| ralonsoh | we are talking about informing nova about the driver and when are we sending the event | 10:25 |
| sean-k-mooney | 1 descibing the os-vif behavior and 2 sending the event at plug time correct | 10:25 |
| ralonsoh | in Neutron only one so far | 10:25 |
| ralonsoh | yes | 10:25 |
| ralonsoh | so how do you want Neutron to inform Nova? | 10:25 |
| ralonsoh | a temporary extension? | 10:26 |
| ralonsoh | just fro X and Y? | 10:26 |
| sean-k-mooney | a tempoary extenion could work or put something in the binding details | 10:26 |
| ralonsoh | that could work, adding this info (temporarily) to the vif-details | 10:26 |
| sean-k-mooney | a tempoary extesion would likely have to exsit for more then 1 cycle for FFU | 10:26 |
| ralonsoh | ok, I'll add this in the spec | 10:27 |
| sean-k-mooney | but would be simple to detect form a nova point of veiw so really its up to you which is cleaner form a neutron point of view | 10:27 |
| ralonsoh | something in the vif-details | 10:27 |
| ralonsoh | and could be temporary for X and Y | 10:28 |
| sean-k-mooney | then drop in Z becaues we jsut always assume ovn is new enough | 10:28 |
| sean-k-mooney | sure | 10:28 |
| ralonsoh | one sec | 10:28 |
| ralonsoh | do we allow migrating from OVS to OVN? | 10:28 |
| ralonsoh | or vice versa? | 10:28 |
| sean-k-mooney | ish | 10:28 |
| ralonsoh | hmmm... | 10:29 |
| sean-k-mooney | it works for vlan networks | 10:29 |
| sean-k-mooney | it wont work for tunnels | 10:29 |
| ralonsoh | I'm saying this because I'll set that info in the port vif details during the binding | 10:29 |
| sean-k-mooney | because of the fact the ml2 drivers will not form the correct mess | 10:29 |
| ralonsoh | of course, if we move to another backend, that could be a problem | 10:29 |
| sean-k-mooney | well multiple port bdining is designed to allow that | 10:30 |
| sean-k-mooney | the main reasons that we dont mix ovn and ml2/ovs today are that they wont from a mesh | 10:30 |
| sean-k-mooney | so tunnels dont really work in a mix env | 10:30 |
| ralonsoh | ok, we can write this info during the live migration, in the destination port binding | 10:31 |
| sean-k-mooney | yep | 10:31 |
| sean-k-mooney | anyway in the mean time im going to push a DNM patch to nova to test the os-vif change | 10:32 |
| opendevreview | Merged openstack/neutron stable/train: Copy existing IPv6 leases to generated lease file https://review.opendev.org/c/openstack/neutron/+/799070 | 10:36 |
| opendevreview | Akihiro Motoki proposed openstack/networking-midonet stable/rocky: Fix networking-l2gw location https://review.opendev.org/c/openstack/networking-midonet/+/798993 | 10:55 |
| opendevreview | Akihiro Motoki proposed openstack/neutron-fwaas stable/rocky: Fix networking-l2gw location https://review.opendev.org/c/openstack/neutron-fwaas/+/798976 | 11:28 |
| opendevreview | Akihiro Motoki proposed openstack/neutron-fwaas stable/queens: Fix networking-l2gw location https://review.opendev.org/c/openstack/neutron-fwaas/+/798987 | 11:31 |
| opendevreview | Merged openstack/neutron-tempest-plugin master: Cleanup Security Group rules after TC execution https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/799115 | 11:38 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron-tempest-plugin master: Fix test_allocate_ips_are_from_correct_subnet on ML2/OVN backend https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/799648 | 11:53 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: Use IP_VERSION_{4,6} constants in ovn_client module https://review.opendev.org/c/openstack/neutron/+/799650 | 11:56 |
| opendevreview | Lajos Katona proposed openstack/neutron master: Privsep with timeout for get_link_devices call https://review.opendev.org/c/openstack/neutron/+/794994 | 11:59 |
| zigo | frickler: I don't, sorry. It's just we experienced it... | 12:00 |
| opendevreview | Lajos Katona proposed openstack/networking-bgpvpn stable/train: stable only: Fix failing jobs and make l-c non-voting https://review.opendev.org/c/openstack/networking-bgpvpn/+/796808 | 13:01 |
| *** osmanlicilegi is now known as Guest4 | 13:34 | |
| slaweq | #startmeeting networking | 14:00 |
| opendevmeet | Meeting started Tue Jul 6 14:00:08 2021 UTC and is due to finish in 60 minutes. The chair is slaweq. 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 |
| mlavalle | o/ | 14:00 |
| gibi | o/ | 14:00 |
| bcafarel | \o | 14:00 |
| lajoskatona | o/ | 14:00 |
| obondarev | hi | 14:00 |
| amotoki | o/ | 14:00 |
| slaweq | let's start | 14:01 |
| slaweq | #topic Announcements | 14:01 |
| ralonsoh | hi | 14:01 |
| slaweq | Xena-2 milestone is next week | 14:01 |
| slaweq | we should start focusing now on the implementation for our accepted BPs :) | 14:01 |
| rubasov | o/ | 14:01 |
| slaweq | we will talk more about them in the next section | 14:02 |
| slaweq | also, I think it's good time to do releases of stable branches | 14:02 |
| slaweq | wdyt? | 14:02 |
| mlavalle | +1 | 14:02 |
| slaweq | I mean next week | 14:02 |
| mlavalle | yeah | 14:02 |
| bcafarel | +1 with milestone sync :) | 14:02 |
| obondarev | +1 | 14:02 |
| ralonsoh | +1 | 14:02 |
| amotoki | +1 | 14:03 |
| slaweq | thx, bcafarel, amotoki can You take care of it next week? | 14:03 |
| bcafarel | sure thing | 14:03 |
| amotoki | sure | 14:03 |
| slaweq | my question is related to the next announcement which is | 14:03 |
| slaweq | PTL on vacation next week :) | 14:03 |
| slaweq | thx bcafarel and amotoki | 14:03 |
| mlavalle | +1 | 14:03 |
| mlavalle | well deserved | 14:03 |
| obondarev | me too :) | 14:04 |
| slaweq | I will be on PTO starting this Friday and for whole next week | 14:04 |
| amotoki | slaweq: enjoy your vacation :) | 14:04 |
| slaweq | thx :) | 14:04 |
| ralonsoh | so no drivers meeting? | 14:04 |
| slaweq | I think I will cancel this and next week's drivers meeting | 14:04 |
| bcafarel | enjoy the time off both! | 14:04 |
| slaweq | and next week team meeting also | 14:04 |
| slaweq | unless anyone wants to chair it | 14:04 |
| ralonsoh | I had a pending rfe from last week | 14:05 |
| ralonsoh | and I would not like to postpone so much time | 14:05 |
| slaweq | mlavalle: can You maybe chair drivers meeting this friday? | 14:05 |
| slaweq | I will prepare agenda for it on Thursday and will send email | 14:05 |
| mlavalle | I can chair the drivers meeting this coming Friday | 14:05 |
| slaweq | thx a lot | 14:05 |
| ralonsoh | thank you | 14:05 |
| mlavalle | you beat me to it :-) | 14:05 |
| slaweq | :) | 14:05 |
| amotoki | one confirmation: do we have the drivers meeting this week and cancel next week's one? | 14:06 |
| ralonsoh | so far yes | 14:06 |
| slaweq | amotoki: I think it will be ok to do it like that | 14:06 |
| mlavalle | +1 | 14:06 |
| amotoki | sounds good | 14:06 |
| slaweq | ok, I will send email about it later this week for sure | 14:07 |
| slaweq | so we are good with meetings in this and next week :) | 14:07 |
| slaweq | so we can move on | 14:07 |
| slaweq | review.opendev.org server upgrade 18/19th July 2021 - http://lists.openstack.org/pipermail/openstack-discuss/2021-July/023434.html | 14:07 |
| slaweq | this is just in case if You missed email | 14:07 |
| opendevreview | Przemyslaw Szczerbik proposed openstack/neutron master: Use os-resource-classes lib for resource classes names https://review.opendev.org/c/openstack/neutron/+/799676 | 14:08 |
| slaweq | there will be new IPs of the gerrit server, so maybe You will need to update some of Your firewall rules, etc. | 14:09 |
| slaweq | next one | 14:09 |
| slaweq | October PTG announcement http://lists.openstack.org/pipermail/openstack-discuss/2021-June/023370.html | 14:09 |
| slaweq | it will be again virtual event :( | 14:09 |
| bcafarel | cross fingers for 2022 | 14:09 |
| slaweq | ++ | 14:09 |
| slaweq | and that are all announcements from me today | 14:10 |
| slaweq | do You have anything else? | 14:10 |
| opendevreview | Przemyslaw Szczerbik proposed openstack/neutron master: Use os-resource-classes lib for resource classes names https://review.opendev.org/c/openstack/neutron/+/799676 | 14:11 |
| slaweq | ok, so let's move on | 14:11 |
| slaweq | #topic Blueprints | 14:12 |
| slaweq | #link https://bugs.launchpad.net/neutron/+milestone/xena-2 | 14:12 |
| slaweq | we already finished https://blueprints.launchpad.net/neutron/+spec/distributed-dhcp-for-ml2-ovs | 14:12 |
| opendevreview | Przemyslaw Szczerbik proposed openstack/neutron-lib master: Add a TODO note for resource classes constants https://review.opendev.org/c/openstack/neutron-lib/+/799034 | 14:12 |
| slaweq | and https://blueprints.launchpad.net/neutron/+spec/default-dns-zone-per-tenant | 14:12 |
| slaweq | thx for reviews and for patches :) | 14:12 |
| slaweq | regarding 4 Ericsson RFEs, we merged specs for all of them | 14:13 |
| slaweq | big thx for reviewing them | 14:13 |
| mlavalle | +1 | 14:13 |
| mlavalle | That BGPaaS spec was really interesting | 14:13 |
| rubasov | thanks a lot for everyone | 14:13 |
| mlavalle | and well explained | 14:14 |
| slaweq | mlavalle: I have a question about https://blueprints.launchpad.net/neutron/+spec/sg-sharing-as-readonly | 14:14 |
| slaweq | do You maybe know if rm_work is going to work on it? | 14:14 |
| mlavalle | I'll ask today during our stanbd up. we'll get back to you tomorrow | 14:15 |
| slaweq | thank You | 14:15 |
| mlavalle | if he doesn't I might offer to take over | 14:15 |
| slaweq | would be great, thx a lot | 14:15 |
| slaweq | ralonsoh: and what about https://blueprints.launchpad.net/neutron/+spec/nftables-migration ? | 14:16 |
| slaweq | patches which are listed there are merged (or abandoned) | 14:16 |
| slaweq | what is current status of that BP? | 14:16 |
| ralonsoh | for now, it is closed | 14:16 |
| ralonsoh | or done | 14:16 |
| ralonsoh | eventually we'll need to move to native nftables | 14:17 |
| slaweq | so "implemented" or "deferred"? | 14:17 |
| ralonsoh | but this is more complicated and should have another BP | 14:17 |
| ralonsoh | implemented | 14:17 |
| slaweq | ok, thx | 14:17 |
| mlavalle | I think he meant implemented for now | 14:17 |
| slaweq | thx, I set it as implemented | 14:17 |
| slaweq | so that list looks pretty ok now | 14:18 |
| slaweq | thank You | 14:18 |
| slaweq | any other BPs You want to talk about today? | 14:18 |
| slaweq | if not, let's move on to the next topic | 14:19 |
| slaweq | #topic Bugs | 14:19 |
| slaweq | mlavalle was bug deputy last week. Report http://lists.openstack.org/pipermail/openstack-discuss/2021-July/023473.html | 14:19 |
| slaweq | mlavalle: any bugs You want to highlight now? | 14:19 |
| mlavalle | I had two that I wanted to highlight. One lacked owner but Elvira just assigned it to herself: https://bugs.launchpad.net/neutron/+bug/1934420 | 14:20 |
| mlavalle | and the other, https://bugs.launchpad.net/neutron/+bug/1933813, resolved to invalid 1 hour ago. THanks to ralonsoh for helping with this one | 14:21 |
| fungi | mlavalle: are you still serving as neutron's vulnerability management liaison? just trying to make sure our contact info for that is properly up to date: https://wiki.openstack.org/wiki/CrossProjectLiaisons#Vulnerability_management | 14:21 |
| ralonsoh | sorry for not providing a solution | 14:21 |
| mlavalle | ralonsoh: you provided great guidance that helped reporter to reach a conclusion :-) | 14:22 |
| rm_work | Hey sorry just my quick response is, priorities shifted internally, and also I had no freaking clue how to do that so it was taking me forever to make progress (though I assume somebody who knows neutron's codebase could do most of it in like 30 min) | 14:22 |
| mlavalle | rm_work: I'll take a look into it if that's ok with you | 14:22 |
| rm_work | Please feel free | 14:22 |
| mlavalle | slaweq: you have your answer | 14:23 |
| slaweq | rm_work: mlavalle thx :) | 14:23 |
| rm_work | I have the Octavia side (which uses it) basically ready, would need rebasing but that I can handle :P | 14:23 |
| mlavalle | fungi: yes, if slaweq is ok with it | 14:23 |
| slaweq | mlavalle: of course I am | 14:23 |
| fungi | thanks for confirming, i just try to make sure we have current info for that | 14:23 |
| mlavalle | ok, following my fearless leader benevolent guidance, I am the liaison | 14:24 |
| fungi | heh | 14:24 |
| slaweq | LOL | 14:24 |
| slaweq | any other bugs anyone wants to discuss today? | 14:27 |
| slaweq | if not, I think we can move on | 14:29 |
| slaweq | our bug deputy this week is rubasov | 14:29 |
| slaweq | and next week is ralonsoh's turn | 14:29 |
| rubasov | on it | 14:29 |
| mlavalle | who will be bugs deputy next week and the following one | 14:29 |
| ralonsoh | ack | 14:29 |
| slaweq | ralonsoh: sorry, but I will not send You reminder early next week | 14:29 |
| ralonsoh | np | 14:29 |
| slaweq | please remember about it :) | 14:30 |
| mlavalle | I can do it ;-) | 14:30 |
| slaweq | thx mlavalle :) | 14:30 |
| mlavalle | seriusly, do we need to send a reminder to the person who goes after ralonsoh? | 14:30 |
| mlavalle | seriously^^^ | 14:30 |
| slaweq | after ralonsoh will be lucasagomes but I will be back so I will take care of it | 14:31 |
| slaweq | thx mlavalle | 14:31 |
| mlavalle | ack | 14:31 |
| slaweq | and that was my last thing for today's meeting | 14:31 |
| slaweq | do You have anything else You want to discuss today? | 14:32 |
| ralonsoh | no thanks | 14:32 |
| bcafarel | nothing from me | 14:32 |
| lucasagomes | slaweq, ack, thanks! So I'm bug deputy for next week ? I will add to my calendar :D thanks for the heads up | 14:32 |
| mlavalle | lucasagomes: no, week of July 19th | 14:32 |
| slaweq | thx mlavalle :) | 14:33 |
| lucasagomes | mlavalle, ack, thanks! | 14:33 |
| mlavalle | de nada | 14:33 |
| amotoki | lucasagomes: https://wiki.openstack.org/wiki/Network/Meetings#Bug_deputy | 14:33 |
| * lucasagomes set it to the calendar, I won't forget :-) | 14:33 | |
| lucasagomes | thanks folks | 14:33 |
| slaweq | ok, if there are no other topics for today, I will give You some time back | 14:34 |
| bcafarel | lucasagomes: you will not avoid the PTL reminder :) | 14:34 |
| mlavalle | \o/ | 14:34 |
| lucasagomes | lol fair | 14:34 |
| slaweq | thx for attending the meeting and have a great week | 14:34 |
| slaweq | o/ | 14:34 |
| slaweq | #endmeeting | 14:35 |
| opendevmeet | Meeting ended Tue Jul 6 14:35:01 2021 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:35 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/networking/2021/networking.2021-07-06-14.00.html | 14:35 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/networking/2021/networking.2021-07-06-14.00.txt | 14:35 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/networking/2021/networking.2021-07-06-14.00.log.html | 14:35 |
| ralonsoh | bye | 14:35 |
| lajoskatona | bye | 14:35 |
| amotoki | o/ | 14:35 |
| rubasov | o/ | 14:35 |
| opendevreview | Przemyslaw Szczerbik proposed openstack/neutron master: Use os-resource-classes lib for resource classes names https://review.opendev.org/c/openstack/neutron/+/799676 | 14:51 |
| opendevreview | Lajos Katona proposed openstack/networking-bgpvpn stable/train: stable only: Fix failing jobs and make l-c non-voting https://review.opendev.org/c/openstack/networking-bgpvpn/+/796808 | 14:55 |
| slaweq | #startmeeting neutron_ci | 15:00 |
| opendevmeet | Meeting started Tue Jul 6 15:00:17 2021 UTC and is due to finish in 60 minutes. The chair is slaweq. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:00 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:00 |
| opendevmeet | The meeting name has been set to 'neutron_ci' | 15:00 |
| ralonsoh | hi | 15:00 |
| bcafarel | hi again | 15:00 |
| slaweq | hi (again)! | 15:00 |
| slaweq | Grafana dashboard: http://grafana.openstack.org/dashboard/db/neutron-failure-rate | 15:00 |
| slaweq | Please open now :) | 15:00 |
| slaweq | ok, let's start | 15:01 |
| slaweq | #topic Actions from previous meetings | 15:01 |
| slaweq | amotoki to clean failing jobs in networking-odl rocky and older | 15:01 |
| lajoskatona | Hi | 15:01 |
| amotoki | hi | 15:01 |
| obondarev | hi | 15:02 |
| amotoki | zuul configuration errors happen in networking-odl, neutron-fwaas and -midonet stable/rocky or older. | 15:02 |
| amotoki | networking-odl rocky and queens have been done. | 15:03 |
| amotoki | others are under reviews except networking-odl ocata. | 15:03 |
| amotoki | networking-odl ocata needs to be EOL'ed as neutron ocata is already EOL. lajoskatona prepared the release patch. | 15:03 |
| lajoskatona | Jsut for reference the original mail: http://lists.openstack.org/pipermail/openstack-discuss/2021-June/023321.html | 15:03 |
| lajoskatona | and the patch: https://review.opendev.org/c/openstack/releases/+/799473 | 15:04 |
| slaweq | thx amotoki and lajoskatona for taking care of all of that | 15:04 |
| lajoskatona | no problem | 15:05 |
| amotoki | https://review.opendev.org/c/openstack/releases/+/799472 also will clean up newton or older unofficial branches under the neutron governance curretly. | 15:05 |
| lajoskatona | actually there's another release patch for older branches from elod: https://review.opendev.org/c/openstack/releases/+/799472 | 15:05 |
| amotoki | that's all from me | 15:05 |
| lajoskatona | yeah, exactly as amotoki wrote it :-) | 15:05 |
| slaweq | thx | 15:05 |
| slaweq | ok, so next one | 15:05 |
| slaweq | lajoskatona to start EOL process for networking-odl rocky and older | 15:05 |
| slaweq | I assume it's already covered as well :) | 15:06 |
| lajoskatona | yeah | 15:06 |
| slaweq | thx | 15:06 |
| slaweq | so next one | 15:06 |
| lajoskatona | I haven't sent mail, as I assumethe original mail from You slaweq covered odl as well | 15:06 |
| slaweq | ralonsoh to check if there is better way to check dns nameservers in cirros | 15:06 |
| ralonsoh | still testing how to do it in cirros | 15:06 |
| ralonsoh | in a different way | 15:06 |
| slaweq | ok | 15:06 |
| slaweq | I will add it for You for next time to not forget | 15:07 |
| slaweq | ok? | 15:07 |
| ralonsoh | sure | 15:07 |
| slaweq | #action ralonsoh to check if there is better way to check dns nameservers in cirros | 15:07 |
| slaweq | thx ralonsoh | 15:07 |
| slaweq | and that are all actions from last week | 15:07 |
| slaweq | #topic Stadium projects | 15:07 |
| slaweq | lajoskatona: any other updates/issues with stadium | 15:08 |
| slaweq | *stadium's ci | 15:08 |
| lajoskatona | The most interesting is to eol old branches which we covered | 15:08 |
| bcafarel | cleanups++ | 15:08 |
| lajoskatona | and the gerrit server restart (http://lists.openstack.org/pipermail/openstack-discuss/2021-July/023434.html ) wihc you mentioned on previous meeting, so after that taas will be under openstack/ namepace | 15:09 |
| slaweq | great | 15:09 |
| slaweq | thx | 15:09 |
| slaweq | I think we can move on | 15:09 |
| amotoki | during fixing netwokring-l2gw location in zuul, I noticed a lot of failures in stadium old branhces. I moved them to the experimental queue for the record. | 15:10 |
| amotoki | is it the right action? | 15:10 |
| slaweq | amotoki: if we can't/don't have time to fix such failing jobs, then yes | 15:10 |
| amotoki | I tried to fix them if they have simple straight-forward fixes, but otherwise i did so. | 15:10 |
| slaweq | I think it's good approach to move such jobs to experimental queue | 15:10 |
| lajoskatona | I am fine with it, those fixes are really time consuming | 15:11 |
| slaweq | thx a lot | 15:11 |
| amotoki | if you have memory on devstack-gate and pip/pip3 issues, it would be nice to check it for example in neutron-fwaas. | 15:11 |
| bcafarel | +1, if it can be easily fixed it is nice, but for EM branches we mostly keep the light on | 15:11 |
| slaweq | I will try to check but I don't promise anything | 15:11 |
| amotoki | most failing jobs still use the legacy way and it takes time :( | 15:12 |
| slaweq | yeah, that is not something I'm really familiar with | 15:13 |
| bcafarel | :) | 15:13 |
| amotoki | anyway let's fix it separetely :) I think we can move on | 15:14 |
| slaweq | thx | 15:14 |
| slaweq | so next topic | 15:14 |
| slaweq | #topic Stable branches | 15:14 |
| slaweq | bcafarel: anything regarding stable branches ci in neutron? | 15:14 |
| slaweq | IMO it works pretty ok recently | 15:15 |
| slaweq | but maybe I missed some issues there | 15:15 |
| bcafarel | indeed, backports are getting in quite nicely | 15:15 |
| bcafarel | one that went back to rocky required a few rechecks | 15:15 |
| bcafarel | but nothing specific, just an array of separate tests (including compute and storage) failing | 15:15 |
| bcafarel | and that branch is low on backports so no need to dig further atm | 15:16 |
| bcafarel | full support branches almost all patches are in at the moment, which is nice for the incoming stable releases :) | 15:16 |
| slaweq | ++ | 15:17 |
| slaweq | great news | 15:17 |
| slaweq | #topic Grafana | 15:17 |
| slaweq | #link http://grafana.openstack.org/dashboard/db/neutron-failure-rate | 15:17 |
| slaweq | in grafana failure rates seems to be pretty ok for all of the jobs recently | 15:18 |
| slaweq | I think we can move on, unless You see any issue with grafana and wants to talk about it now | 15:19 |
| ralonsoh | nothing from me | 15:20 |
| bcafarel | that looks good to me | 15:20 |
| slaweq | ok, let's move on | 15:20 |
| slaweq | #topic fullstack/functional | 15:20 |
| bcafarel | also this is with updated job names right? | 15:20 |
| slaweq | bcafarel: I think it is | 15:21 |
| slaweq | regarding those jobs I have only one small update about fullstack issue https://bugs.launchpad.net/neutron/+bug/1933234 | 15:21 |
| slaweq | I just confirmed today with extra logs why this test is failing | 15:21 |
| slaweq | basically router is still processed and its ports aren't added to the RouterInfo.internal_ports cache | 15:22 |
| slaweq | when network_update rpc message comes | 15:22 |
| slaweq | due to that, it can't find port attached to that router from the network | 15:23 |
| slaweq | and router update isn't scheduled | 15:23 |
| *** gthiemon1e is now known as gthiemonge | 15:23 | |
| ralonsoh | good catch | 15:23 |
| slaweq | so this is in fact real bug, not test issue | 15:23 |
| ralonsoh | indeed a race condition | 15:23 |
| ralonsoh | how are we updating the network before adding the port to the internal cache? | 15:23 |
| slaweq | and I don't know yet how to fix it | 15:23 |
| slaweq | ralonsoh: so router is processed and goes to https://github.com/openstack/neutron/blob/3764969b82c6e7b8c74172a1ec4d230ce4ddedcc/neutron/agent/l3/router_info.py#L636 | 15:24 |
| slaweq | there it should be added to the internal ports cache | 15:24 |
| ralonsoh | right | 15:24 |
| slaweq | but in the meantime, network_update is called https://github.com/openstack/neutron/blob/3764969b82c6e7b8c74172a1ec4d230ce4ddedcc/neutron/agent/l3/agent.py#L602 | 15:24 |
| ralonsoh | ok, I see | 15:25 |
| slaweq | and if there is no this port in cache yet, it fails to schedule router update | 15:25 |
| ralonsoh | and nothing to say "stop, we are updating this..." | 15:25 |
| slaweq | yes, I think I will add some kind of flag (lock) in https://github.com/openstack/neutron/blob/3764969b82c6e7b8c74172a1ec4d230ce4ddedcc/neutron/agent/l3/router_info.py#L1256 | 15:26 |
| slaweq | and then we can check it in network_update if it's processing that router info or not | 15:26 |
| slaweq | if yes, we can wait a bit with looking for ports | 15:26 |
| slaweq | but I didn't implement anything yet | 15:26 |
| slaweq | I have it on my todo list for tomorrow :) | 15:26 |
| lajoskatona | regarding https://bugs.launchpad.net/neutron/+bug/1930401 and privsep timeout | 15:27 |
| lajoskatona | https://review.opendev.org/c/openstack/neutron/+/794994 is failing with old timeouts :-( | 15:27 |
| lajoskatona | I started to check, but have no (quick) idea what should be there | 15:28 |
| ralonsoh | but this is because what was failing is not the privsep command | 15:28 |
| ralonsoh | but the daemon spawn process, right? | 15:28 |
| ralonsoh | the daemon was not spawning | 15:28 |
| obondarev | ++ | 15:28 |
| lajoskatona | ralonsoh: yeah | 15:29 |
| ralonsoh | the timeout will prevent from a dead lock during a command execution | 15:29 |
| ralonsoh | but not prevent or mitigate the problem we have here | 15:29 |
| slaweq | so we still need to have 2 dhcp agents in those test which helped us a lot to workaround that issue | 15:31 |
| ralonsoh | yes for now until we now what is preventing the daemon to start | 15:32 |
| slaweq | k | 15:32 |
| lajoskatona | ok | 15:32 |
| slaweq | #topic Tempest/Scenario | 15:33 |
| slaweq | here I just wanted to ask You to review https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/799648 | 15:33 |
| slaweq | it's small patch but it is causing issues in the tripleo based jobs which runs ovn | 15:34 |
| slaweq | thx in advance | 15:34 |
| slaweq | regarding issues in those jobs, I didn't found anything new worth to discuss today | 15:34 |
| slaweq | and that's all from me for today regarding CI jobs | 15:35 |
| slaweq | do You have anything else You want to discuss? | 15:35 |
| ralonsoh | I'm fine | 15:35 |
| bcafarel | same here | 15:36 |
| slaweq | so one last thing - do You want to cancel next week's meeting or anyone wants to chair it? | 15:36 |
| ralonsoh | (I'm fine with having a "free" week) | 15:37 |
| lajoskatona | +1 | 15:37 |
| ralonsoh | if there is nothing catastrophic, of course | 15:37 |
| bcafarel | worst case we know where to ping people :) | 15:37 |
| ralonsoh | hehehe yes | 15:37 |
| slaweq | ok, good | 15:37 |
| slaweq | so I will cancel next week's meeting | 15:37 |
| slaweq | and I think we are done for today | 15:38 |
| slaweq | thx a lot for attending and for keeping our ci up and running :) | 15:38 |
| slaweq | it seems really good recently | 15:38 |
| slaweq | have a great week and see You online | 15:38 |
| slaweq | o/ | 15:38 |
| ralonsoh | bye | 15:38 |
| slaweq | #endmeeting | 15:38 |
| opendevmeet | Meeting ended Tue Jul 6 15:38:47 2021 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:38 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/neutron_ci/2021/neutron_ci.2021-07-06-15.00.html | 15:38 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/neutron_ci/2021/neutron_ci.2021-07-06-15.00.txt | 15:38 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/neutron_ci/2021/neutron_ci.2021-07-06-15.00.log.html | 15:38 |
| lajoskatona | o/ | 15:38 |
| amotoki | bye | 15:38 |
| bcafarel | o/ | 15:39 |
| opendevreview | Lajos Katona proposed openstack/neutron master: Privsep with timeout for get_link_devices call https://review.opendev.org/c/openstack/neutron/+/794994 | 15:56 |
| *** rpittau is now known as rpittau|afk | 16:39 | |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-specs master: [WIP]Create intermediate OVS bridge to improve live-migration in OVN https://review.opendev.org/c/openstack/neutron-specs/+/799198 | 17:01 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-specs master: Create intermediate OVS bridge to improve live-migration in OVN https://review.opendev.org/c/openstack/neutron-specs/+/799198 | 17:02 |
| opendevreview | Merged openstack/neutron stable/wallaby: [OVN] neutron-ovn-metadat-agent add retry logic for sb_idl https://review.opendev.org/c/openstack/neutron/+/799356 | 17:03 |
| opendevreview | Merged openstack/neutron stable/victoria: [OVN] neutron-ovn-metadat-agent add retry logic for sb_idl https://review.opendev.org/c/openstack/neutron/+/799357 | 17:10 |
| opendevreview | Merged openstack/neutron stable/wallaby: Update arp entry of snat port on qrouter ns https://review.opendev.org/c/openstack/neutron/+/799375 | 17:10 |
| opendevreview | Merged openstack/neutron stable/victoria: Update arp entry of snat port on qrouter ns https://review.opendev.org/c/openstack/neutron/+/799476 | 17:11 |
| opendevreview | Merged openstack/neutron stable/train: Add extension unit tests for conntrack_helper plugin https://review.opendev.org/c/openstack/neutron/+/774492 | 17:11 |
| opendevreview | Merged openstack/neutron stable/ussuri: Update arp entry of snat port on qrouter ns https://review.opendev.org/c/openstack/neutron/+/799478 | 17:11 |
| *** mgoddard- is now known as mgoddard | 17:26 | |
| opendevreview | Merged openstack/os-vif master: update os-vif ci to account for devstack default changes https://review.opendev.org/c/openstack/os-vif/+/798038 | 18:52 |
| opendevreview | Elvira García Ruiz proposed openstack/neutron-tempest-plugin master: Add test for mac learning https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/796267 | 19:04 |
| opendevreview | Merged openstack/neutron-lib master: Add a TODO note for resource classes constants https://review.opendev.org/c/openstack/neutron-lib/+/799034 | 19:47 |
| opendevreview | Brian Haley proposed openstack/ovn-octavia-provider master: Move linters dependencies to tox.ini https://review.opendev.org/c/openstack/ovn-octavia-provider/+/779060 | 21:55 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!