| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-03-07 | |||
| 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 | |
| 14:40:24 | bauzas | sean-k-mooney: maybe we could have an experimental job that would pull os-traits from git | |
| 14:40:39 | bauzas | if that helps | |
| 14:40:43 | sean-k-mooney | that wont help they wanted to get a clen ci run | |
| 14:40:48 | bauzas | sean-k-mooney: ah I see, that's for a new feature | |
| 14:41:00 | sean-k-mooney | yep https://review.opendev.org/c/openstack/nova/+/876075/ | |
| 14:41:36 | dvo-plv | I will re-execute all tests locally to be sure that all works properly and will wait for spec approve and os-traits merge. Thnak you | |
| 14:41:38 | sean-k-mooney | dvo-plv: for now you could just use a custom trait and or comment it out | |
| 14:42:01 | bauzas | is it for helping to implement ? | |
| 14:42:07 | bauzas | or is it for us reviewing ? | |
| 14:42:31 | bauzas | if the former, nothing prevents you to test your patch against a running devstack which pulls ostraits from git | |
| 14:42:35 | sean-k-mooney | i.e more the request filter and traits reproting to a seocnd patch | |
| 14:43:13 | bauzas | if the latter, add a bit fat comment in the commit msg saying the CI is broken because of the missing release | |
| 14:43:26 | sean-k-mooney | https://review.opendev.org/c/openstack/nova/+/876075/3/nova/scheduler/request_filter.py is just brekign things currently | |
| 14:43:35 | sean-k-mooney | since os_traits.COMPUTE_NET_VIRTIO_PACKED does not exist yet | |
| 14:43:54 | dvo-plv | yes, exactly | |
| 14:43:54 | bauzas | yeah got it | |