| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-03-07 | |||
| 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 | |
| 13:04:29 | opendevreview | Jorge San Emeterio proposed openstack/nova master: Re-enable "test_tagged_attachment" test. https://review.opendev.org/c/openstack/nova/+/876699 | |
| 13:06:45 | opendevreview | Merged openstack/nova master: Update master for stable/2023.1 https://review.opendev.org/c/openstack/nova/+/876553 | |
| 13:41:25 | dvo-plv | Sean, do I need to fix all gates or only for integrated-compute. Because it looks like depends-on veriable does not work for all tempest jobs | |
| 14:26:01 | sean-k-mooney | it will work only for required proejct im not sure we want to supprot it in general for os-vif | |
| 14:26:06 | sean-k-mooney | *os-traits | |
| 14:26:14 | sean-k-mooney | tis somethign we shoud discuss | |
| 14:26:41 | sean-k-mooney | if we were to fix it for all tempet jobs we woudl do so in the devstack base job | |
| 14:27:03 | sean-k-mooney | however os-traits is a libary and we normally want ot only use that from pypi release | |
| 14:27:29 | sean-k-mooney | dvo-plv: it is expected that your code will fail until an os-traits release is doen | |
| 14:28:01 | sean-k-mooney | i woudl focuse on the reapoval of the spec and we can do an os-traits release with teh traits after march 20th | |
| 14:28:28 | sean-k-mooney | that will fix all the jobs with out always installing os-traits form git | |
| 14:29:19 | sean-k-mooney | if we add os-traits as a required proejct it will always beinstalled form git instead of the most recent release even when we dont have a depens-on | |
| 14:29:27 | sean-k-mooney | which is not what we want in general | |
| 14:32:10 | bauzas | sean-k-mooney: well, we could ask devstack to run our librairies from git | |
| 14:32:23 | bauzas | but I'm not super OK if this is about ci jobs | |
| 14:32:36 | bauzas | it's simplier to release a library if needed | |