Wednesday, 2025-03-12

opendevreviewTakashi Kajinami proposed openstack/tap-as-a-service master: Use generated config files in devstack  https://review.opendev.org/c/openstack/tap-as-a-service/+/92987204:32
opendevreviewyatin proposed x/whitebox-neutron-tempest-plugin master: [DNM] Check debug false tempest  https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/94408505:11
opendevreviewyatin proposed x/whitebox-neutron-tempest-plugin master: Revert "Fetch only SGL log entries, drop count assertion"  https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/94408605:28
vsaienkohello neutron team, do you know what was the reason to do not host external port on the same chassis where VM sits? https://github.com/openstack/neutron/commit/770914f98df9abda2b5b8ca2896ff05618ea0181#diff-f2c894a40a002f3c9b2591bd0619f65b9582f0b9001a6564284479b0acd1f170R1166-R117306:40
ralonsohvsaienko, https://dani.foroselectronica.es/ovn-external-ports-604/. The traffic on an external port (and in this case I think you are referring to SR-IOV), doesn't hit the local OVN bridge. It is needed that another chassis (GW chassis or with ``enable-chassis-as-extport-host``) hosts this port to receive and reply DHCP, IGMP or routing06:51
vsaienkoIt depends on the deployment architecture I think in case A it will hit OVN bridge https://paste.openstack.org/show/bpVnLKVp3FkU9JFKv7Et/ rigth?07:00
vsaienkobut I got the point, thanks. 07:01
ralonsohvsaienko, SR-IOV it never hits the local OVN, the traffic egress the VM using the VF. It can "goes back" to the local OVN bridge if that is connected to the physnet and is a GW07:02
ralonsohso you can make this compute as a GW chassis07:03
ralonsohbut the egress traffic doesn't hit OVN because this traffic doesn't use OVS07:03
vsaienkobut in case A dhcp request from VM will go out of eth1 and than switch will flood broadcast to eth0, which will not happen in caseB (as switches do not flood broadcast to same port where request come from to avoid loops)07:04
ralonsohvsaienko, again, the VF traffic egress the VM07:06
ralonsohwhatever you want to do with this traffic depends on the infra config07:06
ralonsohthis traffic does NOT hit the local OVN 07:06
ralonsohyou need additional infra config to redirect again this traffic to the same host07:07
vsaienkoyes exactly07:07
opendevreviewVasyl Saienko proposed openstack/neutron master: Add note in sync_ha_chassis_group_network method  https://review.opendev.org/c/openstack/neutron/+/94409107:27
opendevreviewMaor Blaustein proposed x/whitebox-neutron-tempest-plugin master: Limit logging in limit tests  https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/94409307:35
opendevreviewRodolfo Alonso proposed openstack/neutron master: WIP == [OVN] Delete a resource provider removed from the agent config  https://review.opendev.org/c/openstack/neutron/+/94405308:07
ralonsohslaweq, hello! just a few seconds, already approved before08:10
ralonsohhttps://review.opendev.org/c/openstack/neutron/+/94366408:10
ralonsohif you have another minute: https://review.opendev.org/c/openstack/neutron/+/94394208:10
ralonsohthanks in advance08:11
slaweqralonsoh hi08:13
slaweqboth approved08:13
ralonsohthanks!08:13
opendevreviewMaor Blaustein proposed x/whitebox-neutron-tempest-plugin master: Limit logging in security group logging tests  https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/94409308:34
opendevreviewMaor Blaustein proposed x/whitebox-neutron-tempest-plugin master: Limit logging in security group logging tests  https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/94409308:45
opendevreviewFernando Royo proposed openstack/ovn-octavia-provider master: Add sync floating IP support  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/92903908:53
opendevreviewMaor Blaustein proposed x/whitebox-neutron-tempest-plugin master: Limit logging in security group logging tests  https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/94409309:00
opendevreviewMaor Blaustein proposed x/whitebox-neutron-tempest-plugin master: Limit logging in security group logging tests  https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/94409309:00
opendevreviewRodolfo Alonso proposed openstack/neutron master: Reimplement the singleton class  https://review.opendev.org/c/openstack/neutron/+/94409909:06
opendevreviewMaor Blaustein proposed x/whitebox-neutron-tempest-plugin master: Limit logging in security group logging tests  https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/94409309:28
opendevreviewMaor Blaustein proposed x/whitebox-neutron-tempest-plugin master: Limit logging in security group logging tests  https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/94409310:12
opendevreviewRodolfo Alonso proposed openstack/neutron master: [OVN][Placement] Reduce log level to error when Placement exception  https://review.opendev.org/c/openstack/neutron/+/94410110:23
opendevreviewRodolfo Alonso proposed openstack/neutron master: [OVN] Delete a resource provider removed from the agent config  https://review.opendev.org/c/openstack/neutron/+/94405310:28
opendevreviewMerged openstack/neutron master: [OVN] Do not call ``QoSAddCommand`` without max-bw/DSCP rule  https://review.opendev.org/c/openstack/neutron/+/94366410:29
opendevreviewMerged openstack/neutron master: [fullstack] Re-implement ``wait_until_true`` with eventlet  https://review.opendev.org/c/openstack/neutron/+/94394210:29
opendevreviewSlawek Kaplonski proposed openstack/neutron master: [S-RBAC] Fix policies for the SG rules API  https://review.opendev.org/c/openstack/neutron/+/94402210:36
opendevreviewFernando Royo proposed openstack/ovn-octavia-provider master: Add sync floating IP support  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/92903910:56
ralonsohslaweq, https://5671b50e1757693146fd-d2676880cd4ce07846d9d5d9e5ab28d1.ssl.cf1.rackcdn.com/944022/4/check/openstack-tox-py312/6d88a78/testr_results.html11:04
ralonsohit is failing in py312 and py3911:04
slaweqdamn, I didn't check that one :/11:20
slaweqI will fix it right now11:20
ralonsohcool!11:20
opendevreviewSlawek Kaplonski proposed openstack/neutron master: [S-RBAC] Fix policies for the SG rules API  https://review.opendev.org/c/openstack/neutron/+/94402211:36
tafkamaxI have a question regarding external/internal networking in neutron. We set up our second cluster and for some reason the provider networks we use are working correctly, but the external/internal is not working. Where... (full message at <https://matrix.org/oftc/media/v1/media/download/AW40TWas3p1nI7ZYaY76H3PgTZogxdcCPMZGqrNFhbaBzwCBM379YWEcnI5p246f8NSnpyTv-2TCYFPM-vptfUZCeV0kXkXQAG1hdHJpeC5vcmcvWWpEdlR5TGFaRnVQWFhtQ0JKUERHV0ZK>)11:45
tafkamaxit sends out the request, but gets nothing back11:46
tafkamaxin the compute node where the neutron also resides it looks like this: 13:20:44.517076 tap9381a27d-6e P   IP 10.100.0.100 > 192.168.100.1: ICMP echo request, id 18522, seq 10, length 6411:47
opendevreviewRodolfo Alonso proposed openstack/neutron master: Reimplement the singleton class  https://review.opendev.org/c/openstack/neutron/+/94409912:06
opendevreviewRodolfo Alonso proposed openstack/neutron master: [OVN][Placement] Reduce log level to error when Placement exception  https://review.opendev.org/c/openstack/neutron/+/94410112:07
opendevreviewRodolfo Alonso proposed openstack/neutron master: [OVN] Delete a resource provider removed from the agent config  https://review.opendev.org/c/openstack/neutron/+/94405312:07
opendevreviewRodolfo Alonso proposed openstack/neutron-tempest-plugin master: Revert "Make neutron-tempest-plugin-designate-scenario non-voting temporary"  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/94411412:28
opendevreviewRodolfo Alonso proposed openstack/neutron-tempest-plugin master: Revert "Make neutron-tempest-plugin-designate-scenario non-voting temporary"  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/94411412:37
haleybralonsoh: https://review.opendev.org/c/openstack/neutron/+/944099 has a pep8 error12:37
haleybso all the children do too :(12:37
ralonsohlet me check12:38
haleybi have a zuul status plugin that shows me live status in the changes12:40
opendevreviewRodolfo Alonso proposed openstack/neutron master: DNM - Test errors in ``neutron-functional-with-pyroute2-master``  https://review.opendev.org/c/openstack/neutron/+/94285912:42
ralonsohI'll check this after lunch12:42
opendevreviewMaor Blaustein proposed x/whitebox-neutron-tempest-plugin master: Wait in both log start track/retrieve  https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/94392212:54
opendevreviewVasyl Saienko proposed openstack/ovn-octavia-provider master: Prepare to handle ha_chassis_group for LRP  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/94324313:02
*** elodilles is now known as elodilles_pto13:22
*** whoami-rajat_ is now known as whoami-rajat13:55
opendevreviewRodolfo Alonso proposed openstack/neutron-tempest-plugin master: Make ``neutron-tempest-plugin-*-ubuntu-jammy`` no voting  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/94411914:25
opendevreviewMerged openstack/ovn-octavia-provider master: Add sync floating IP support  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/92903914:35
*** svinota is now known as svinota[afk]14:45
opendevreviewTobias Urdin proposed openstack/neutron master: Add new default policy for device_id field on ports  https://review.opendev.org/c/openstack/neutron/+/86116915:08
opendevreviewTakashi Kajinami proposed openstack/tap-as-a-service master: Use generated config files in devstack  https://review.opendev.org/c/openstack/tap-as-a-service/+/92987215:11
spatelralonsoh Hello! 15:22
ralonsohspatel, hello15:25
spatelIf you remember our previous conversation related setting priority on chassis. 15:26
spatelhttps://paste.opendev.org/show/bOmV9K3x7ogilW1yIO49/15:26
spatelMy goal is to set os-chn-gen-comp-001 & os-chn-gen-comp-002 have higher priority so they serve DHCP for sriov 15:27
ralonsohok15:28
spatelI set priority manually and life was good but recently I noticed something happened in OVN and they switch the leaders and that reset priority back to lower on those two chassis 15:28
spatelis this normal behavior?15:28
spatelHow do I make they permanent ?15:29
spatels/they/them/15:30
ralonsohspatel, OVN doesn't change the HA_Chassis.priority field15:30
ralonsohthat should be Neutron15:31
ralonsohdid you remove any chassis register from the SB database/15:31
ralonsoh?15:31
spatelBut priority are part of OVN right?15:31
spatelNo, I didn't make any change15:31
ralonsohOVN uses the defined priority on these registers, but this is set by the user (Neutron in this case)15:32
spateltwo days ago I found SRIOV stopped receive IP address and turn out priority issue15:32
spatelHow do I tell neutron to set high priority of A and B chassis parmanently? 15:33
ralonsohFirst, it is needed to find who is changing this. Second, Neutron does not have this API to set the priority per chassis, it is assigned randomly15:34
spatelIn logs I have noticed OVN leader changed and look like that caused priority number reset 15:34
spatelI am building LAB to test that. I thought OVN reset priority. 15:35
ralonsohno, OVN doesn't change a user option15:36
spatelhmm! 15:36
ralonsohthe only place in Neutron where this is changed is in ``ChassisEvent``15:36
ralonsohand that could be caused because a GW chassis has been removed from the DB15:37
spatelYou are saying if I restart ovn-controller that may change priority right?15:37
ralonsohit can be removed and added again, but this event will happen eventually15:37
ralonsohyes, the ovn-controller graceful stop deletes the chassis register15:37
spatelFor that workaround I put it in crontab to set priority back to what I need.. 15:38
spatelits ugly hack but that is all I have to fix it 15:38
spatelI wish there are a way to set it in file so it will be always desire value no matter add or remove of chassis 15:39
ralonsohbut if you remove a chassis, you can't expect a ha_chassis register to be point there15:41
ralonsohto be pointing there*15:41
spatelI know! but what if I want to re-join with desire priority 15:54
ralonsohspatel, this is not exactly what you look for, but could be also a solution15:59
ralonsohfor external ports there is also a tag for the chassis16:00
ralonsohthat will make this chassis to be used only by these ports16:00
ralonsohone sec16:00
ralonsohenable-chassis-as-extport-host16:01
ralonsohhttps://docs.openstack.org/neutron/2024.2/admin/ovn/external_ports.html16:01
ralonsohso you can mark these 2 GWs with 'enable-chassis-as-extport-host'16:02
spatelralonsoh I read about that and also tried it to set enable-chassis-as-extport-host but it was causing some issue with metadata service and then my host started getting IP but not setting hostname and cloud-init etc..16:02
spatelwhen I did enable-chassis-as-gw that solved the issue.. 16:03
ralonsohspatel, then open a bug with this issue16:03
spatelHmm! 16:03
ralonsohspatel, what was the problem using 'enable-chassis-as-extport-host'16:03
ralonsoh?16:03
ralonsohyou need to deploy the metadata agent on these hosts too16:04
spatelwhen I replaced enable-chassis-as-gw with enable-chassis-as-extport-host then I have noticed that vms getting IP but metadata service was not reachable somehow...16:04
ralonsohthe metadata agent needs to be spawned in these hosts too16:05
spatelI am using kolla-ansible and it does deploy those service 16:05
ralonsohwhere the external port is going to be bound16:05
spatelI have to create entire lab and test again...16:06
ralonsohI think you can also use both tags, "enable-chassis-as-extport-host,enable-chassis-as-gw"16:06
ralonsohI never tried this16:06
spatelHmm! really?16:06
ralonsohbut if there is one chassis with this tag, then the ha_chassis_group will use it only16:07
ralonsohI need to go now16:07
spatelYou are saying if I set enable-chassis-as-extport-host that will override all the priority stuff right?16:07
ralonsohplease open a LP bug with the issue above16:07
spatelralonsoh Thank you for the help! 16:07
spatelI will 16:07
opendevreviewRodolfo Alonso proposed openstack/neutron master: Reimplement the singleton class  https://review.opendev.org/c/openstack/neutron/+/94409916:08
ralonsohotherwiseguy, ^^ please check this16:08
ralonsohI need to go now16:08
opendevreviewMerged x/whitebox-neutron-tempest-plugin master: Limit logging in security group logging tests  https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/94409316:11
opendevreviewMerged x/whitebox-neutron-tempest-plugin master: Wait in both log start track/retrieve  https://review.opendev.org/c/x/whitebox-neutron-tempest-plugin/+/94392217:16
opendevreviewMerged openstack/neutron master: [OVN] Retrieve the HA_Chassis_Group in ``_sync_ha_chassis_group``  https://review.opendev.org/c/openstack/neutron/+/94368918:04
opendevreviewMerged openstack/tap-as-a-service master: Use generated config files in devstack  https://review.opendev.org/c/openstack/tap-as-a-service/+/92987218:33
-opendevstatus- NOTICE: One of our Zuul job log storage providers is experiencing errors. We have removed that storage target from base jobs. You should be able to safely recheck changes now.20:23
opendevreviewBrian Haley proposed openstack/tap-as-a-service unmaintained/2023.1: Update .gitreview for unmaintained/2023.1  https://review.opendev.org/c/openstack/tap-as-a-service/+/93511321:15
opendevreviewMerged openstack/neutron-dynamic-routing master: Use constants from neutron-lib  https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/92949022:25
opendevreviewRodolfo Alonso proposed openstack/neutron master: Reimplement the singleton class  https://review.opendev.org/c/openstack/neutron/+/94409922:30
opendevreviewRodolfo Alonso proposed openstack/neutron master: [OVN][Placement] Reduce log level to error when Placement exception  https://review.opendev.org/c/openstack/neutron/+/94410122:30
opendevreviewRodolfo Alonso proposed openstack/neutron master: [OVN] Delete a resource provider removed from the agent config  https://review.opendev.org/c/openstack/neutron/+/94405322:31
opendevreviewRodolfo Alonso proposed openstack/neutron stable/2024.2: [OVN] Retrieve the HA_Chassis_Group in ``_sync_ha_chassis_group``  https://review.opendev.org/c/openstack/neutron/+/94417022:36
opendevreviewRodolfo Alonso proposed openstack/neutron-tempest-plugin master: Revert "Make neutron-tempest-plugin-designate-scenario non-voting temporary"  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/94411423:03
opendevreviewBrian Haley proposed openstack/neutron-dynamic-routing unmaintained/2023.1: Update .gitreview for unmaintained/2023.1  https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/93509023:06
opendevreviewBrian Haley proposed openstack/networking-bgpvpn unmaintained/2023.1: Update .gitreview for unmaintained/2023.1  https://review.opendev.org/c/openstack/networking-bgpvpn/+/93508523:09
opendevreviewRodolfo Alonso proposed openstack/neutron master: DNM - Test errors in ``neutron-functional-with-pyroute2-master``  https://review.opendev.org/c/openstack/neutron/+/94285923:33

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