Wednesday, 2024-08-28

opendevreviewMerged openstack/networking-bagpipe master: Add os_ken_app to EVPN OVSDataplaneDriver  https://review.opendev.org/c/openstack/networking-bagpipe/+/88469500:59
opendevreviewGhanshyam proposed openstack/neutron master: Remove default override for RBAC config options  https://review.opendev.org/c/openstack/neutron/+/92608502:59
opendevreviewGhanshyam proposed openstack/neutron master: DNM debugging designate job  https://review.opendev.org/c/openstack/neutron/+/92694502:59
opendevreviewGhanshyam proposed openstack/neutron master: Remove default override for RBAC config options  https://review.opendev.org/c/openstack/neutron/+/92608503:03
opendevreviewGhanshyam proposed openstack/neutron master: DNM debugging designate job  https://review.opendev.org/c/openstack/neutron/+/92694503:03
opendevreviewMerged openstack/ovn-octavia-provider stable/2024.1: Maintenance task: do not change IPv4 ip_port_mappings  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/92671309:01
opendevreviewMerged openstack/ovn-octavia-provider stable/2023.1: Maintenance task: do not change IPv4 ip_port_mappings  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/92671509:01
opendevreviewMerged openstack/ovn-octavia-provider stable/2023.2: Maintenance task: do not change IPv4 ip_port_mappings  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/92671409:01
opendevreviewSlawek Kaplonski proposed openstack/neutron master: WIP Add trusted vif api extension for the port  https://review.opendev.org/c/openstack/neutron/+/92606809:21
opendevreviewElvira García Ruiz proposed openstack/neutron master: Use oslo netutils for get_noscope_ipv6 function  https://review.opendev.org/c/openstack/neutron/+/92737110:10
opendevreviewMerged openstack/neutron stable/2024.1: Get ips from system dns resolver without scope  https://review.opendev.org/c/openstack/neutron/+/92729210:49
mlavalleralonsoh_, slaweq, lajoskatona if you have a chance, please look at https://review.opendev.org/c/openstack/neutron/+/917800. Thanks in advance11:03
ralonsoh_ok11:03
*** ralonsoh_ is now known as ralonsoh11:04
mlavalle:-)11:04
opendevreviewMerged openstack/neutron stable/2023.1: Get ips from system dns resolver without scope  https://review.opendev.org/c/openstack/neutron/+/92729411:36
opendevreviewyatin proposed openstack/neutron master: [DNM] Check ovn-ovs source jobs git submodule  https://review.opendev.org/c/openstack/neutron/+/92738111:40
opendevreviewElvira García Ruiz proposed openstack/neutron master: Use oslo netutils for get_noscope_ipv6 function  https://review.opendev.org/c/openstack/neutron/+/92737112:18
opendevreviewMiro Tomaska proposed openstack/neutron stable/2023.2: [ovn] Add support for IPv6 metadata  https://review.opendev.org/c/openstack/neutron/+/92732412:24
opendevreviewLajos Katona proposed openstack/networking-bagpipe master: Doc: remove sphinxcontrib-*diag from doc dependencies  https://review.opendev.org/c/openstack/networking-bagpipe/+/92738412:38
*** ralonsoh__ is now known as ralonsoh12:42
opendevreviewLajos Katona proposed openstack/networking-bgpvpn master: Doc: remove sphinxcontrib-*diag from doc dependencies  https://review.opendev.org/c/openstack/networking-bgpvpn/+/92739013:19
ralonsohslaweq, ykarel if you have 1 min: https://review.opendev.org/c/openstack/neutron/+/92723613:29
ralonsohthis is not solving the issue but adding an extra check (done in other FTs)13:29
lajoskatonahaleyb: stephenfin posted a mail to remove sphinxcontrib-*diag dependencies (https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/4IID4UEXY4PJJGBTMFMTRYLKJIN4GOQ6/ )13:30
lajoskatonahaleyb: I started to check that: https://review.opendev.org/q/topic:%22remove_sphinxcontrib%22 , neutron-specs also affected, please check if my method (convert the diag to png, and include that) is acceptable for all neutron projects and I can continue this topic13:31
haleyblajoskatona: hey. i had looked in neutron but didn't see specs, let me look at the change. what other neutron repos are affected?13:34
lajoskatonahaleyb: bagpipe, bgpvpn, taas, and specs as I see13:40
lajoskatonahaleyb: I pushed a first spec for bagpipe and bgpvpn, I suppose the same method can work for the others also (convert the diag to png and include that)13:40
opendevreviewRodolfo Alonso proposed openstack/neutron master: Add new "tagging" API method: create (POST)  https://review.opendev.org/c/openstack/neutron/+/92472414:03
opendevreviewSlawek Kaplonski proposed openstack/neutron master: WIP Add trusted vif api extension for the port  https://review.opendev.org/c/openstack/neutron/+/92606814:17
opendevreviewSlawek Kaplonski proposed openstack/neutron-lib master: Add port trusted vif extension  https://review.opendev.org/c/openstack/neutron-lib/+/92386014:28
opendevreviewMerged openstack/neutron-dynamic-routing master: Ignore disabled routers for advertising  https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/88256014:37
opendevreviewMerged openstack/neutron master: Remove remnants of NSX  https://review.opendev.org/c/openstack/neutron/+/92722414:43
opendevreviewTakashi Kajinami proposed openstack/neutron-vpnaas master: Remove redundant neutron prefix from ovn vpn agent config  https://review.opendev.org/c/openstack/neutron-vpnaas/+/90551915:05
ihrachysykarel: is there a reason why e.g job-templates or tempest-multinode are ignored for neutron-ovn-base jobs?15:21
ihrachysAFAIU I can change neutron-ovn-base configuration and no jobs from tempest* will trigger if they inherit from this template. seems wrong?15:22
ihrachysnoticed that because non-plugin ovs-master was not triggered here: https://review.opendev.org/c/openstack/neutron/+/92721915:22
opendevreviewIhar Hrachyshka proposed openstack/neutron master: Add ovs-master job with neutron-plugin enabled  https://review.opendev.org/c/openstack/neutron/+/92721915:26
haleybihrachys: probably due to the irrelevant-files block in the definition? it ignores everything except project.yaml15:36
ihrachyshaleyb: yes. q: is why do we have this15:48
ihrachysleaning towards just removing it but wonder if there's a reason for this. (if so it should probably be captured in a comment there)15:48
haleybihrachys: i think it just has to do with not running jobs unnecesarily, but ykarel would maybe give a better answer15:50
opendevreviewIhar Hrachyshka proposed openstack/neutron master: Introduce ovs-lts jobs that run against the latest ovn LTS branch  https://review.opendev.org/c/openstack/neutron/+/92722115:54
ihrachysI mean... I understand the intent of the filters; I just wonder if these filters are correct and don't skip legit jobs15:54
ykarelihrachys, the idea is to run only the jobs which changed in the PR16:03
ykarelif you change base, all child jobs will triggered16:03
ihrachysok; so let's say I touch ovn-tempest-base definition. and it's in singlenode yaml; and then - shouldn't all jobs that spin off this parent get triggered? because they are not I think.16:04
ykarelin that patch you have neither changed  neutron-ovn-base nor neutron-ovn-tempest-ovs-master so that didn't ran 16:04
ykarelyes if you change ovn-tempest-base, jobs inheriting it will get triggered16:04
ykarelirrespective of irrelevant-files set16:05
ihrachysohhh. irrespective? is there some alternative path that ignores the filters?16:05
ykarelyes16:06
ihrachysack. that's what I missed. how does it work?16:06
ykarelhttps://zuul-ci.org/docs/zuul/latest/config/job.html#attr-job.match-on-config-updates16:06
ihrachys thank you!16:08
*** ykarel is now known as ykarel|away16:09
ihrachyshaleyb: FYI the nested snat set of patches is ready for review since we confirmed it works (with necessary ovn patches)18:13
haleybihrachys: thanks, i'm slowly going through them now was a little backlogged18:14
haleybihrachys: and there seems to be a duplicate bug reported at https://review.opendev.org/c/openstack/devstack/+/838782 but with a slightly different workaround18:16
opendevreviewMerged openstack/neutron master: Make RPC event cast synchronous with the event  https://review.opendev.org/c/openstack/neutron/+/92692218:29
opendevreviewRodolfo Alonso proposed openstack/neutron master: Monkey patch the system libraries before calling them  https://review.opendev.org/c/openstack/neutron/+/92537619:00
opendevreviewMerged openstack/neutron-vpnaas master: Remove redundant neutron prefix from ovn vpn agent config  https://review.opendev.org/c/openstack/neutron-vpnaas/+/90551920:26
haleybihrachys: just tested, seems to work in various tests - fixed to fixed, fip to fip and other variants. Only thing I can't do is associate a floating IP to a VM on the nested network, but think there is an open bug for that (it's not directly reachable so the API barfs)20:56
ihrachyshaleyb: is it the RFE that florian reported some time ago?21:01
haleybihrachys: maybe, just seemed very familiar21:02
ihrachyshttps://bugs.launchpad.net/neutron/+bug/2072505 ?21:02
ihrachysyeah this is a different thing.21:02
ihrachyshaleyb: you pulled the ovn patch from Ales for fip-to-fip checks, or did you run with e.g. 23.09?21:04
haleybit seems to be the same issue to me21:04
haleybihrachys: i pulled his branch and built everything21:05
ihrachysack. just wanted to make sure I haven't missed anything and it somehow passed without the ovn patch. good then.21:06
haleybovn_northd|INFO|OVN internal version is : [24.09.90-20.37.0-73.6]21:07
haleybihrachys: and the running one seems to match the built one, i maybe should have tweaked the version to be sure21:18
ihrachysI am convinced by gate runs that the patch fixes the east-west scenarios with nested snat on; but having your confirmation is helpful21:20
haleybthe harder part is fixing it in 22.04.x if it's even possible21:22
haleybi need to take off, and am offline until Friday21:26
*** haleyb is now known as haleyb|out21:26
Artsiom_Hi! Is there a way to separate network namespaces for network devices on compute nodes? We're using OVS. I've found an option `ovs_use_veth`, can it support ports on compute nodes or it can be actual only on network nodes for fips/nat/dhcp? 22:43
opendevreviewMerged openstack/neutron-tempest-plugin master: [pagination tests] Do not fail on zero resources  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/92710323:08
opendevreviewMerged openstack/neutron master: [ML2/OVN] Add external_ids.neutron:is_static_route key for Static Routes  https://review.opendev.org/c/openstack/neutron/+/90748923:23
opendevreviewMerged openstack/neutron-tempest-plugin master: Remove unstable_test from FIP test_east_west  https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/61855723:43
opendevreviewMerged openstack/neutron master: Use oslo netutils for get_noscope_ipv6 function  https://review.opendev.org/c/openstack/neutron/+/92737123:43

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