Thursday, 2024-02-15

slaweqralonsoh bcafarel lajoskatona hi, can You check backports https://review.opendev.org/q/topic:%22bug/2028846-stable/2023.2%22 and https://review.opendev.org/q/topic:%22bug/2028846-stable/2023.1%22 ?07:53
slaweqthx in advance07:53
lajoskatonaslaweq: checking08:03
slaweqlajoskatona: thx08:03
opendevreviewLajos Katona proposed openstack/tap-as-a-service master: GRE/ERSPAN mirroring for taas  https://review.opendev.org/c/openstack/tap-as-a-service/+/88535708:37
opendevreviewLajos Katona proposed openstack/tap-as-a-service master: Tap Mirror OVS driver  https://review.opendev.org/c/openstack/tap-as-a-service/+/89308608:37
opendevreviewLajos Katona proposed openstack/tap-as-a-service master: TAAS tap-mirror OVN driver  https://review.opendev.org/c/openstack/tap-as-a-service/+/89350908:37
opendevreviewTakashi Kajinami proposed openstack/ovn-octavia-provider master: Fix ovs branch used with ovn main branch  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/90907408:47
tkajinam^^^ hope this fixes the broken ovn-octavia-provider-functional-master08:48
tkajinamexample: https://zuul.opendev.org/t/openstack/build/b2b1126d99874600a6613c1a98dc65c108:48
opendevreviewSlawek Kaplonski proposed openstack/neutron master: [S-RBAC] Change policies for port's binding:profile field  https://review.opendev.org/c/openstack/neutron/+/90907508:58
opendevreviewTakashi Kajinami proposed openstack/ovn-octavia-provider master: ovn master: Switch to newer ovs stable branch  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/90907409:02
bcafarelslaweq: looks like Lajos already did all the work :)09:03
slaweqthx lajoskatona :)09:03
slaweqand thx bcafarel for looking into it also :)09:03
opendevreviewLajos Katona proposed openstack/tap-as-a-service master: CLI for Tap Mirrors  https://review.opendev.org/c/openstack/tap-as-a-service/+/88608509:03
lajoskatonabcafarel: not alone :-)09:05
slaweqbbezak hi, can You test Your Ironic patch with Service role enabled on top of the https://review.opendev.org/c/openstack/neutron/+/909075 ?09:33
slaweqthx in advance09:33
bbezakI'll take a look thx slaweq. Unfortunately glance/nova didn't competed their service role support part, therefore ironic user can't have only service role in caracal (for now)10:03
slaweqbbezak I will need to update some more unit tests which are failing with my patch10:04
slaweqI will ping You once this will be done10:04
slaweqbut You can try to run Your job to test my patch - fixes in UT shouldn't imact it10:04
slaweq*impact10:04
opendevreviewLajos Katona proposed openstack/tap-as-a-service master: Documentation for tap-mirrors  https://review.opendev.org/c/openstack/tap-as-a-service/+/90722010:08
bbezakok, thx slaweq10:16
opendevreviewMerged openstack/neutron stable/2023.2: [OVN] Warn about invalid OVN and FIP PF config during start of Neutron  https://review.opendev.org/c/openstack/neutron/+/90879810:31
opendevreviewMerged openstack/neutron stable/2023.2: Log warning about port forwardings that won't work properly  https://review.opendev.org/c/openstack/neutron/+/90879910:31
opendevreviewMerged openstack/neutron stable/2023.2: [Docs] Add info about incompatible PF configuration in ML2/OVN gaps  https://review.opendev.org/c/openstack/neutron/+/90880010:31
opendevreviewMerged openstack/neutron stable/2023.1: [OVN] Warn about invalid OVN and FIP PF config during start of Neutron  https://review.opendev.org/c/openstack/neutron/+/90882210:31
opendevreviewMerged openstack/neutron stable/2023.1: Log warning about port forwardings that won't work properly  https://review.opendev.org/c/openstack/neutron/+/90882310:31
opendevreviewMerged openstack/neutron stable/2023.1: [Docs] Add info about incompatible PF configuration in ML2/OVN gaps  https://review.opendev.org/c/openstack/neutron/+/90882410:31
opendevreviewSlawek Kaplonski proposed openstack/neutron master: [S-RBAC] Change policies for port's binding:profile field  https://review.opendev.org/c/openstack/neutron/+/90907511:39
ralonsohslaweq, sorry, today I'm on PTO13:36
ralonsohI'll check the patches tomorrow morning!13:37
slaweqralonsoh no worries, I didn't notice that when I was asking for reviews :)13:37
slaweqtake Your time13:37
ralonsohhehehe thanks13:37
slaweqand all backports are already merged13:37
opendevreviewSlawek Kaplonski proposed openstack/neutron master: [S-RBAC] Change policies for port's binding:profile field  https://review.opendev.org/c/openstack/neutron/+/90907513:42
*** jph5 is now known as jph13:45
bbezakslaweq: change of yours looks good - port bindings update by ironic user with only service role is successful - https://review.opendev.org/c/openstack/kolla-ansible/+/909121 - it is failing on glance not supporting service role yet - therefore we can't get rid of admin just yet - https://review.opendev.org/c/openstack/kolla-ansible/+/90858017:37
opendevreviewBrian Haley proposed openstack/neutron master: Fix disallowed-name warnings  https://review.opendev.org/c/openstack/neutron/+/90914021:53
opendevreviewBrian Haley proposed openstack/neutron master: Fix global-variable-not-assigned warnings  https://review.opendev.org/c/openstack/neutron/+/90914121:58
opendevreviewBrian Haley proposed openstack/neutron master: Fix import-outside-toplevel warnings  https://review.opendev.org/c/openstack/neutron/+/90914222:01
opendevreviewBrian Haley proposed openstack/neutron master: Fix misplaced-bare-raise warning  https://review.opendev.org/c/openstack/neutron/+/90914322:03
opendevreviewBrian Haley proposed openstack/neutron master: Fix pointless-string-statement warnings  https://review.opendev.org/c/openstack/neutron/+/90914422:06
opendevreviewBrian Haley proposed openstack/neutron master: Fix redundant-keyword-arg warnings  https://review.opendev.org/c/openstack/neutron/+/90914522:08
opendevreviewBrian Haley proposed openstack/neutron master: Fix undefined-loop-variable warnings  https://review.opendev.org/c/openstack/neutron/+/90914622:11
opendevreviewBrian Haley proposed openstack/neutron master: Fix unidiomatic-typecheck warnings  https://review.opendev.org/c/openstack/neutron/+/90914722:12
opendevreviewBrian Haley proposed openstack/neutron master: Fix unnecessary-comprehension warnings  https://review.opendev.org/c/openstack/neutron/+/90914822:14
opendevreviewBrian Haley proposed openstack/neutron master: Fix used-before-assignment warnings  https://review.opendev.org/c/openstack/neutron/+/90914922:16
opendevreviewBrian Haley proposed openstack/neutron master: Fix useless-else-on-loop warning  https://review.opendev.org/c/openstack/neutron/+/90915022:18
opendevreviewMiguel Lavalle proposed openstack/neutron master: Enable HA for OVN router flavors  https://review.opendev.org/c/openstack/neutron/+/90151323:39

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