| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-04-13 | |||
| 15:32:46 | bauzas | gibi: why are you holding https://review.opendev.org/c/openstack/nova/+/782171 ? | |
| 15:32:58 | bauzas | the master branch is already Xena | |
| 15:33:24 | bauzas | just in case the service version is updated for Wallaby ? | |
| 15:33:56 | gibi | bauzas: in the unlikely event that we got a last minute critical problem that need to be fixed with a service version bump before the W release goes out tomorrow | |
| 15:34:03 | bauzas | kk | |
| 15:34:10 | gibi | it is really unlikely | |
| 15:34:28 | gibi | but it is cheap to hold it for an extra day | |
| 15:34:50 | gibi | same with the db schema backport placeholders | |
| 15:34:51 | bauzas | gibi: I just reviewed it ,) | |
| 15:34:55 | gibi | thanks | |
| 15:35:08 | bauzas | gibi: which change for the db schema ? | |
| 15:35:28 | gibi | yours https://review.opendev.org/c/openstack/nova/+/778923 | |
| 15:36:08 | bauzas | ah k | |
| 15:36:24 | gibi | :) | |
| 15:43:38 | openstackgerrit | Balazs Gibizer proposed openstack/nova-specs master: Change error code when vDPA is unsupported https://review.opendev.org/c/openstack/nova-specs/+/781259 | |
| 16:23:16 | openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Document options for remaining nova-* commands https://review.opendev.org/c/openstack/nova/+/726195 | |
| 16:23:18 | openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Populate "See Also" section with meaningful links https://review.opendev.org/c/openstack/nova/+/729917 | |
| 16:23:20 | openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Add man pages for 'nova-policy' https://review.opendev.org/c/openstack/nova/+/729918 | |
| 17:08:15 | openstackgerrit | Merged openstack/nova stable/stein: Add config parameter 'live_migration_scheme' to live migration with tls guide https://review.opendev.org/c/openstack/nova/+/783199 | |
| 23:06:49 | guilhermesp | hey there! quick question: if we, in any case, do a `virsh undefine <guest>` and then `openstack server reboot --hard <id>`, is the guest going to be recreated? | |
| #openstack-nova - 2021-04-14 | |||
| 00:55:47 | melwitt | guilhermesp: you can just do the openstack server reboot --hard to regenerate the guest xml | |
| 00:58:32 | guilhermesp | indeed! thanks melwitt | |
| 00:59:51 | melwitt | don't think you needed the undefine but I've been wrong before | |
| 02:26:02 | guilhermesp | melwitt: i'm my case ( an openstack-helm deployment ) my nova compute pod wasnt starting for some reason after a power shutdown. All guests were present on the node but in shutoff state and refusing to start, causing the compute node to crash loop and never be UP | |
| 02:26:02 | guilhermesp | melwitt: i'm my case ( an openstack-helm deployment ) my nova compute pod wasnt starting for some reason after a power shutdown. All guests were present on the node but in shutoff state and refusing to start, causing the compute node to crash loop and never be UP | |
| 02:27:31 | guilhermesp | so using cli wasnt an option... after undefine all guests that the pod was able to start the compute and things got back to normal :) ( guests were recreated ofc ) | |
| 02:27:31 | guilhermesp | so using cli wasnt an option... after undefine all guests that the pod was able to start the compute and things got back to normal :) ( guests were recreated ofc ) | |
| 06:47:10 | openstackgerrit | zitoun proposed openstack/nova master: Explaining when a base image is considered unused to avoid confusion why some images aren't deleted. https://review.opendev.org/c/openstack/nova/+/786059 | |
| 06:47:10 | openstackgerrit | zitoun proposed openstack/nova master: Explaining when a base image is considered unused to avoid confusion why some images aren't deleted. https://review.opendev.org/c/openstack/nova/+/786059 | |
| 09:08:10 | hemanth_n | sean-k-mooney: hi, this is pending final review for a long time https://review.opendev.org/c/openstack/nova/+/761824 (rocky backport), this is not urgent so please review only when you get some time.. thanks | |
| 09:08:10 | hemanth_n | sean-k-mooney: hi, this is pending final review for a long time https://review.opendev.org/c/openstack/nova/+/761824 (rocky backport), this is not urgent so please review only when you get some time.. thanks | |
| 09:51:23 | openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Add man pages for 'nova-policy' https://review.opendev.org/c/openstack/nova/+/729918 | |
| 10:30:28 | openstackgerrit | Balazs Gibizer proposed openstack/nova-specs master: Change error code when vDPA is unsupported https://review.opendev.org/c/openstack/nova-specs/+/781259 | |
| 10:30:29 | openstackgerrit | Balazs Gibizer proposed openstack/nova-specs master: Change error code when vDPA is unsupported https://review.opendev.org/c/openstack/nova-specs/+/781259 | |
| 10:54:48 | openstackgerrit | Merged openstack/nova-specs master: Change error code when vDPA is unsupported https://review.opendev.org/c/openstack/nova-specs/+/781259 | |
| 10:54:48 | openstackgerrit | Merged openstack/nova-specs master: Change error code when vDPA is unsupported https://review.opendev.org/c/openstack/nova-specs/+/781259 | |
| 11:04:58 | openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Document location of nova.conf files https://review.opendev.org/c/openstack/nova/+/786202 | |
| 11:55:49 | zigo | Hi. Could we get reviews on that trivial doc (config file help) patch? https://review.opendev.org/c/openstack/nova/+/786059 | |
| 11:55:49 | zigo | Hi. Could we get reviews on that trivial doc (config file help) patch? https://review.opendev.org/c/openstack/nova/+/786059 | |
| 12:02:43 | gibi | zigo: done | |
| 12:02:43 | gibi | zigo: done | |
| 12:02:51 | zigo | Thanks. | |
| 12:02:51 | zigo | Thanks. | |
| 12:18:13 | stephenfin | ooh, we have our 23.0.0 tag \o/ | |
| 12:18:13 | stephenfin | ooh, we have our 23.0.0 tag \o/ | |
| 12:18:24 | stephenfin | gibi: Does this mean we can start merging larger patches again now? | |
| 12:18:24 | stephenfin | gibi: Does this mean we can start merging larger patches again now? | |
| 12:18:41 | gibi | stephenfin: yes it is | |
| 12:18:41 | sean-k-mooney | stephenfin: the release was yesterday so yes | |
| 12:18:41 | gibi | stephenfin: yes it is | |
| 12:18:41 | sean-k-mooney | stephenfin: the release was yesterday so yes | |
| 12:18:44 | stephenfin | * stephenfin has an alembic migration series _almost_ completed locally | |
| 12:18:50 | stephenfin | hurrah | |
| 12:18:50 | gibi | sounds cool | |
| 12:18:50 | stephenfin | hurrah | |
| 12:18:50 | gibi | sounds cool | |
| 12:19:17 | stephenfin | * stephenfin will try wrap this up and spend tomorrow on reviews so | |
| 12:19:50 | stephenfin | sean-k-mooney: I fixed https://review.opendev.org/c/openstack/nova/+/729918/ and added a follow-up explaining where nova.conf comes from | |
| 12:19:50 | stephenfin | sean-k-mooney: I fixed https://review.opendev.org/c/openstack/nova/+/729918/ and added a follow-up explaining where nova.conf comes from | |
| 12:20:22 | stephenfin | just fyi | |
| 12:20:22 | stephenfin | just fyi | |
| 13:01:09 | gibi | core: there are two patches needed on master before we get deep into xena: | |
| 13:01:09 | gibi | core: there are two patches needed on master before we get deep into xena: | |
| 13:01:13 | gibi | https://review.opendev.org/c/openstack/nova/+/782171 | |
| 13:01:13 | gibi | https://review.opendev.org/c/openstack/nova/+/782171 | |
| 13:01:17 | gibi | https://review.opendev.org/c/openstack/nova/+/778923 | |
| 13:01:17 | gibi | https://review.opendev.org/c/openstack/nova/+/778923 | |
| 13:01:28 | gibi | both needs a second core | |
| 13:01:28 | gibi | both needs a second core | |
| 13:12:10 | openstackgerrit | Johannes Kulik proposed openstack/nova master: vmware: OptionValue.value cannot be Boolean https://review.opendev.org/c/openstack/nova/+/786246 | |
| 13:12:10 | openstackgerrit | Johannes Kulik proposed openstack/nova master: vmware: OptionValue.value cannot be Boolean https://review.opendev.org/c/openstack/nova/+/786246 | |
| 13:18:00 | bauzas | gibi: ack lookinf | |
| 13:18:00 | bauzas | gibi: ack lookinf | |
| 13:18:06 | bauzas | lol | |
| 13:18:27 | bauzas | * bauzas just clicked and saw the changes :p | |
| 13:19:03 | gibi | :D | |
| 13:19:38 | sean-k-mooney | we likely should look at doing the libvirt min microverion bumps early too | |
| 13:19:38 | sean-k-mooney | we likely should look at doing the libvirt min microverion bumps early too | |
| 13:19:45 | sean-k-mooney | before m1 in any case | |
| 13:19:45 | sean-k-mooney | before m1 in any case | |
| 13:20:06 | sean-k-mooney | not as urgent though | |
| 13:20:06 | sean-k-mooney | not as urgent though | |
| 13:20:43 | sean-k-mooney | for https://review.opendev.org/c/openstack/nova/+/782171/2/nova/objects/service.py#205 | |
| 13:20:43 | sean-k-mooney | for https://review.opendev.org/c/openstack/nova/+/782171/2/nova/objects/service.py#205 | |
| 13:20:54 | sean-k-mooney | should we really be removing the victoria alias | |
| 13:20:54 | sean-k-mooney | should we really be removing the victoria alias | |
| 13:21:06 | sean-k-mooney | i would have just appended wallaby | |
| 13:21:06 | sean-k-mooney | i would have just appended wallaby | |
| 13:21:22 | sean-k-mooney | otherwise what is the point of having a dict | |
| 13:21:22 | sean-k-mooney | otherwise what is the point of having a dict | |
| 13:29:47 | gibi | sean-k-mooney: I can remove the dict. I thought originally that we need a list but the code only uses that latest one so the dict is not needed | |
| 13:29:47 | gibi | sean-k-mooney: I can remove the dict. I thought originally that we need a list but the code only uses that latest one so the dict is not needed | |
| 13:30:39 | gibi | but we both need a service number and the name | |
| 13:30:40 | gibi | but we both need a service number and the name | |
| 13:31:29 | sean-k-mooney | yep well we could leave it as is too it just feels a little clunky currently | |
| 13:31:29 | sean-k-mooney | yep well we could leave it as is too it just feels a little clunky currently | |
| 13:32:52 | sean-k-mooney | not that i really want to open this can of worms but our pms keep brinign up the idea of having nova support more then n-1 computes | |
| 13:32:52 | sean-k-mooney | not that i really want to open this can of worms but our pms keep brinign up the idea of having nova support more then n-1 computes | |
| 13:33:15 | sean-k-mooney | im partly wondering if we want this to be confiugrable at some point | |
| 13:33:15 | sean-k-mooney | im partly wondering if we want this to be confiugrable at some point | |