| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-09-14 | |||
| 09:12:20 | auniyal_ | O/ | |
| 09:12:29 | opendevreview | Eigil Obrestad proposed openstack/nova master: Config parameters for min/max cpu/memory allocation. https://review.opendev.org/c/openstack/nova/+/857595 | |
| 09:12:30 | auniyal_ | please review this -https://review.opendev.org/c/openstack/nova/+/852737 | |
| 09:13:14 | auniyal_ | https://review.opendev.org/c/openstack/nova/+/857339 | |
| 09:14:54 | obre | gibi: I submitted a change to illustrate what I consider to be the simplest solution. Ill have a look at your three alternatives in a little while. First I need to eat and do a little bit of other work. | |
| 09:15:23 | gibi | obre: sure :) | |
| 09:25:54 | opendevreview | Ghanshyam proposed openstack/osc-placement master: Switch to 2023.1 Python3 unit tests and generic template name https://review.opendev.org/c/openstack/osc-placement/+/856787 | |
| 09:28:53 | opendevreview | Ghanshyam proposed openstack/os-vif master: Switch to 2023.1 Python3 unit tests and generic template name https://review.opendev.org/c/openstack/os-vif/+/856783 | |
| 09:29:18 | opendevreview | Ghanshyam proposed openstack/python-novaclient master: Switch to 2023.1 Python3 unit tests and generic template name https://review.opendev.org/c/openstack/python-novaclient/+/856791 | |
| 10:15:22 | obre | gibi: To "try to put it on the agenda of the next nova meeting"; is that simply updating the agenda on the wikipage? | |
| 10:15:57 | gibi | obre: there is an #topic Open discussion and under that you can add your topic | |
| 11:32:04 | opendevreview | Takashi Natsume proposed openstack/nova master: Update min supported service version for Zed https://review.opendev.org/c/openstack/nova/+/856895 | |
| 11:32:51 | sean-k-mooney | bauzas: gibi can we land this before RC1 https://review.opendev.org/c/openstack/nova/+/831844 | |
| 11:33:15 | sean-k-mooney | its addign the centos 9 fips job to the weekly periodic | |
| 11:33:24 | sean-k-mooney | pipelien and experimental | |
| 11:33:42 | sean-k-mooney | its finally passing again so it woudl be nice to include that so it can run on stable | |
| 11:34:10 | sean-k-mooney | without having to backport it explcitly to stable when the branch is cut | |
| 11:34:30 | sean-k-mooney | gmann:^ you have reviewd that before | |
| 11:36:48 | gmann | sean-k-mooney: +W | |
| 11:37:00 | sean-k-mooney | gmann++ many thanks | |
| 11:37:51 | sean-k-mooney | gmann: did you see my question here https://review.opendev.org/c/openstack/os-vif/+/856783/2#message-3aaeaa39501922e15ae5bfebb78fdf8e5c4c262c | |
| 11:38:38 | sean-k-mooney | are we goign to branch the openstack-python3-jobs via job vairiant | |
| 11:38:53 | sean-k-mooney | to make sure that the jobs in that template do not change on stable branches | |
| 11:41:19 | gmann | sean-k-mooney: yes, replied, like this we can do - https://review.opendev.org/c/openstack/openstack-zuul-jobs/+/856903/2/zuul.d/project-templates.yaml#1600 | |
| 11:41:41 | gmann | we will pin the changed job for stable and master once new release and change in python version jobs | |
| 11:44:22 | sean-k-mooney | ack thats all i needed to know. i assuemd ye would use job variant and the branches key | |
| 11:46:33 | sean-k-mooney | gmann: so with this change if we do it on nova and all the other compute deliverable we never need to review/merge these defualt job template paches again right :) | |
| 11:46:57 | sean-k-mooney | its just the central review going forward for the job variant | |
| 11:55:35 | gmann | sean-k-mooney: right. we do not need to update template name in nova side | |
| 11:56:10 | sean-k-mooney | that will be nice | |
| 11:56:26 | sean-k-mooney | its not like this takes much time but less manual stuff every release it a good thing | |
| 13:09:31 | bauzas | cores, can we please review https://review.opendev.org/c/openstack/nova/+/857467 (prelude) | |
| 13:09:37 | bauzas | this becomes critical | |
| 13:09:46 | bauzas | sean-k-mooney: gibi: others | |
| 13:09:53 | sean-k-mooney | oh the prelude | |
| 13:09:57 | sean-k-mooney | sure ill do it now | |
| 13:10:08 | bauzas | gracias and danke | |
| 13:10:15 | sean-k-mooney | i tought we had already merge that but it was the release highlights i was thinking of | |
| 13:10:31 | sean-k-mooney | for marketing | |
| 13:10:59 | bauzas | correct | |
| 13:11:18 | bauzas | cycle highlights go to some other website | |
| 13:11:37 | bauzas | here, the prelude is just for ops not wanting to read the long list of things we have in the relnotes | |
| 13:11:41 | sean-k-mooney | -1 i think you ment zed https://review.opendev.org/c/openstack/nova/+/857467/1/releasenotes/notes/zed-prelude-a3cddb8b2ac8e293.yaml | |
| 13:12:08 | sean-k-mooney | otherwise i think it looks ok to me | |
| 13:13:04 | sean-k-mooney | if you fix that im +1 | |
| 13:13:13 | sean-k-mooney | or +2 i guess | |
| 13:13:56 | sean-k-mooney | also set review priorty lable on that | |
| 13:14:17 | sean-k-mooney | bauzas: you were on pto when i had this converstaion with gibi and stephenfin | |
| 13:14:31 | bauzas | sean-k-mooney: I can respin quickly | |
| 13:14:43 | sean-k-mooney | bauzas: but the tl;dr is that review priorty is a littel weird for core so i use it slightly differnt then the doc | |
| 13:15:16 | bauzas | sean-k-mooney: honestly, I was about to write a topic for the PTG agenda about the use of review-prio | |
| 13:15:18 | sean-k-mooney | bauzas: if i set +1 it means im looking at it but im not askign other cores to look at it if i set +2 it means im looking at it and would like other cores to look at it too | |
| 13:15:24 | bauzas | and how we could make this better | |
| 13:15:48 | sean-k-mooney | so im using +1 to comunicate to the author that they have my attention | |
| 13:16:04 | sean-k-mooney | and +2 to comunciate to the core team that i think its imporant for them to look at too | |
| 13:16:18 | opendevreview | Sylvain Bauza proposed openstack/nova master: Prelude section for Zed release https://review.opendev.org/c/openstack/nova/+/857467 | |
| 13:16:26 | sean-k-mooney | im not commiting them to but usign it to signal asyc that i think its more important for wider review | |
| 13:16:27 | bauzas | sean-k-mooney ack, then let's discuss this at the PTG | |
| 13:16:38 | bauzas | sean-k-mooney: gibi: updated prelude ^ | |
| 13:16:38 | sean-k-mooney | cool | |
| 13:16:51 | sean-k-mooney | im trying to decople priorty from quality | |
| 13:17:15 | sean-k-mooney | i.e. rp +2 my be with a buch of -1 but may have topic that need wider input | |
| 13:19:54 | sean-k-mooney | bauzas: when will that move to zed https://6bc7844fa7b995ed5b77-cdf523d3b16150ab0b9ddc512a79d512.ssl.cf2.rackcdn.com/857467/1/check/build-openstack-releasenotes/b8db3f7/docs/unreleased.html should that also be in the patch or is it when we cut the stable branch | |
| 13:20:21 | bauzas | sean-k-mooney: ah | |
| 13:20:24 | sean-k-mooney | stephenfin: ^ pbr has some magic based on commit messages right it it tied into that or do we need to do somethign esle | |
| 13:20:30 | bauzas | you mean when it will be told "zed" | |
| 13:20:37 | sean-k-mooney | yep | |
| 13:20:46 | bauzas | sean-k-mooney: that's because reno works with git branches | |
| 13:20:49 | sean-k-mooney | this shoudl be rended under zed in rc1 right | |
| 13:20:55 | bauzas | for the moment, we haven't released zed | |
| 13:21:03 | bauzas | even with rc1 | |
| 13:21:12 | sean-k-mooney | so on stable zed this will render under zed automatically | |
| 13:21:20 | sean-k-mooney | have you checked that locally? | |
| 13:21:21 | bauzas | yup, because it branches | |
| 13:21:40 | bauzas | no, this is just the releases team which creates the correct new pages | |
| 13:21:57 | bauzas | https://6bc7844fa7b995ed5b77-cdf523d3b16150ab0b9ddc512a79d512.ssl.cf2.rackcdn.com/857467/1/check/build-openstack-releasenotes/b8db3f7/docs/unreleased.html#relnotes-25-0-0-195 will be seen |
|
| 13:23:31 | sean-k-mooney | ok +2 then i captured that in the review message | |
| 13:23:42 | bauzas | sean-k-mooney: example of a release patch creating the yoga page https://github.com/openstack/nova/commit/d2cba990a6205e5b0361e750c173850cf209a9e6 | |
| 13:24:22 | sean-k-mooney | right i tought we needed one like that i woudl prefer to do that before rc1 | |
| 13:24:36 | sean-k-mooney | we can do it after but i find that messier | |
| 13:24:39 | bauzas | https://review.opendev.org/c/openstack/nova/+/833243 was the patch | |
| 13:25:30 | sean-k-mooney | yep so why are we not doing that in the prelude patch or at least proposing it now | |
| 13:25:32 | bauzas | as you see in the "included in", this was merged *after* yoga GA | |
| 13:25:35 | sean-k-mooney | so we can merge both together | |
| 13:25:41 | bauzas | you can't | |
| 13:25:45 | sean-k-mooney | right i think that is wrong | |
| 13:25:52 | bauzas | prelude has to be merged before RC1 | |
| 13:26:07 | sean-k-mooney | yes and after RC1 master is A | |
| 13:26:10 | opendevreview | Merged openstack/nova master: Test setting the nova job to centos-9-stream https://review.opendev.org/c/openstack/nova/+/831844 | |
| 13:26:12 | bauzas | correct | |
| 13:26:18 | sean-k-mooney | so we would have to backport it to stable | |
| 13:26:25 | sean-k-mooney | i would like to avoid that backport | |
| 13:26:30 | bauzas | correct, we did it once and we had hard times with ops | |
| 13:26:35 | sean-k-mooney | by merging it at RC1 | |
| 13:26:40 | bauzas | we merged prelude after RC1 | |
| 13:26:43 | sean-k-mooney | and using it as the base of the rc tag | |
| 13:26:48 | bauzas | in Newton IIRC | |
| 13:26:58 | bauzas | that's the usual thing we do | |
| 13:27:10 | bauzas | but prelude can be merged a bit earler | |
| 13:27:34 | sean-k-mooney | yep that fin i would just like to also incldue the other patch in rc1 | |