| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-09-17 | |||
| 20:02:45 | mloza | is rbd_secret_uuid in [libvirt] for nova.conf supports multiple uuids? | |
| 20:59:29 | opendevreview | melanie witt proposed openstack/nova master: Update quota apis with keystone limits and usage https://review.opendev.org/c/openstack/nova/+/713499 | |
| 20:59:29 | opendevreview | melanie witt proposed openstack/nova master: Add legacy limits and usage to placement unified limits https://review.opendev.org/c/openstack/nova/+/713498 | |
| 20:59:30 | opendevreview | melanie witt proposed openstack/nova master: Add reno for unified limits https://review.opendev.org/c/openstack/nova/+/715271 | |
| 21:08:02 | opendevreview | melanie witt proposed openstack/nova master: WIP Enable unified limits in the nova-next job https://review.opendev.org/c/openstack/nova/+/789963 | |
| 23:36:55 | opendevreview | melanie witt proposed openstack/nova master: WIP Enable unified limits in the nova-next job https://review.opendev.org/c/openstack/nova/+/789963 | |
| #openstack-nova - 2021-09-18 | |||
| 01:09:55 | opendevreview | melanie witt proposed openstack/nova master: WIP Enable unified limits in the nova-next job https://review.opendev.org/c/openstack/nova/+/789963 | |
| 04:04:42 | opendevreview | melanie witt proposed openstack/nova master: WIP Enable unified limits in the nova-next job https://review.opendev.org/c/openstack/nova/+/789963 | |
| 05:57:06 | opendevreview | melanie witt proposed openstack/nova master: WIP Enable unified limits in the nova-next job https://review.opendev.org/c/openstack/nova/+/789963 | |
| 07:14:55 | oklhost | sean-k-mooney: what does that mean? is there some documentation around? | |
| 14:01:14 | opendevreview | Takashi Natsume proposed openstack/nova master: Update min supported service version for Yoga. https://review.opendev.org/c/openstack/nova/+/809932 | |
| 15:10:30 | 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 | |
| #openstack-nova - 2021-09-19 | |||
| 02:18:09 | opendevreview | Takashi Natsume proposed openstack/nova master: Update contributor guide for Yoga https://review.opendev.org/c/openstack/nova/+/809936 | |
| 06:08:03 | opendevreview | Brin Zhang proposed openstack/nova-specs master: Re-propose Remove tenant_id https://review.opendev.org/c/openstack/nova-specs/+/809939 | |
| 06:42:16 | opendevreview | Wenping Song proposed openstack/placement master: Modify the comment that is confused https://review.opendev.org/c/openstack/placement/+/809948 | |
| 12:11:25 | opendevreview | Merged openstack/os-vif master: Add Python3 yoga unit tests https://review.opendev.org/c/openstack/os-vif/+/808455 | |
| 15:38:09 | opendevreview | Elod Illes proposed openstack/nova stable/ussuri: [stable-only] Set lower-constraints job as non-voting https://review.opendev.org/c/openstack/nova/+/809955 | |
| 21:16:12 | opendevreview | Elod Illes proposed openstack/nova stable/ussuri: [stable-only] Set lower-constraints job as non-voting https://review.opendev.org/c/openstack/nova/+/809955 | |
| 21:25:29 | opendevreview | Elod Illes proposed openstack/nova stable/ussuri: [stable-only] Set lower-constraints job as non-voting https://review.opendev.org/c/openstack/nova/+/809955 | |
| #openstack-nova - 2021-09-20 | |||
| 07:37:06 | bauzas | good morning Nova | |
| 07:37:16 | bauzas | gibi: stephenfin: hola folks | |
| 07:48:16 | lyarwood | \o morning | |
| 07:58:27 | gibi | o. | |
| 07:58:30 | gibi | o/ | |
| 08:56:51 | oklhost | sean-k-mooney: thanks, seems we got less log entries. | |
| 09:14:29 | opendevreview | Thomas Goirand proposed openstack/nova master: Add missing __init__.py in nova/db/api https://review.opendev.org/c/openstack/nova/+/809980 | |
| 09:15:17 | zigo | bauzas: Good morning, there's a ooopsy there, no ? ^ | |
| 09:15:59 | bauzas | zigo: well, good question, I'm not sure we need it | |
| 09:16:16 | bauzas | what kind of issue you have ? | |
| 09:16:51 | zigo | bauzas: When I later on generate nova.conf with oslo-config-generator (with nova installed debian/tmp/usr/lib/python3/dist-package) I get a stack dump, with this folder not installed ... | |
| 09:17:32 | sean-k-mooney | bauzas: without __init__.py its technially not a python module | |
| 09:17:39 | bauzas | yup | |
| 09:17:40 | bauzas | I know | |
| 09:17:48 | bauzas | but I wonder if it's an issue | |
| 09:17:49 | zigo | So basically, nova/db/api isn't getting installed when doing python3 setup.py install ... | |
| 09:17:55 | bauzas | that changed a bit with py3 IIRC | |
| 09:19:49 | bauzas | but I wonder why we have this issue now | |
| 09:19:54 | bauzas | and not before | |
| 09:21:02 | zigo | bauzas: 2 things: first, that folder didn't exist in Wallaby, 2/ it's the symptoms of "it works in devstack" ... | |
| 09:21:13 | zigo | (ie: setup.py install isn't being run...) | |
| 09:21:24 | sean-k-mooney | bauzas: presumably because steph change thing with his alembic module | |
| 09:21:27 | sean-k-mooney | work | |
| 09:21:29 | sean-k-mooney | https://github.com/openstack/nova/commit/bf8b5fc7d05e0a66031a03e50e8f6bb76a921046#diff-6137249efc22cd455ac118bde1598a27beea93adcfcd49b36a2329318fc33c6e | |
| 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 | |