| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-09-20 | |||
| 09:21:35 | zigo | Yeah, also this ... | |
| 09:21:41 | sean-k-mooney | The two remaining modules, 'api_models' and 'api_migrations', are | |
| 09:21:43 | sean-k-mooney | moved to the new 'nova.db.api' module. | |
| 09:21:55 | sean-k-mooney | so bauzas stephenfin created that module in august | |
| 09:22:00 | sean-k-mooney | but missed that file | |
| 09:22:33 | sean-k-mooney | bauzas: so this is a xena release regressions | |
| 09:23:23 | sean-k-mooney | zigo: devstack will be installing this more or less the same way as the distro | |
| 09:23:34 | zigo | Ok. | |
| 09:23:44 | zigo | Well, I don't know, but my patch needs to be merged ! :) | |
| 09:23:46 | sean-k-mooney | we do not install with -e in devstack | |
| 09:24:01 | sean-k-mooney | so it should be copying the files to the site-packages directory | |
| 09:24:04 | sean-k-mooney | and using it form there | |
| 09:24:19 | sean-k-mooney | zigo: did you file a bug | |
| 09:24:41 | zigo | For a single "touch __init__.py" ?!? Seriously ? | |
| 09:24:42 | zigo | :) | |
| 09:24:45 | sean-k-mooney | yes | |
| 09:24:45 | gibi | in the past (stable/wallaby) nova/db/api contained only the migration scripts that was always independently executed to the main nova services but now it contains code that nova services try to import. So I agree we need to fix this | |
| 09:24:54 | sean-k-mooney | zigo: we will need to do an RC2 | |
| 09:25:02 | zigo | Ok, filing the bug. | |
| 09:26:21 | gibi | zigo: thank you for catching this | |
| 09:29:53 | opendevreview | Thomas Goirand proposed openstack/nova master: Add missing __init__.py in nova/db/api https://review.opendev.org/c/openstack/nova/+/809980 | |
| 09:29:57 | zigo | There you go... | |
| 09:30:11 | zigo | Bug filled, PR closing it. | |
| 09:30:32 | zigo | I'm used to often do single char patches, this one will be ZERO chars ! :0 | |
| 09:32:03 | sean-k-mooney | zigo: technially master is now yoga, which is why we need the bug for backporting and if we want to do an RC2 | |
| 09:32:25 | sean-k-mooney | if you saw this last week before we created RC1 we proably could have just merged it | |
| 09:33:50 | zigo | sean-k-mooney: Ok, thanks for letting me know. Should I wait until my first patch is merged before opening the backport PR ? | |
| 09:34:25 | sean-k-mooney | am we should jsut be able to cherry pick this via the api so i think its safe to do it now | |
| 09:34:26 | zigo | Oh also, is there anything to know from the user's point of view about the sqla-migrate -> alembic switch? | |
| 09:34:47 | zigo | Or is it fully transparent? | |
| 09:34:48 | sean-k-mooney | i dont expect this to take long to review given its size :) | |
| 09:34:59 | sean-k-mooney | zigo: well other then the packageing impact no | |
| 09:35:11 | zigo | Ok, cheers. | |
| 09:35:27 | sean-k-mooney | i.e. the nova-mange command is still the same but you obvioulsy need alembic installed | |
| 09:38:45 | sean-k-mooney | zigo: ok that has the flags and marked it as critical since it blocks packaging https://bugs.launchpad.net/nova/+bug/1944111 | |
| 09:41:29 | zigo | The stable/xena branch is missing a defaultbranch=stable/rocky in the .gitreview file no ? | |
| 09:42:22 | gibi | https://review.opendev.org/c/openstack/nova/+/809759 | |
| 09:42:54 | gibi | the setup of stable/xena is not fully done yet as RC1 and the branch was cut last Friday | |
| 09:43:23 | zigo | Ok, so I guess I must wait for that one to merge ... :/ | |
| 09:45:56 | gibi | lyarwood, bauzas, elodilles: you you look at https://review.opendev.org/q/topic:create-xena+project:openstack/nova ? | |
| 09:46:06 | gibi | s/you/could/ | |
| 09:47:58 | lyarwood | Yup happy to | |
| 09:50:56 | bauzas | gibi: +Wd | |
| 09:51:57 | bauzas | fwiw, also +wd https://review.opendev.org/c/openstack/nova/+/809761/1 and the above one | |
| 09:52:05 | bauzas | so we will have the xena release notes | |
| 09:52:44 | bauzas | we miss a second core on https://review.opendev.org/c/openstack/nova/+/809762 | |
| 09:52:55 | opendevreview | Takashi Kajinami proposed openstack/nova master: Fix the wrong exception used to retry detach API calls https://review.opendev.org/c/openstack/nova/+/809934 | |
| 09:53:08 | gibi | bauzas: done | |
| 09:53:24 | bauzas | https://review.opendev.org/q/project:openstack/nova+owner:infra-root%2540openstack.org+is:open shows me all the xena paperwork for our jobs and reno are done | |
| 09:53:29 | bauzas | gibi: thanks | |
| 09:53:43 | bauzas | zigo: sorry was taxidriving my daughter from school | |
| 09:55:36 | bauzas | zigo: +Wd your change | |
| 09:56:06 | bauzas | zigo: please provide a backport change for stable/xena too | |
| 09:56:16 | bauzas | so we will create a RC2 | |
| 10:05:26 | gibi | bauzas: after you are back placement also needs care after RC1 https://review.opendev.org/c/openstack/placement/+/809366 | |
| 10:06:15 | gibi | bauzas: I will look into the lower constraints failre in placemenet stable/xena setup patches, probably that impacts placement master too | |
| 10:07:52 | opendevreview | Balazs Gibizer proposed openstack/placement master: [DNM]: Trigger lower-constaints job https://review.opendev.org/c/openstack/placement/+/809994 | |
| 10:46:11 | gibi | yepp it seems master lower-constraints also times out in placement too ^^ | |
| 10:46:28 | gibi | will fix it based on how neutron fixed it | |
| 11:19:40 | gibi | lyarwood, stephenfin: is it an RC critical fix https://review.opendev.org/c/openstack/nova/+/809934 ? | |
| 11:21:07 | stephenfin | Ah, whoops, probably not (though I'll defer to lyarwood to be sure). I was thinking all bugfixes were fair play right now since we'd branched already /o\ | |
| 11:21:14 | stephenfin | gibi: feel free to pull it back out | |
| 11:21:45 | bauzas | gibi: ack, taxying back my kid but I'll be around in 15 mins | |
| 11:21:48 | lyarwood | I was under the same impression | |
| 11:22:04 | lyarwood | gibi: isn't master open for Yoga now? | |
| 11:22:27 | gibi | I think until the final RC we need to keep master close to stable/xena for any last minute backport | |
| 11:23:04 | lyarwood | that's fair, yeah it isn't critical so feel free to yank it out if you can | |
| 11:23:41 | opendevreview | Balazs Gibizer proposed openstack/nova master: Fix the wrong exception used to retry detach API calls https://review.opendev.org/c/openstack/nova/+/809934 | |
| 11:23:41 | lyarwood | and FWIW I'm not a huge fan of policy we can't enforce in the tooling like this | |
| 11:23:44 | gibi | yanked | |
| 11:23:48 | lyarwood | thanks | |
| 12:07:30 | bauzas | gibi: all placement Xena changes are now +Wd https://review.opendev.org/q/project:openstack/placement+owner:infra-root%2540openstack.org+is:open | |
| 12:07:51 | gibi | bauzas: thanks but lower constraints fix will be still be needed | |
| 12:07:56 | gibi | as the job will time out | |
| 12:08:01 | gibi | I'm working on it | |
| 12:08:17 | bauzas | gibi: ack thanks | |
| 12:08:25 | bauzas | gibi: what is the change ? | |
| 12:08:46 | gibi | lower constraint bump on master first | |
| 12:08:50 | gibi | as it effects master | |
| 12:09:23 | gibi | then we can lament on either we bump lower on stable/xena too, or look into somehow pinning setuptools version on stable branches | |
| 12:10:01 | gibi | the thing is that we dont pin setuptools coming from virtualenv package on stable so we use basically the latest on every stable | |
| 12:10:40 | gibi | an the latest setuptools removed support for some features old packages are depends on | |
| 12:10:55 | bauzas | (14:08:46) gibi: lower constraint bump on master first | |
| 12:10:58 | bauzas | can't see it | |
| 12:11:03 | gibi | haven't proposed yet | |
| 12:11:09 | bauzas | oh ok | |
| 12:11:15 | gibi | 14:08 < gibi> I'm working on it | |
| 12:11:16 | gibi | 14:08 < gibi> I'm working on it | |
| 12:11:20 | gibi | 14:08 < gibi> I'm working on it | |
| 12:11:24 | gibi | ups | |
| 12:11:26 | gibi | sorry | |
| 12:15:16 | opendevreview | Balazs Gibizer proposed openstack/placement master: Bump min decorator to 4.0.0 https://review.opendev.org/c/openstack/placement/+/810001 | |
| 12:15:23 | gibi | bauzas: now here it is | |
| 12:15:28 | bauzas | gibi: heh sorry | |
| 12:15:37 | gibi | I was lost couple of hour figuring out what happened | |
| 12:15:53 | gibi | especially as I don't like the ide to bump a lower constraint on stable branch | |
| 12:16:01 | gibi | s/ide/idea/ | |
| 12:16:07 | bauzas | agreed | |
| 12:16:29 | bauzas | taxying | |
| 12:18:23 | gibi | as like https://en.wikipedia.org/wiki/Cardassian ? :) | |