Wednesday, 2021-11-03

opendevreviewuchenily proposed openstack/neutron master: Delete stale SNAT gateway device  https://review.opendev.org/c/openstack/neutron/+/81643203:59
nmk94hi05:44
nmk94with security group with attached with remote_security_group  I can see different cookies flows in br-int bridge .  one with default cookie and another new cookie with conjunction flows , is it expected ?05:45
*** redrobot2 is now known as redrobot05:58
*** gibi is now known as gibi_pto_back_thu06:11
opendevreviewDr. Jens Harbott proposed openstack/neutron-dynamic-routing master: Add a StaticScheduler without automatic scheduling  https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/81526506:14
opendevreviewOleg Bondarev proposed openstack/neutron master: [WIP] Add Local IP fullstack test case  https://review.opendev.org/c/openstack/neutron/+/81632707:41
ralonsohslaweq, good morning. I'm checking neutron-tempest-plugin-scenario-ovn timeouts07:44
ralonsohaccording to https://zuul.opendev.org/t/openstack/builds?job_name=neutron-tempest-plugin-scenario-ovn&result=SUCCESS07:44
ralonsohthere are some builds that take almost 2h45m - 2h55m to finish07:44
ralonsohcurrent timeout is 10.000 secs (2h46m, just the test case execution)07:45
ralonsohI would increase this limit07:46
slaweqralonsoh: hi, we can increase this timeout but not longer than 3h as this is global zuul timeout07:48
ralonsohslaweq, thanks, I'll do it07:48
opendevreviewOleg Bondarev proposed openstack/neutron master: Add Local IP Association policy rules  https://review.opendev.org/c/openstack/neutron/+/81643507:48
slaweqbut as we have more "advanced" tests, maybe that is what we should do now07:48
slaweqthx a lot ralonsoh 07:48
opendevreviewOleg Bondarev proposed openstack/neutron master: [WIP] Add Local IP fullstack test case  https://review.opendev.org/c/openstack/neutron/+/81632707:49
lajoskatonaslaweq, ralonsoh: I checked the dstat log of such runs07:49
lajoskatonaslaweq, ralonsoh: what is clearly visible is that there are periods when all the mem is used on the host07:50
lajoskatonaslaweq, ralonsoh: not sure if we can somehow tune that, like asking with serial execution07:51
slaweqlajoskatona: In that job we are still using "regular" ubuntu image for advanced tests07:51
slaweqand maybe that's why we are just below the limits and nodes are slowing down07:51
ralonsohcan we swap to ubuntu minimal in ovn?07:51
lajoskatonaslaweq: ahhh, I remember for ovn the minimal ubuntu is not good07:52
slaweqralonsoh: unfortunatelly not because of multicast igmp snooping test07:52
ralonsohright, now I remember07:52
slaweqwe would need some custom image with kernel with enabled multicast support07:52
lajoskatonaslaweq: infra can help with that? I suppose we need to build/store that  somewhere07:54
slaweqlajoskatona: tbh I don't know, I never tried to ask about it07:58
slaweqI can try to look into it but quick "solution" may be to increase that timeout to full 3h :)07:58
slaweqhopefully it will give us some time07:58
lajoskatonaslaweq: that's sure :-)07:58
lajoskatonaI ask gmann what we need to push this patch: https://review.opendev.org/c/openstack/tempest/+/807790 07:59
slaweq++07:59
slaweqthat also can help07:59
slaweqlajoskatona: and I will try to prepare some image which could work with that multicast test and then I will try to discuss it with infra08:00
slaweqother workaround can be to disable igmp_snooping in the ovn job - then we can use ubuntu minimal and it should works fine08:01
opendevreviewOleg Bondarev proposed openstack/neutron master: Add Local IP Association policy rules  https://review.opendev.org/c/openstack/neutron/+/81643508:15
opendevreviewOleg Bondarev proposed openstack/neutron master: [WIP] Add Local IP fullstack test case  https://review.opendev.org/c/openstack/neutron/+/81632708:15
opendevreviewRodolfo Alonso proposed openstack/neutron-tempest-plugin master: Increase "neutron-tempest-plugin-scenario-ovn" timeout to 3h  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/81643808:32
opendevreviewOleg Bondarev proposed openstack/neutron master: [WIP] Add Local IP fullstack test case  https://review.opendev.org/c/openstack/neutron/+/81632708:36
opendevreviewRodolfo Alonso proposed openstack/neutron master: [OVN] Check if OVN NB supports stateless NAT rules  https://review.opendev.org/c/openstack/neutron/+/81637608:45
opendevreviewRodolfo Alonso proposed openstack/neutron master: [OVN] Check if OVN SB supports virtual ports  https://review.opendev.org/c/openstack/neutron/+/81638309:26
opendevreviewDr. Jens Harbott proposed openstack/neutron-dynamic-routing master: Add a StaticScheduler without automatic scheduling  https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/81526509:50
opendevreviewMerged openstack/neutron master: Don't setup bridge controller if it is already set  https://review.opendev.org/c/openstack/neutron/+/81525510:23
opendevreviewPrzemyslaw Szczerbik proposed openstack/neutron-lib master: Move RULE_TYPE_MINIMUM_PACKET_RATE to neutron-lib  https://review.opendev.org/c/openstack/neutron-lib/+/81644710:24
opendevreviewSlawek Kaplonski proposed openstack/neutron master: Add functional and fullstack jobs with FIPS enabled  https://review.opendev.org/c/openstack/neutron/+/81400910:56
opendevreviewOleg Bondarev proposed openstack/neutron master: Add Local IP L2 extension  https://review.opendev.org/c/openstack/neutron/+/80711610:57
opendevreviewOleg Bondarev proposed openstack/neutron master: Add Local IP L2 extension flows  https://review.opendev.org/c/openstack/neutron/+/81510210:57
opendevreviewOleg Bondarev proposed openstack/neutron master: Add Local IP Association policy rules  https://review.opendev.org/c/openstack/neutron/+/81643510:57
opendevreviewOleg Bondarev proposed openstack/neutron master: [WIP] Add Local IP fullstack test case  https://review.opendev.org/c/openstack/neutron/+/81632710:57
opendevreviewSlawek Kaplonski proposed openstack/neutron stable/xena: Don't setup bridge controller if it is already set  https://review.opendev.org/c/openstack/neutron/+/81645411:05
opendevreviewSlawek Kaplonski proposed openstack/neutron stable/wallaby: Don't setup bridge controller if it is already set  https://review.opendev.org/c/openstack/neutron/+/81634811:05
opendevreviewSlawek Kaplonski proposed openstack/neutron stable/victoria: Don't setup bridge controller if it is already set  https://review.opendev.org/c/openstack/neutron/+/81645711:12
opendevreviewSlawek Kaplonski proposed openstack/neutron stable/ussuri: Don't setup bridge controller if it is already set  https://review.opendev.org/c/openstack/neutron/+/81634911:13
opendevreviewSlawek Kaplonski proposed openstack/neutron stable/train: Don't setup bridge controller if it is already set  https://review.opendev.org/c/openstack/neutron/+/81645911:16
opendevreviewSlawek Kaplonski proposed openstack/neutron stable/stein: Don't setup bridge controller if it is already set  https://review.opendev.org/c/openstack/neutron/+/81647011:16
opendevreviewSlawek Kaplonski proposed openstack/neutron stable/rocky: Don't setup bridge controller if it is already set  https://review.opendev.org/c/openstack/neutron/+/81647111:16
opendevreviewSlawek Kaplonski proposed openstack/neutron stable/queens: Don't setup bridge controller if it is already set  https://review.opendev.org/c/openstack/neutron/+/81647211:17
opendevreviewOleg Bondarev proposed openstack/neutron master: Add Local IP Extension and DB  https://review.opendev.org/c/openstack/neutron/+/80452311:51
opendevreviewOleg Bondarev proposed openstack/neutron master: Add Local IP L2 extension  https://review.opendev.org/c/openstack/neutron/+/80711611:51
opendevreviewOleg Bondarev proposed openstack/neutron master: Add Local IP L2 extension flows  https://review.opendev.org/c/openstack/neutron/+/81510211:51
opendevreviewOleg Bondarev proposed openstack/neutron master: Add Local IP Association policy rules  https://review.opendev.org/c/openstack/neutron/+/81643511:51
opendevreviewOleg Bondarev proposed openstack/neutron master: [WIP] Add Local IP fullstack test case  https://review.opendev.org/c/openstack/neutron/+/81632711:51
opendevreviewOleg Bondarev proposed openstack/neutron-lib master: Add Local IP constants needed for policy checks  https://review.opendev.org/c/openstack/neutron-lib/+/81646811:54
opendevreviewDmitrii Shcherbakov proposed openstack/neutron-specs master: Off-path SmartNIC Port Binding with OVN  https://review.opendev.org/c/openstack/neutron-specs/+/78882112:06
dmitriislajoskatona: Thanks a lot for the review! Besides a +1 from the CI, I am assuming other cores need to review it before it can be WF+1-ed?12:24
opendevreviewLajos Katona proposed openstack/networking-bgpvpn stable/victoria: Make lower-constraints job non-voting  https://review.opendev.org/c/openstack/networking-bgpvpn/+/81650012:25
lajoskatonadmitriis: yes we need 2 +2 from cores and a wf+1 (same as in nova), I try to fetch Liu (liuyulong) as he has comments previously12:26
dmitriislajoskatona: ack, tyvm12:26
opendevreviewMerged openstack/neutron stable/wallaby: [DVR] Fix update of the MTU in the SNAT namespace  https://review.opendev.org/c/openstack/neutron/+/81293912:43
opendevreviewOleg Bondarev proposed openstack/neutron master: Add Local IP L2 extension  https://review.opendev.org/c/openstack/neutron/+/80711613:00
opendevreviewOleg Bondarev proposed openstack/neutron master: Add Local IP L2 extension flows  https://review.opendev.org/c/openstack/neutron/+/81510213:00
opendevreviewOleg Bondarev proposed openstack/neutron master: Add Local IP Association policy rules  https://review.opendev.org/c/openstack/neutron/+/81643513:00
opendevreviewOleg Bondarev proposed openstack/neutron master: [WIP] Add Local IP fullstack test case  https://review.opendev.org/c/openstack/neutron/+/81632713:00
opendevreviewMerged openstack/neutron stable/xena: [OVN Migration] Remove trunk's subports from the nodes  https://review.opendev.org/c/openstack/neutron/+/81543213:06
*** kmasterson is now known as Guest481113:13
opendevreviewOleg Bondarev proposed openstack/neutron master: Add Local IP L2 extension  https://review.opendev.org/c/openstack/neutron/+/80711613:13
opendevreviewOleg Bondarev proposed openstack/neutron master: Add Local IP L2 extension flows  https://review.opendev.org/c/openstack/neutron/+/81510213:13
opendevreviewOleg Bondarev proposed openstack/neutron master: Add Local IP Association policy rules  https://review.opendev.org/c/openstack/neutron/+/81643513:13
opendevreviewOleg Bondarev proposed openstack/neutron master: [WIP] Add Local IP fullstack test case  https://review.opendev.org/c/openstack/neutron/+/81632713:13
opendevreviewOleg Bondarev proposed openstack/neutron master: [WIP] Add Local IP fullstack test case  https://review.opendev.org/c/openstack/neutron/+/81632714:08
opendevreviewRodolfo Alonso proposed openstack/neutron stable/xena: Check a namespace existence by checking only its own directory  https://review.opendev.org/c/openstack/neutron/+/81651214:12
opendevreviewRodolfo Alonso proposed openstack/neutron stable/wallaby: Check a namespace existence by checking only its own directory  https://review.opendev.org/c/openstack/neutron/+/81651314:12
opendevreviewLajos Katona proposed openstack/networking-bgpvpn stable/victoria: Run TripleO jobs on CentOS8 instead of CentOS7  https://review.opendev.org/c/openstack/networking-bgpvpn/+/80950214:13
opendevreviewRodolfo Alonso proposed openstack/neutron stable/victoria: Check a namespace existence by checking only its own directory  https://review.opendev.org/c/openstack/neutron/+/81651514:14
opendevreviewRodolfo Alonso proposed openstack/neutron stable/ussuri: Check a namespace existence by checking only its own directory  https://review.opendev.org/c/openstack/neutron/+/81651614:15
opendevreviewRodolfo Alonso proposed openstack/neutron stable/train: Check a namespace existence by checking only its own directory  https://review.opendev.org/c/openstack/neutron/+/81651814:15
ralonsohslaweq, lajoskatona https://8bc8364a1ac612214bd7-a9cc155153d08dd30dfffbbf1d71d234.ssl.cf2.rackcdn.com/816459/1/check/neutron-tempest-plugin-scenario-linuxbridge-train/fb552c6/job-output.txt14:22
ralonsohdid you see this problem?14:22
ralonsoh2021-11-03 11:39:36.703634 | controller | + /opt/stack/neutron-tempest-plugin/tools/customize_ubuntu_image:mount_image:108 :   sudo -E guestmount -i --add /tmp/tmp.ymixy102zA/ubuntu-20.04-minimal-cloudimg-amd64.img --pid-file /tmp/tmp.ymixy102zA/pid /tmp/tmp.ymixy102zA/mount14:22
ralonsoh2021-11-03 11:39:36.708004 | controller | sudo: guestmount: command not found14:22
ralonsohI'll open a bug for this14:22
ralonsohI think we should first install guestmount in train14:23
lajoskatonaralonsoh: not yet14:23
lajoskatonaralonsoh: shall we add it to bindep.txt?14:25
ralonsohI think so, I'm checking it now14:25
lajoskatonaralonsoh: that is not directly for customers as I remember but for CI mostly....14:26
ralonsohright14:26
ralonsohlajoskatona, ok, this is already addressed in https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/81551814:31
slaweqralonsoh:  sorry, I just saw Your question, yes, it's already addressed by bcafarel :)14:33
bcafarelwhat did I break?14:33
ralonsohjust the opposite!14:34
bcafarelaah yes guestmount :)14:34
ralonsohexactly14:34
opendevreviewOleg Bondarev proposed openstack/neutron master: [WIP] Add Local IP fullstack test case  https://review.opendev.org/c/openstack/neutron/+/81632714:35
opendevreviewSlawek Kaplonski proposed openstack/neutron master: Cleanup router for which processing added router failed  https://review.opendev.org/c/openstack/neutron/+/81489214:37
gmannlajoskatona: slaweq ack on 807790, I will continue working on that soon15:01
lajoskatonagmann: thanks, I jut wanted to wander around qa channel :-)15:10
slaweqgmann: thx a lot15:11
opendevreviewOleg Bondarev proposed openstack/neutron master: Add Local IP L2 extension  https://review.opendev.org/c/openstack/neutron/+/80711615:20
opendevreviewOleg Bondarev proposed openstack/neutron master: Add Local IP L2 extension flows  https://review.opendev.org/c/openstack/neutron/+/81510215:20
opendevreviewOleg Bondarev proposed openstack/neutron master: Add Local IP Association policy rules  https://review.opendev.org/c/openstack/neutron/+/81643515:20
opendevreviewOleg Bondarev proposed openstack/neutron master: Add Local IP fullstack test case  https://review.opendev.org/c/openstack/neutron/+/81632715:20
opendevreviewOleg Bondarev proposed openstack/neutron master: Add Local IP fullstack test case  https://review.opendev.org/c/openstack/neutron/+/81632715:56
opendevreviewRodolfo Alonso proposed openstack/neutron master: [OVS][QOS] Dataplane enforcement is limited to min-bw egress direction  https://review.opendev.org/c/openstack/neutron/+/81653016:03
opendevreviewSlawek Kaplonski proposed openstack/neutron master: Add functional and fullstack jobs with FIPS enabled  https://review.opendev.org/c/openstack/neutron/+/81400916:14
opendevreviewRodolfo Alonso proposed openstack/neutron master: [WIP][OVS][QoS] Implement QoS for HW offload  https://review.opendev.org/c/openstack/neutron/+/81503716:26
opendevreviewMerged openstack/neutron stable/wallaby: [OVN Migration] Remove trunk's subports from the nodes  https://review.opendev.org/c/openstack/neutron/+/81543316:52
opendevreviewMerged openstack/os-vif master: Use TCP keepalives for ovsdb connections  https://review.opendev.org/c/openstack/os-vif/+/80522317:00
opendevreviewRodolfo Alonso proposed openstack/neutron master: [WIP][OVS][QoS] Implement QoS for HW offload, min-bw egress  https://review.opendev.org/c/openstack/neutron/+/81653717:01
opendevreviewMerged openstack/os-vif master: only register tables used by os-vif  https://review.opendev.org/c/openstack/os-vif/+/80562517:03
opendevreviewMerged openstack/networking-bgpvpn stable/ussuri: Run TripleO jobs on CentOS8 instead of CentOS7  https://review.opendev.org/c/openstack/networking-bgpvpn/+/80950318:30
dasp_Regarding OVN network without gateway, where DHCP would use metadata port's IP if one exists... is there any reason to create a proper DHCP port when metadata port does not exist, or is it okay to create a "metadata" port for that purpose (when no gateway) even when metadata is otherwise disabled?18:54
dasp_It seems wasteful to potentially have a DHCP port AND a metadata port, especially that a metadata port criteria is that... it actually can be a DHCP port as long as it's not used for DHCP, except it never is in OVN.18:55
dasp_not used for DHCP agent*18:55
dasp_This is RE https://bugs.launchpad.net/neutron/+bug/167066618:56
fricklertobias-urdin: zigo: added a question in https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/815265 , I'd be particularly interested in your opinion as operators/deployers19:07
opendevreviewDaniel Speichert proposed openstack/neutron master: [OVN] Support native DHCP service for subnets without gateway IP  https://review.opendev.org/c/openstack/neutron/+/70801120:27
opendevreviewMerged openstack/neutron stable/victoria: [OVN Migration] Remove trunk's subports from the nodes  https://review.opendev.org/c/openstack/neutron/+/81543422:10

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