| ralonsoh | bcafarel, lajoskatona slaweq hi folks, please review https://review.opendev.org/q/topic:bug%252F2003455 | 08:31 |
|---|---|---|
| ralonsoh | thanks! | 08:31 |
| lajoskatona | ralonsoh: checking | 08:32 |
| ralonsoh | ykarel_, hey, do you know if in zuul there is a way to pass FORCE=yes? | 08:33 |
| ralonsoh | context: https://d6e1f78c8bf69cb17ff8-e9858f2cd7cb620f684618f2ae76d049.ssl.cf5.rackcdn.com/878441/2/check/neutron-ovn-grenade-multinode-skip-level/78cb3d0/job-output.txt | 08:33 |
| ralonsoh | grenade-multijobs are failing | 08:33 |
| ralonsoh | output: | 08:33 |
| ralonsoh | 2023-03-24 19:25:09.236208 | compute1 | + ./stack.sh:main:230 : SUPPORTED_DISTROS='bullseye|focal|f35|opensuse-15.2|opensuse-tumbleweed|rhel8|rhel9|openEuler-20.03' | 08:33 |
| ralonsoh | 2023-03-24 19:25:09.238442 | compute1 | + ./stack.sh:main:232 : [[ ! jammy =~ bullseye|focal|f35|opensuse-15.2|opensuse-tumbleweed|rhel8|rhel9|openEuler-20.03 ]] | 08:33 |
| ralonsoh | 2023-03-24 19:25:09.240679 | compute1 | + ./stack.sh:main:233 : echo 'WARNING: this script has not been tested on jammy' | 08:33 |
| ralonsoh | 2023-03-24 19:25:09.245168 | compute1 | + ./stack.sh:main:235 : die 235 'If you wish to run this script anyway run with FORCE=yes' | 08:33 |
| frickler | ralonsoh: you'd need more patches in devstack to support jammy. what base branch is that? | 08:34 |
| ralonsoh | frickler, I'm checking that now, but it should be zed | 08:35 |
| ralonsoh | or at least this should be the purpose of skip-level | 08:35 |
| frickler | grenade_from_branch: stable/yoga | 08:35 |
| ykarel_ | ralonsoh, that is devstack var, so can be put in devstackdevstack_localrc: | 08:36 |
| ykarel_ | and zed should already support jammy | 08:36 |
| ralonsoh | ykarel_, yes but we are then checking out other version | 08:36 |
| ralonsoh | (yoga most probably) | 08:37 |
| ykarel_ | hmm right, so it's general problem with skip level jobs | 08:38 |
| frickler | yoga doesn't support jammy. zed isn't slurp. so you should test 2023.1 to master at best | 08:38 |
| ralonsoh | ykarel_, we have this hardcoded in our jobs | 08:38 |
| ralonsoh | grenade_from_branch: stable/yoga | 08:38 |
| ralonsoh | I've seen that now | 08:38 |
| ralonsoh | I'll push a patch | 08:38 |
| ralonsoh | thanks folks!! | 08:38 |
| ykarel_ | frickler, but with 2023.1 to master there is no release in b/w shouldn't those job be zed to master(skipping 2023.1)? | 08:40 |
| bcafarel | ralonsoh: I wonder about reverts in stable strategy, is it better to revert each indepedently (like you did) which causes different change-ids, or backport the revert from master itself (same change-id, but probably no"Revert Created" tag)? | 08:40 |
| ykarel_ | i mean grenade_from_branch: stable/zed | 08:41 |
| frickler | ykarel_: no, skip-level-upgrades are only done from every other release | 08:41 |
| ralonsoh | bcafarel, maybe it will be better to revert and backport | 08:41 |
| frickler | maybe there shouldn't be a skip-level job for this cycle at all | 08:41 |
| frickler | need to check with gmann, who masters this all | 08:41 |
| ralonsoh | bcafarel, actually no if there are differences in the backports | 08:42 |
| ralonsoh | frickler, it was decided to have during Antelope a skip-level job | 08:42 |
| ralonsoh | and even in the comments of these jobs it is said that | 08:42 |
| bcafarel | TBH specific reverts are fine for me :) I don't remember recent examples to see how we did (we did some partial reverts but not full I think) | 08:42 |
| ralonsoh | target release. Right now, this is xena because master is zed. | 08:42 |
| ralonsoh | # When master is A, this should become yoga, and so forth. | 08:42 |
| bcafarel | ralonsoh: ack good point on the differences, going to +2 the series | 08:43 |
| ralonsoh | bcafarel, thanks! | 08:43 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Bump skip-level lower version to stable/zed https://review.opendev.org/c/openstack/neutron/+/878632 | 08:44 |
| ykarel_ | i remember a revert all the way :) https://review.opendev.org/q/I159460be27f2c5f105be4b2865ef84aeb9a00094 | 08:44 |
| frickler | Current master is a non SLURP release so we do not need to run | 08:47 |
| frickler | greande-skip-level job as default | 08:47 |
| frickler | from https://review.opendev.org/c/openstack/tempest/+/878000 | 08:47 |
| ralonsoh | this is why is only in the check queue and voting:false | 08:48 |
| ralonsoh | but maybe we can move it to experimental and periodic | 08:48 |
| ralonsoh | I'll bring this topic to the next neutron meeting | 08:48 |
| ralonsoh | lajoskatona, sorry for pinging you again | 08:57 |
| ralonsoh | can you check https://review.opendev.org/c/openstack/neutron/+/878479/ and https://review.opendev.org/c/openstack/neutron/+/878526? | 08:57 |
| ralonsoh | thanks! | 08:57 |
| opendevreview | Frode Nordahl proposed openstack/neutron master: Add extra router attributes for ECMP and BFD https://review.opendev.org/c/openstack/neutron/+/874797 | 09:19 |
| *** bpetermann is now known as Guest9011 | 09:45 | |
| opendevreview | Luis Tomas Bolivar proposed openstack/neutron master: Ensure redirect-type=bridged not used for geneve networks https://review.opendev.org/c/openstack/neutron/+/878450 | 10:00 |
| ralonsoh | fnordahl, qq about https://review.opendev.org/c/openstack/neutron-lib/+/870887/11/neutron_lib/exceptions/l3_ext_gw_multihoming.py | 10:07 |
| ralonsoh | should we add the router ID on these exception messages? | 10:07 |
| fnordahl | ralonsoh: good question, that information is definitively available in the context where they are used and would definitively be useful for the operator. I'll have a peek at adding those, thank you for pointing it out | 10:17 |
| ralonsoh | fnordahl, ping me if you push a new PS. I think the patch is almost ready | 10:17 |
| fnordahl | 👍 | 10:18 |
| opendevreview | Frode Nordahl proposed openstack/neutron-lib master: ext-gw-multihoming: api-def and api-ref https://review.opendev.org/c/openstack/neutron-lib/+/870887 | 10:45 |
| opendevreview | Frode Nordahl proposed openstack/neutron master: Allow Multiple External Gateways https://review.opendev.org/c/openstack/neutron/+/873593 | 11:05 |
| opendevreview | Frode Nordahl proposed openstack/neutron master: Add extra router attributes for ECMP and BFD https://review.opendev.org/c/openstack/neutron/+/874797 | 11:05 |
| fnordahl | ralonsoh: new neutron-lib patch set up, also updated the two first patches of the implementation series so that we get a test run with the updated neutron-lib patch set | 11:07 |
| ralonsoh | cool, I'll check them today | 11:07 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Change name of ``_TestIsSessionActive`` test case. https://review.opendev.org/c/openstack/neutron/+/878641 | 11:15 |
| lajoskatona | ykarel, slaweq, ralonsoh: things are strange in zuul, like: https://zuul.opendev.org/t/openstack/status#877944 | 11:52 |
| lajoskatona | ykarel, slaweq, ralonsoh: neutron-tempest-plugin-ovn queued (95 hr 25 min) | 11:53 |
| lajoskatona | ykarel, slaweq, ralonsoh: I don't know if other runs has similar issue | 11:53 |
| ralonsoh | I'll check the zuul queue | 12:00 |
| ralonsoh | in any case, just rebase the patch to force the refresh | 12:01 |
| ralonsoh | lajoskatona, I'm going to push a new PS (changing the text) | 12:40 |
| ralonsoh | is that ok? | 12:40 |
| ralonsoh | I can't rebase, it is on top of master already | 12:41 |
| lajoskatona | ralonsoh: sure | 12:46 |
| lajoskatona | ralonsoh: strange situation | 12:47 |
| ralonsoh | yeah I've seen that before with other patches | 12:47 |
| ralonsoh | this is not strictly related to the job in particular | 12:47 |
| ykarel_ | ralonsoh, so only neutron plugin looks impacted, seems there was some issue around that time which was resolved | 13:10 |
| ralonsoh | nice to know, thanks! | 13:10 |
| ykarel_ | will check with infra just to be sure | 13:11 |
| *** ykarel_ is now known as ykarel | 13:21 | |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-tempest-plugin master: Skip stateless SG related tests for backends which don't support it https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/877944 | 13:27 |
| opendevreview | Frode Nordahl proposed openstack/neutron-specs master: Active-active L3 Gateway with Multihoming https://review.opendev.org/c/openstack/neutron-specs/+/870030 | 13:32 |
| opendevreview | Merged openstack/neutron master: Cleanup before executing "test_get_all_devices" https://review.opendev.org/c/openstack/neutron/+/878219 | 14:24 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: rbacs: filter out model that are already owned by context https://review.opendev.org/c/openstack/neutron/+/871113 | 15:08 |
| opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: rbacs: fix typo, s/cxt/ctx https://review.opendev.org/c/openstack/neutron/+/872762 | 15:09 |
| ralonsoh | lajoskatona, do you know what is the meeting passcode? | 16:02 |
| lajoskatona | ralonsoh: I was able to join without passcode | 16:19 |
| lajoskatona | but just arrived :-) | 16:19 |
| ralonsoh | lajoskatona, I installed the app | 16:19 |
| ralonsoh | lajoskatona, why don't you talk about the friday's meeting? | 16:39 |
| ralonsoh | to atract new cores | 16:40 |
| lajoskatona | good idea | 16:50 |
| opendevreview | Brian Haley proposed openstack/neutron master: Update the quota guide examples https://review.opendev.org/c/openstack/neutron/+/878720 | 21:41 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!