Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2023-03-07
17:53:27 bauzas and a specific grenade-skip-level job that tests the previous SLURP release (or yoga in our case)
17:53:29 dansmith right, which for a slurp release will be the same for both jobs
17:53:38 bauzas correct
17:53:52 bauzas but that doesn't tell what a project should be testing
17:54:03 bauzas when we have a slurp release
17:54:18 bauzas the -always will always run and voting, as its name says :)
17:54:45 bauzas but at the beginning of C, we would then enable the grenade-skip-level job and make it voting too, then ?
17:54:59 dansmith no, because it would be the exact same as the -always job in C
17:55:02 dansmith there's no reason to run both
17:55:22 bauzas so we would just change zuul to use grenade-skip-level instead of -always ?
17:55:33 bauzas or not changing anything ? :)
17:55:35 dansmith I don't know how we could avoid inheriting the regular job from the template, but we can worry about that when C comes, IMHO
17:55:56 bauzas meh ok
17:56:18 dansmith we could just remove the -always job in the slurp releases since we will get the regular job automatically, but I'd prefer to avoid that, I just don't know the zuul-fu to make that happen, but it's a problem for later, IMHO :)
17:56:37 bauzas honestly, if you want MHO, now that we have an -always job that tests N-2 even on non-SLURP, I'm quite intended to leave it as it is, whether it's a slurp or not in the master branch :)
17:56:54 dansmith that's what I'm trying to say
17:56:58 bauzas and just pretend the grenade-skip-level job never existed :)
17:57:06 dansmith right, that's what we should do
17:57:16 bauzas cool, then I understand it better :)
17:57:37 dansmith I guess the skip-level job is not in the template, actually
17:57:49 dansmith I was thinking we'd inherit that from the template regardless, but we won't
17:58:08 dansmith so yes, all we need to do is enable -always and voting=true and we're good forever
17:58:51 bauzas all good
17:58:54 bauzas wfm
18:09:04 bauzas gmann: now we branched 2023.1, do you want to explicitly mark the branches on https://review.opendev.org/c/openstack/nova/+/875773 ?
18:41:34 gmann bauzas: no that is not needed as it is defined as '-always' now so we will open this to run on everywhere it is added. for example once we will have stable/2023.2 it will continue running there to rest stable/zed->stable/2023.2
18:42:57 gmann and setting of those and on future master will be taken care on grenade side
18:46:41 gmann dansmith: you mean grenade-skip-level-always to be always run and only skip level job we will have and if project want they can stop it to run on non-slurp they can do. That way grenade-skip-level will disappear ?
18:47:24 dansmith gmann: no I think if the project only wants to test skip-level on slurps, they use skip-level, if they want to always test N-2->master, they use skip-level-always
18:47:30 gmann and we can add grenade-skip-level-always in integrated gate as voting in SLURP release only
18:48:03 gmann dansmith: in SLURP grenade-skip-level and grenade-skip-level-always will be with same setting so why we cannot just keep grenade-skip-level-alwaysonly
18:48:12 gmann and remove grenade-skip-level completly
18:48:51 gmann I mean grenade-skip-level-always always do N-2 -> N upgrade and 1. project need to run it mandatory in SLURP 2. it is optional to run in non-SLURP
18:48:57 dansmith you mean have the job branch-limited in the template and make nova override branches to be "all" for that job?
18:49:53 gmann nova having it in check pipeline should run even integrated template does add it but this is something we can test as zuul crazy magic
18:50:33 gmann but idea is to handle 1.mandatory in SLURP 2. optional in non-SLURP can by handled via branch variant
18:50:52 gmann keeping both jobs will confuse people
18:54:13 gmann let me do and show the template and greande side changes once we will branch greande (maybe this or early next week) and we can see/test how that can work for both cases 1. project want to run it in non-SLURP 2. project do not
19:22:20 opendevreview David Hill proposed openstack/nova master: Wait for VM to be paused before cleaning up https://review.opendev.org/c/openstack/nova/+/876776
19:25:49 artom gmann, oh hey, seeing your name here reminds me to gently poke you for input on https://review.opendev.org/c/openstack/nova/+/875653
19:28:43 gmann artom: yeah I opened it few days back after seeing it from channel and it is in my list. I will try to do it today if not tomorrow for sure.
19:28:45 gmann thanks for ping
19:29:28 artom No huge rush, good to know you're aware :)
#openstack-nova - 2023-03-08
08:39:14 opendevreview Jorge San Emeterio proposed openstack/nova master: Adding a default schema for requests to the 'lock', 'migrate' and 'unshelve' actions. https://review.opendev.org/c/openstack/nova/+/875653
10:30:31 opendevreview Merged openstack/nova stable/2023.1: [stable-only] Update .gitreview for stable/2023.1 https://review.opendev.org/c/openstack/nova/+/876551
10:30:37 opendevreview Merged openstack/nova stable/2023.1: [stable-only] Update TOX_CONSTRAINTS_FILE for stable/2023.1 https://review.opendev.org/c/openstack/nova/+/876552
11:24:54 opendevreview Rajesh Tailor proposed openstack/nova master: Add functional regression tests for bug 1857306 https://review.opendev.org/c/openstack/nova/+/700456
11:24:55 opendevreview Rajesh Tailor proposed openstack/nova master: Handle InstanceExists exception for duplicate instance https://review.opendev.org/c/openstack/nova/+/860938
12:22:35 sean-k-mooney bauzas: i just updated https://blueprints.launchpad.net/nova/+spec/cpu-selection-with-hypervisor-consideration since we actully did it as a partial bugfix in antelope
12:22:51 sean-k-mooney bauzas: do we want to mark that as complete for antelope
12:22:57 sean-k-mooney i added the details to the whiteboard
12:29:58 sean-k-mooney bauzas: i dont think this need to be in the release highlights or anything just wanted you to know and see if we wanted to set the blueprint as complete.
12:56:31 bauzas sean-k-mooney: done
13:41:26 opendevreview Sylvain Bauza proposed openstack/nova master: Update min support for Bobcat https://review.opendev.org/c/openstack/nova/+/875621
14:58:39 opendevreview David Hill proposed openstack/nova master: Wait for VM to be paused before cleaning up https://review.opendev.org/c/openstack/nova/+/876776
15:28:24 opendevreview David Hill proposed openstack/nova master: Wait for VM to be paused before cleaning up https://review.opendev.org/c/openstack/nova/+/876776
15:46:32 bauzas dansmith: fwiw, I haven't rebased https://review.opendev.org/c/openstack/nova/+/875621 up from your own change, given I eventually thought it wasn't needed
18:17:00 opendevreview Sylvain Bauza proposed openstack/nova-specs master: Move Antelope implemented specs https://review.opendev.org/c/openstack/nova-specs/+/876887
18:40:27 opendevreview Sylvain Bauza proposed openstack/nova-specs master: fix count_blueprints.py to work with new release naming https://review.opendev.org/c/openstack/nova-specs/+/876888
#openstack-nova - 2023-03-09
03:13:17 opendevreview OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.opendev.org/c/openstack/nova/+/876931
07:08:39 opendevreview Rajesh Tailor proposed openstack/nova master: Fix duplicate cell creation with same name https://review.opendev.org/c/openstack/nova/+/876940
08:37:42 dvo-plv Hello, Could you please help me with zuul regression. https://review.opendev.org/c/openstack/nova/+/876075/ I have two components, nova depends on os-traits. I use Depends-on variable to link them, but it does not work for me.
09:02:16 opendevreview Merged openstack/nova master: Imported Translations from Zanata https://review.opendev.org/c/openstack/nova/+/876931
09:34:46 elodilles bauzas: hi, fyi, reno workaround was released and upper constraints was bumped, so this is good to go now: https://review.opendev.org/c/openstack/nova/+/876553
10:48:19 gibi elodilles: +2
10:52:16 opendevreview Danylo Vodopianov proposed openstack/nova master: Packed virtqueue support was added. https://review.opendev.org/c/openstack/nova/+/876075
10:56:56 elodilles thx :]
10:58:14 bauzas elodilles: sent to the gate
10:58:41 bauzas Uggla: btw. not sure you've seen my comment https://review.opendev.org/c/openstack/releases/+/876758
10:59:01 bauzas Uggla: you probably need to update both your Gerrit config and your git ownership
10:59:16 Uggla Hello bauzas , not yet I will have a look
11:00:41 bauzas Uggla: tl;dr your git user.email is different from your gerrit config
11:01:31 bauzas and make sure you have a full name in https://review.opendev.org/settings/#Profile
11:01:38 Uggla bauzas, yes I use my personal one for the fondation and gerrit
11:02:07 bauzas Uggla: cool, so please changing your git user.email value for this repo
11:03:00 bauzas I'll myself update then the liaison email to be your personal email
11:03:15 Uggla bauzas, ok I will check after lunch.
11:05:19 bauzas Uggla: elodilles: updated https://review.opendev.org/c/openstack/releases/+/876758
11:26:53 elodilles bauzas: +2'd. let me know if we can merge that.
12:19:56 sean-k-mooney elodilles: from my perspective yes that is suitable to merge
12:41:07 ratailor_ sean-k-mooney, elodilles bauzas gibi can I get reviews on my patches https://review.opendev.org/c/openstack/nova/+/873901 https://review.opendev.org/c/openstack/nova/+/860938 https://review.opendev.org/q/Id4c8c5f3b32985ac7d3d7c833b82e0876f7367c1 https://review.opendev.org/q/Ia738a0972b050f549f446c85171d3f33e60ada4f saome are under review since long time.
12:42:06 sean-k-mooney we are currently minimising the patches we merge to master until after the final RC deadline
12:42:13 sean-k-mooney which is march 20th
12:43:06 sean-k-mooney we technically could merge this now but we likely will wait until after that
12:43:20 ratailor_ sean-k-mooney, ack. sure. Thanks!
12:43:32 sean-k-mooney ill see if i can find tiem to review but we will likely hold +w for another week or so
12:43:56 ratailor_ sean-k-mooney, ack. np.
12:50:15 bauzas we can merge some bugfixes if we want, that's not a problem
12:51:47 dvo-plv Hello, Could you please help me with zuul regression. https://review.opendev.org/c/openstack/nova/+/876075/ I have two components, nova depends on os-traits. I use Depends-on variable to link them, but it does not work for me.
12:52:26 dvo-plv I found that it install os-trait from pip
12:52:27 dvo-plv 2023-03-09 11:15:37.630 | Collecting os-traits===2.10.0 2023-03-09 11:15:37.634 | Downloading https://mirror-int.dfw.rax.opendev.org/pypifiles/packages/77/ce/66b9fc86ab69e4ad60d07901e9324cb44d72c75b9b0a989bfef0fc4ad4e3/os_traits-2.10.0-py3-none-any.whl (41 kB)
12:52:38 dvo-plv this log from here
12:52:39 dvo-plv https://6e5de493453a596bf54b-e12a251c5f6363a4d35eb8aac39c4442.ssl.cf2.rackcdn.com/876075/3/check/tempest-integrated-compute-ubuntu-focal/270fa66/controller/logs/devstacklog.txt
12:55:09 sean-k-mooney if its not listed as a required project depends on does not work
12:55:18 sean-k-mooney i would guess that that is the isseu there with that job
12:57:08 sean-k-mooney looking at the parent of the job that is indeed the case
12:58:25 sean-k-mooney you could add it as a required project here https://opendev.org/openstack/tempest/src/branch/master/zuul.d/integrated-gate.yaml#L146-L166
12:59:14 sean-k-mooney you woudl add
12:59:18 sean-k-mooney required-projects:
12:59:28 sean-k-mooney - openstack/os-traits

Earlier   Later