Thursday, 2021-10-14

*** tbachman is now known as Guest280400:15
opendevreviewMerged openstack/neutron master: Bump os-ken to 2.2.0  https://review.opendev.org/c/openstack/neutron/+/81366601:30
opendevreviewLajos Katona proposed openstack/neutron-lib master: BFD validator  https://review.opendev.org/c/openstack/neutron-lib/+/80997807:35
opendevreviewElvira García Ruiz proposed openstack/networking-ovn stable/train: [ovn] Add logs for ovs to ovn migration  https://review.opendev.org/c/openstack/networking-ovn/+/81396308:19
opendevreviewliuyulong proposed openstack/neutron master: Make flow installation trunk size automatic adjustment  https://review.opendev.org/c/openstack/neutron/+/76507208:35
fricklerslaweq: does OVN inhibit the use of the dns extensions? wasn't that fixed some time ago? I get e.g. "Unrecognized attribute(s) 'dns_domain'" when I try to create a port with that option on a fresh devstack deployment (service q-dns enabled)08:37
frickleroh, I think those other dns_* extension also need to be added in neutron/common/ovn/extensions.py08:44
fricklerpatch upcoming08:45
frickleralso indicates that test coverage could be improved08:45
opendevreviewRodolfo Alonso proposed openstack/networking-ovn stable/train: [OVN] Update the DHCP options when the metadata port is modified  https://review.opendev.org/c/openstack/networking-ovn/+/81271408:45
opendevreviewRodolfo Alonso proposed openstack/networking-ovn stable/train: [OVN] Update the DHCP options when the metadata port is modified  https://review.opendev.org/c/openstack/networking-ovn/+/81271408:46
ralonsohfrickler, if I'm not wrong, that was added in https://review.opendev.org/c/openstack/neutron/+/686343/31/neutron/common/ovn/extensions.py08:49
fricklerralonsoh: yes, but that's incomplete. needs dns_domain_ports and dns_publish_fixed_ip, too. I have a patch ready, just checking on that comment before the block regarding docs08:50
opendevreviewRodolfo Alonso proposed openstack/neutron stable/ussuri: [OVN] Update the DHCP options when the metadata port is modified  https://review.opendev.org/c/openstack/neutron/+/81341108:54
opendevreviewRodolfo Alonso proposed openstack/neutron master: Use importlib instead of imp  https://review.opendev.org/c/openstack/neutron/+/80459609:00
opendevreviewRodolfo Alonso proposed openstack/neutron master: [OVN] Check if OVN NB supports "Port_Group"  https://review.opendev.org/c/openstack/neutron/+/81217609:07
dalvarezlucasagomes: i left some comments https://review.opendev.org/c/openstack/neutron/+/813610/09:08
lucasagomesdalvarez, will take a look09:08
dalvarezihrachys_: lucasagomes ^ what if we fix them to those values and avoid setting a config option? maybe im missing some case09:08
dalvarezthe only 'catch' is that we'll send one ARP occasionally and even in the worst case, with these options we'll be like we're today 09:09
opendevreviewRodolfo Alonso proposed openstack/neutron master: [OVN] Metadata ports device_owner is "network:distributed" only  https://review.opendev.org/c/openstack/neutron/+/80770709:10
lucasagomesdalvarez, I would defer to whether hardcode those tunnings or not to ihar, not sure if someone may want to disable those options09:13
opendevreviewDr. Jens Harbott proposed openstack/neutron master: Fix list of DNS extensions supported by OVN  https://review.opendev.org/c/openstack/neutron/+/81396909:22
fricklerralonsoh: slaweq: ^^ also added a bug so it can get backported09:22
ralonsohfrickler, I think there was a problem when those extensions where used at the same time09:23
ralonsohIf I remember correctly, those extensions had the same base class09:23
ralonsohand calling twice the same parent method produced an error09:24
fricklerralonsoh: the only issue I know of is that you may only set one of them in the ml2.ini09:24
ralonsohbut I need to check that09:24
ralonsohfrickler, in OVN the extensions are loaded from this variable09:24
ralonsohthis is an static configuration09:24
ralonsohanyway, let me check what I'm saying09:25
fricklerralonsoh: well I'm testing my patch locally in devstack and it seems to work just fine, so I'm pretty confident09:27
opendevreviewMaxim Babushkin proposed openstack/neutron master: Add "FLAVOR_NAME" to ovn migration resources creation  https://review.opendev.org/c/openstack/neutron/+/81397209:37
ralonsohlajoskatona, slaweq hi. With all those new patches in os-ken, I would request a new version ASAP10:01
ralonsohjust to test it 10:01
lajoskatonaralonsoh: ok, thanks10:02
-opendevstatus- NOTICE: zuul was stuck processing jobs and has been restarted. pending jobs will be re-enqueued10:02
opendevreviewMaxim Babushkin proposed openstack/neutron master: Add "FLAVOR_NAME" to ovn migration resources creation  https://review.opendev.org/c/openstack/neutron/+/81397210:09
slaweqralonsoh++10:20
slaweqralonsoh: lajoskatona but we have os-ken-from-master job in periodic queue10:21
ralonsohyou are right10:21
slaweqso when the patches will be merged we can check it I think10:21
ralonsohso maybe this is a better place to check10:21
slaweqralonsoh: also, we have tempest-integrated-networking job run in the os-ken check queue so it should be more or less ok :)10:22
slaweqI hope so at least10:22
slaweqmaybe we can add to os-ken some ovs based scenario job, wdyt?10:22
ralonsohI think that's enough, having the periodic job and the tempest test in the check queue10:23
lajoskatonadon't we have some experimental job for it?10:29
lajoskatonahmmm, it seems no10:29
opendevreviewSlawek Kaplonski proposed openstack/neutron master: Change tobiko CI job in the periodic queue  https://review.opendev.org/c/openstack/neutron/+/81397710:46
ralonsohslaweq, can you send the link for the OpenInfra live event?11:17
ralonsohthe youtube link11:17
ralonsoh1400UTC for everyone who wants to watch it11:18
opendevreviewLajos Katona proposed openstack/os-ken master: add ofproto 1.3 coverage, check key-error and attribute-error.  https://review.opendev.org/c/openstack/os-ken/+/81398511:27
lajoskatonaralonsoh: https://www.youtube.com/watch?v=4ZLqILbLIpQ&ab_channel=OpenInfrastructureFoundation11:32
ralonsohlajoskatona, thanks!11:32
lajoskatonaslaweq: Do You think we need rbac session: https://etherpad.opendev.org/p/policy-popup-yoga-ptg ?12:21
lajoskatonaslaweq: on this pad there's Neutron mentioned as how nova can reach Neutron API12:21
slaweqlajoskatona: let me check it12:30
lajoskatonaslaweq: thanks12:30
slaweqlajoskatona: If gibi and other folks from Nova wants us to be on that discussion, we can probably12:32
slaweqbut other than that do we need to discuss it on our side?12:32
slaweqI'm not sure12:32
lajoskatonaslaweq: ok, I ask around12:32
slaweqk12:33
eugenmayerHello, trying to setup the vpnaas using the docs https://docs.openstack.org/neutron/latest/admin/vpnaas-scenario.html .. configured the controller and now trying to create the endpoint group12:53
eugenmayeropenstack --os-cloud=openstack vpn ike policy create ikepolicy12:53
eugenmayerThe resource could not be found.12:53
eugenmayerNeutron server returns request_ids: ['req-d5314da8-7336-4795-af5e-c4bd4a3520a0']12:53
eugenmayerI run this on the controller node, having that clouds.yaml: https://gist.github.com/EugenMayer/83f79d061e175ba2b2610fa991e128d4 (i am using the training-lab setup)12:54
ihrachys_dalvarez lucasagomes I would prefer to hardcode, yes. like we did for stateless FIPs. I thought we are trying to leave a path to back out from the change if something (I don't know what) happens. But that's a silly argument - we would need a knob per bug fix13:10
lucasagomesihrachys_, thanks! I agree it's silly if we do not have any data backing up that we shouldn't have those options enabled13:11
lucasagomesI wasn't too confident to make that call as I am not super familiar with those configs13:11
lucasagomesthanks, I will change the patch there13:12
lucasagomesdalvarez, ^13:12
ihrachys_not that I am... but I think we understand what's the worst case (a arp request here or there)13:12
ihrachys_there's a reason why we don't pre-populate arp tables in the real world13:12
lucasagomesindeed13:12
eugenmayeranybody uses vpnaas in here?13:16
eugenmayeri'am not sure this project is working with the current openstack release at all ;/ But i'am new to the project and it might be a configuration issue on the other hand13:16
lajoskatonaeugenmayer: have You checked master?13:17
opendevreviewSlawek Kaplonski proposed openstack/neutron master: Add functional and fullstack jobs with FIPS enabled  https://review.opendev.org/c/openstack/neutron/+/81400913:19
eugenmayerlajoskatona, i'am on master right now, using the training-lab setup13:21
eugenmayerlajoskatona, that is my setup (on ubuntu 18.04) https://gist.github.com/EugenMayer/83f79d061e175ba2b2610fa991e128d413:24
dalvarezihrachys_: yeah hardcore++++++ :)13:36
dalvarezi mean hardcode13:36
opendevreviewLucas Alvares Gomes proposed openstack/neutron master: [OVN] Tune OVN routers to reduce the mem footprint for ML2/OVN  https://review.opendev.org/c/openstack/neutron/+/81361013:48
eugenmayerWhen i have several compute nodes and and i create a network (LAN), can i span it across those compute nodes automatically or do i have to configure custom ipsec site2site connections?14:06
gmannslaweq: i left comment on this oslo patch in case you did not notice, almost good but I think having a reno will be helpful for this new thing - https://review.opendev.org/c/openstack/oslo.policy/+/804980/314:22
opendevreviewDaniel Speichert proposed openstack/neutron master: Fix OVN sync with new ACL label field  https://review.opendev.org/c/openstack/neutron/+/81388314:41
slaweqgmann: ok, thx a lot, I will update it15:05
opendevreviewBernard Cafarelli proposed openstack/neutron stable/stein: [DNM] Test neutron-tempest-plugin fix  https://review.opendev.org/c/openstack/neutron/+/81384115:10
gmannslaweq: thanks, no hurry just pinged in case you missed review comments15:13
opendevreviewLajos Katona proposed openstack/neutron-vpnaas master: DNM: check master  https://review.opendev.org/c/openstack/neutron-vpnaas/+/81249215:24
lucasagomesihrachys_, thanks for the review in https://review.opendev.org/c/openstack/neutron/+/813610, I added a question for you there15:26
lucasagomesand my rationale on that maintenance method15:26
lajoskatonaeugenmayer: personally I never used vpnaas, so I would check what kind of config we have in our CI (like checking on a current patch), and compare it with what you have15:27
fricklermnaser_: not sure where best to ask this, followup to what you said in the openinfra live session: is that bgp overlay setup of yours documented somewhere? is it opensource or a closed inhouse solution?15:28
opendevreviewLajos Katona proposed openstack/neutron master: Routed provider nets: Fix subnet allocation logic  https://review.opendev.org/c/openstack/neutron/+/77744315:28
ihrachys_lucasagomes thank you, I misunderstood the intent, now all good15:34
lucasagomesihrachys_, thanks! Great reviews btw15:36
mlavallelajoskatona: what about tomorrow's drivers meeting?16:32
lajoskatonamlavalle: I hope I sent out that we will have16:52
lajoskatonahttp://lists.openstack.org/pipermail/openstack-discuss/2021-October/025365.html16:52
mlavalleCool, got it!16:53
lajoskatonamlavalle: I attached logs from last week as we had quite good conversation 16:53
ihrachyslajoskatona still haven't gotten to write up the spec, sorry for that. I hope it's ok if it bubbles up to the next week.17:04
mlavallelajoskatona: ack17:20
lajoskatonaihrachys: tomorrow we can vote without the spec to have the RFE approved, after last weeks meeting I believe that will be formality17:51
ihrachysit's fine to wait, no rush17:51
opendevreviewMerged openstack/os-ken master: Enabled point-to-multipoint links in topology discovery.  https://review.opendev.org/c/openstack/os-ken/+/81382518:09
opendevreviewMerged openstack/os-ken master: mrtlib: Support RFC8050  https://review.opendev.org/c/openstack/os-ken/+/81382718:20
opendevreviewMerged openstack/os-ken master: Add support for Adj-RIB-Out (draft-ietf-grow-bmp-adj-rib-out-03)  https://review.opendev.org/c/openstack/os-ken/+/81382018:31
opendevreviewMerged openstack/os-ken master: Fix major bug in child thread cleanup logic  https://review.opendev.org/c/openstack/os-ken/+/81384919:27
opendevreviewDr. Jens Harbott proposed openstack/neutron-dynamic-routing master: DNM: Testing running with OVN  https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/81405519:31
opendevreviewDr. Jens Harbott proposed openstack/neutron-dynamic-routing master: DNM: Testing running with OVN  https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/81405519:50
opendevreviewDr. Jens Harbott proposed openstack/neutron-dynamic-routing master: DNM: Testing running with OVN  https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/81405519:53
opendevreviewMerged openstack/os-ken master: Patch to add 2 parameters for controlling specific fields in EVPN rou…  https://review.opendev.org/c/openstack/os-ken/+/81384519:53
opendevreviewMerged openstack/os-ken master: Use constants imported from API base  https://review.opendev.org/c/openstack/os-ken/+/81384619:53

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!