| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-03-07 | |||
| 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 | |
| 14:38:14 | sean-k-mooney | bauzas: we alredy can do this | |
| 14:38:19 | sean-k-mooney | btu we intentionally dont | |
| 14:38:34 | bauzas | I know and I agree | |
| 14:38:40 | bauzas | hence the "could" | |
| 14:38:45 | sean-k-mooney | ack | |
| 14:39:20 | bauzas | dvo-plv: if you get stuck by a lagging os-traits, come to us | |
| 14:39:54 | sean-k-mooney | dvo-plv: the other thing you can do is for now just use a custome trait in the nova code and you can update it to the real one later but it woudl be simpler ot just approvhe the spec agains and merge the trait and do a release in 2 weeks | |
| 14:39:56 | bauzas | but I think we recently released the last traits update | |
| 14:40:16 | sean-k-mooney | bauzas: the new triat is not merged yet becuase the spec need to be aprpoved first | |