| opendevreview | Jay Jahns proposed openstack/neutron master: Add network dns_domain to dns assignment if present https://review.opendev.org/c/openstack/neutron/+/920459 | 00:27 |
|---|---|---|
| *** dasm is now known as Guest7895 | 00:38 | |
| opendevreview | Takashi Kajinami proposed openstack/neutron-fwaas master: Fix migration script to adopt to SQLAlclehym 2.0 https://review.opendev.org/c/openstack/neutron-fwaas/+/920644 | 01:11 |
| opendevreview | Takashi Kajinami proposed openstack/neutron-fwaas master: DNM: Run migration tool https://review.opendev.org/c/openstack/neutron-fwaas/+/920672 | 01:12 |
| opendevreview | Brian Haley proposed openstack/neutron master: Change OVN client to account for nested networks https://review.opendev.org/c/openstack/neutron/+/907504 | 02:20 |
| opendevreview | Frode Nordahl proposed openstack/neutron-tempest-plugin master: Add Active Active L3 GW scenario test cases https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/898810 | 03:10 |
| opendevreview | Brian Haley proposed openstack/neutron-tempest-plugin master: Remove devstack-gate requirement https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/920708 | 03:21 |
| opendevreview | Brian Haley proposed openstack/neutron-fwaas master: Remove devstack-gate requirement https://review.opendev.org/c/openstack/neutron-fwaas/+/920709 | 03:38 |
| opendevreview | Brian Haley proposed openstack/neutron-tempest-plugin master: Remove devstack-gate requirement https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/920708 | 03:39 |
| lajoskatona | ykarel: Hi, for bagpipe py3.12 failure I opened a bug: https://bugs.launchpad.net/neutron/+bug/2067452 | 07:43 |
| lajoskatona | ykarel: but the issue itself seems to be coming from exabgp, so I opened and issue for it: https://github.com/Exa-Networks/exabgp/issues/1208 , as I see the code there are some refractoring in progress so perhaps in the coming release it will be fine | 07:44 |
| ykarel_ | thx lajoskatona | 07:46 |
| *** ykarel_ is now known as ykarel | 07:46 | |
| lajoskatona | ykarel_: I got a link from elodilles for unmaintained policies: https://docs.openstack.org/project-team-guide/stable-branches.html#unmaintained | 07:56 |
| ykarel | thx lajoskatona | 08:11 |
| ykarel | lajoskatona, slaweq when you get chance please check https://review.opendev.org/q/I447990509cdea9830228d3bc92a97062cc57a472 | 08:18 |
| ykarel | thx in advance | 08:18 |
| slaweq | ykarel done | 08:21 |
| opendevreview | Merged openstack/neutron stable/2024.1: Put monitors dictionary back in MetadataDriverBase class https://review.opendev.org/c/openstack/neutron/+/919871 | 09:04 |
| frickler | ykarel: (and others) fwaas and taas testing may be impacted by ds-gate removal https://zuul.opendev.org/t/openstack/config-errors?project=openstack%2Fneutron-tempest-plugin&name=Project+Not+Found&skip=0 | 09:49 |
| opendevreview | yatin proposed openstack/neutron stable/2024.1: [functional tests] compatibility with ovsdbapp>=2.6.1 https://review.opendev.org/c/openstack/neutron/+/920634 | 09:50 |
| frickler | oh, haleyb already proposed patches earlier | 09:50 |
| ykarel | yes i noticed there were some patches from brian | 09:50 |
| lajoskatona | frickler: thanks, I saw also haleyb-s patch for fwaas | 09:51 |
| opendevreview | yatin proposed openstack/neutron stable/2023.2: [functional tests] compatibility with ovsdbapp>=2.4.2 https://review.opendev.org/c/openstack/neutron/+/920635 | 09:52 |
| opendevreview | Takashi Kajinami proposed openstack/neutron-fwaas master: Remove devstack-gate https://review.opendev.org/c/openstack/neutron-fwaas/+/920727 | 09:52 |
| opendevreview | Takashi Kajinami proposed openstack/neutron-tempest-plugin master: Remove devstack-gate https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/920728 | 09:54 |
| frickler | sadly seems they are in a dependency loop. let me know if one of those should simple get force-merged | 09:54 |
| tkajinam | ah, ok you were already talking about the jobs messed up by devstack-gate retirement | 09:54 |
| tkajinam | I just noticed that tempest jobs do not run in fwaas and noticed the same | 09:55 |
| frickler | tkajinam: yes, seems you created duplicates of haleyb's patches? | 09:55 |
| tkajinam | yeah | 09:55 |
| opendevreview | yatin proposed openstack/neutron stable/2024.1: [functional tests] compatibility with ovsdbapp>=2.6.1 https://review.opendev.org/c/openstack/neutron/+/920634 | 09:56 |
| opendevreview | Merged openstack/neutron-fwaas master: Fix migration script to adopt to SQLAlclehym 2.0 https://review.opendev.org/c/openstack/neutron-fwaas/+/920644 | 09:57 |
| tkajinam | though a few stadium repos may need to be additionally covered. | 09:57 |
| tkajinam | such as neutron-fwaas, I mean | 09:57 |
| frickler | cores should take care not to merge things in the meantime, as they don't get proper testing currently. like e.g. https://review.opendev.org/c/openstack/neutron-fwaas/+/920644 | 09:59 |
| opendevreview | yatin proposed openstack/neutron stable/2023.2: [functional tests] compatibility with ovsdbapp>=2.4.2 https://review.opendev.org/c/openstack/neutron/+/920635 | 10:04 |
| tkajinam | frickler, fortunately that specific patch should be ok because it's not at all tested in functional jobs... though that's a different problem. | 10:11 |
| opendevreview | yatin proposed openstack/neutron stable/2023.1: [functional tests] compatibility with ovsdbapp>=2.2.2 https://review.opendev.org/c/openstack/neutron/+/920730 | 10:11 |
| ykarel | frickler, so can you force merge https://review.opendev.org/c/openstack/neutron-fwaas/+/920709 ? | 10:13 |
| opendevreview | Merged openstack/neutron stable/2024.1: [stable only] Do not fail on missing logical router ports https://review.opendev.org/c/openstack/neutron/+/920608 | 10:28 |
| sahid | o? | 10:29 |
| sahid | o/ | 10:29 |
| sahid | regarding notify behavior | 10:29 |
| sahid | in particular for nova | 10:29 |
| sahid | we can notice two mech to notify nova, one is using registry for port update | 10:30 |
| sahid | and the other sql alchemy, using hook | 10:30 |
| sahid | expecially for vif-plugged | 10:31 |
| sahid | basically an event network changed happen in all cases one a event vif-plugged happens, right? | 10:32 |
| sahid | @registry.receives(resources.PORT, [events.BEFORE_RESPONSE]) | 10:34 |
| sahid | have we already suffered issue with sqlalchemy not emitting event from hooks? | 10:40 |
| lajoskatona | sahid: Hi, I never heard such issue, are you using SQLAlchemy>2 and Neutron close to master or something older? | 11:00 |
| opendevreview | yatin proposed openstack/neutron master: [DNM] check cover job with swap https://review.opendev.org/c/openstack/neutron/+/920648 | 11:24 |
| opendevreview | Merged openstack/neutron stable/2023.1: [stable only] Do not fail on missing logical router ports https://review.opendev.org/c/openstack/neutron/+/920609 | 11:26 |
| opendevreview | Gaudenz Steinlin proposed openstack/neutron master: Use transient systemd units in Process fixture https://review.opendev.org/c/openstack/neutron/+/919834 | 11:34 |
| opendevreview | Gaudenz Steinlin proposed openstack/neutron master: Add L3 HA fullstack failover tests https://review.opendev.org/c/openstack/neutron/+/917429 | 11:34 |
| opendevreview | Gaudenz Steinlin proposed openstack/neutron master: Add conntrackd support to HA routers in L3 agent https://review.opendev.org/c/openstack/neutron/+/917430 | 11:34 |
| opendevreview | Merged openstack/neutron-fwaas master: Remove devstack-gate requirement https://review.opendev.org/c/openstack/neutron-fwaas/+/920709 | 11:37 |
| frickler | tkajinam: haleyb: ^^ done, let's see if the n-t-p change passes now | 11:38 |
| frickler | at least it is running jobs now | 11:41 |
| haleyb | frickler: oh, you had to force it? i was confused when it couldn't find a job definition | 12:26 |
| frickler | haleyb: yes, it was a cyclic dependency. one might have instead dropped the job reference and re-added later, but this was faster | 12:30 |
| haleyb | ah, ok, hopefully we can clean-up the rest think there are others, and there are still references to devstack-gate all over | 12:34 |
| haleyb | like $BASE/new/devstack-gate/devstack-vm-gate.sh | 12:36 |
| opendevreview | Lajos Katona proposed openstack/neutron-dynamic-routing master: netaddr1: except TypeError in validate_ip_addr https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/919790 | 12:56 |
| tkajinam | haleyb, yeah I noticed these references left in shell scripts. I guess we can drop these all, as I don't see any usage of these scripts really, but the guess still needs double-check. | 13:18 |
| haleyb | tkajinam: i don't know how it's not causing issues, but guess removing is the best plan | 13:20 |
| haleyb | lajoskatona: can you look at https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/920708 ? it passed and will un-block the n-t-p gate | 13:22 |
| opendevreview | Elod Illes proposed openstack/neutron-fwaas stable/2024.1: Remove devstack-gate requirement https://review.opendev.org/c/openstack/neutron-fwaas/+/920745 | 13:36 |
| lajoskatona | haleyb: done | 13:55 |
| haleyb | thanks! | 13:55 |
| lajoskatona | frickler, haleyb: the other project you mentioned was taas, but I can't see devstack-gate dependency there | 13:56 |
| frickler | lajoskatona: it was only in the neutron-tempest-plugin-tap-as-a-service job definition, should be fixed by 920708 | 14:00 |
| *** whoami-rajat_ is now known as whoami-rajat | 14:07 | |
| lajoskatona | frickler: ahh, ok | 14:12 |
| sahid | hum an other question regarding performance, i can see that the agent rpc_loop taking more than 300s some time | 14:15 |
| sahid | is there a way to improve performance of that part have we noted something | 14:15 |
| sahid | I think the issue is that the agent is intensivly using flow, perhaps siwtch to the old ipstable way would be better ... | 14:16 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: ovs: log warning when ovs is reported dead https://review.opendev.org/c/openstack/neutron/+/920753 | 14:44 |
| lajoskatona | sahid: 300s sounds high, do you use pure OVS, and which Neutron version? | 14:46 |
| sahid | yes we use ml2/ovs | 14:46 |
| sahid | lajoskatona: https://etherpad.opendev.org/p/ovs-elap | 14:47 |
| sahid | the version is ussuri ... | 14:48 |
| sahid | looks like it's related to the need to clean stale flows | 15:02 |
| opendevreview | Lucas Alvares Gomes proposed openstack/neutron-specs master: OVN BGP Agent EVPN Advertisement API Extension https://review.opendev.org/c/openstack/neutron-specs/+/920681 | 15:21 |
| opendevreview | Merged openstack/neutron-tempest-plugin master: Remove devstack-gate requirement https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/920708 | 15:55 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: ovs: log warning when ovs is reported dead https://review.opendev.org/c/openstack/neutron/+/920753 | 15:55 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron-lib master: api: canonicalize ipv6 addresses in `subnet:allocation_pools` field https://review.opendev.org/c/openstack/neutron-lib/+/918149 | 16:24 |
| opendevreview | Merged openstack/neutron-lib master: Always expect project_id and tenant_id to be in the attributes list https://review.opendev.org/c/openstack/neutron-lib/+/919588 | 17:10 |
| opendevreview | Brian Haley proposed openstack/neutron master: Temporarily add '--concurrency 4' to coverage job https://review.opendev.org/c/openstack/neutron/+/920766 | 17:52 |
| haleyb | ihrachys: ^^ temporary workaround | 18:28 |
| haleyb | running in a throw-away VM shows my two coverage processes growing forever, each at 20%/1.5G, so can see how it happens | 18:29 |
| opendevreview | Brian Haley proposed openstack/neutron-lib master: Revert "Always expect project_id and tenant_id to be in the attributes list" https://review.opendev.org/c/openstack/neutron-lib/+/920789 | 18:50 |
| ihrachys | haleyb ^ the need for this revert points out a problem in gate setup for neutron-lib - it doesn't gate on this job. (we could at least have an early notice if it would run as non-voting) | 19:29 |
| haleyb | ihrachys: right, even though it has the same job it failed to detect this issue. that's why i went for the revert instead of fixing the tests, plus i'm already fighting the other gate fire | 19:32 |
| ihrachys | yeah, I'm not putting more stuff on your plate, just pointing out. maybe yatin could check if we need a change in gate. | 19:32 |
| ihrachys | haleyb is it the same job though? I think neutron-lib job runs neutron-lib's unit tests; not neutron's | 19:35 |
| haleyb | i'm guessing the neutron-lib version should be doing an override-checkout of neutron so it tests with the latest | 19:36 |
| haleyb | ihrachys: the openstack-tox-py311-with-sqlalchemy-master job is defined in the neutron repo, but anyone can use it once it's defined, zuul magic :) | 19:36 |
| ihrachys | I see that the job in neutron-lib repo ran for 7 mins total; clearly it's not neutron suite (one can only dream!) | 19:37 |
| ihrachys | override-checkout won't do - it still runs different tests. it's not the same job (except the name) | 19:38 |
| haleyb | neutron should only need a lint job, we don't ever write buggy code | 19:39 |
| haleyb | oh right, it runs the neutron-lib unit tests, not neutron's | 19:41 |
| ihrachys | the very least, neutron-lib should non-vote the neutron test suite to give early warning | 19:41 |
| ihrachys | reported a bug for this https://bugs.launchpad.net/neutron/+bug/2067515 | 19:44 |
| haleyb | thanks | 19:45 |
| ihrachys | haleyb nice, the revert gets "fatal: Could not read from remote repository." timeouts :) | 19:46 |
| haleyb | that is at least temporary i hope... :( | 19:48 |
| haleyb | ihrachys: you might need to babysit that revert, i'm out early today, and it's still hitting infra issues | 21:20 |
| ihrachys | I'll try, thanks for the effort | 21:24 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!