| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-04-14 | |||
| 22:48:09 | sean-k-mooney | mriedem: we often want to change other value that also are not mutable when we do multi node functional test and simple using another value is not always an option | |
| 22:48:51 | sean-k-mooney | im thinking of the cpu_dedicated_set and other values like that where it would be really nice to have a two node functional test with different values. | |
| 22:49:43 | sean-k-mooney | anyway its not relavent to bauzas patch but its frequently a problem we hit. | |
| 22:59:39 | ianw | hi, nodepool testing hit the removal of "user_data" from the nova client https://opendev.org/openstack/python-novaclient/commit/03dca4bc823c82054869dfaf6925d5e1e068ac51 | |
| 22:59:56 | ianw | we were testing that the user_data specified in the nodepool config makes it through to nova | |
| 23:00:06 | ianw | is there another suggested way to check that? | |
| 23:03:55 | melwitt | ianw: hm, yeah I wondered if anyone would be affected by that change when I saw it go through. I can't immediately think of anything other than using the novaclient python binding. or calling the API directly | |
| 23:05:44 | ianw | yeah, i think maybe using the binding might work; it doesn't seem like it's generically useful enough to put back in behind a switch or something | |
| 23:06:24 | ianw | unless others think it is ... | |
| 23:07:39 | melwitt | yeah, I wondered the same. | |
| 23:09:20 | melwitt | it's been years but I've looked at it before as an end user, as a "did I do that right?" sanity check | |
| 23:11:33 | openstackgerrit | Merged openstack/nova master: Add test coverage of existing flavor extra spec policies https://review.opendev.org/719372 | |
| 23:11:43 | corvus | melwitt: yeah, i think it's worth keeping in some form because of that | |
| 23:12:18 | corvus | but putting it behind a cli flag is fine -- i get that "dump a bunch of base64 by default" isn't optimal ux :) | |
| 23:12:33 | melwitt | heh yeah | |
| 23:13:09 | melwitt | well, it would have been good to have heard this before last thursday :( cause it's in there and locked in for ussuri now | |
| 23:13:30 | melwitt | I guess we could backport | |
| 23:13:30 | corvus | but getting at it from the cli (with a flag) "i put a key in user data, and it didn't show up in the booted vm -- did it actually make it to the cloud?" would be nice | |
| 23:14:02 | melwitt | yeah, I agree | |
| 23:14:46 | melwitt | cc stephenfin | |
| 23:15:14 | corvus | ianw: i guess the -src version of that nodepool job doesn't install the clients from source? | |
| 23:15:42 | corvus | iiuc, we have a -src job to detect pre-release problems with openstacksdk, but maybe it stops there... | |
| 23:16:01 | ianw | coruvs: i guess not; i think it only pulls in nodepool, openstacksdk, dib | |
| 23:16:36 | corvus | that must be how we missed it, since the changed merged in feb, and we've certainly merged nodepool changes which ran that job since then | |
| 23:16:41 | corvus | melwitt: sorry :( | |
| 23:17:00 | gmann | melwitt: and i think (not tried) that patch did not remove that from rebuild operation because there it is present in Rebuild API response as 'user_data' not 'OS-EXT-SRV-ATTR:user_data'. | |
| 23:17:18 | melwitt | gmann: hah, nice | |
| 23:18:26 | gmann | for nodepool case, can it be done via server metadata ? | |
| 23:21:50 | ianw | gmann: we're really just testing that the value in the config file for nodepool made it through to the booted nova instance -- the actual data is not used in the test | |
| 23:23:18 | gmann | ohk i thought it as just tagging the things. | |
| 23:56:41 | openstackgerrit | Merged openstack/nova master: Introduce scope_types in flavor extra spec policy https://review.opendev.org/719375 | |
| #openstack-nova - 2020-04-15 | |||
| 00:14:36 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing remaining servers policies https://review.opendev.org/720104 | |
| 00:30:06 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in remaining servers Policies https://review.opendev.org/720106 | |
| 00:34:32 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in servers Policies https://review.opendev.org/662968 | |
| 00:34:43 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add new default roles in servers policies https://review.opendev.org/662971 | |
| 00:34:54 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing server attributes policies https://review.opendev.org/719726 | |
| 00:35:05 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in servers attributes Policies https://review.opendev.org/719729 | |
| 00:44:34 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add new default roles in servers attributes policies https://review.opendev.org/719730 | |
| 00:45:44 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing remaining servers policies https://review.opendev.org/720104 | |
| 00:45:54 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in remaining servers Policies https://review.opendev.org/720106 | |
| 01:05:03 | openstackgerrit | Brin Zhang proposed openstack/nova-specs master: Re-proposes 'Proposal for a safer remote console with password authentication' https://review.opendev.org/719720 | |
| 01:16:12 | brinzhang_ | gibi, sean-k-mooney: I agree with sean-k-mooney, I think his understand is correct, if we failed to bind, that just leak arq db record in Cyborg, I will check again with Sundar, once I am confirmed, I will respin you again. | |
| 01:34:58 | openstackgerrit | zhufl proposed openstack/nova master: Add missing white spaces between words in log messages https://review.opendev.org/720112 | |
| 01:58:18 | brinzhang_ | gibi, sean-k-monney, dansmith: another thing maybe need to concern, please check, see below: | |
| 01:58:21 | brinzhang_ | While we build_instances() we should cleanup arqs if the _create_and_bind_arqs() failed. | |
| 01:58:30 | brinzhang_ | In _cleanup_when_reschedule_fails()[1] we clean the arqs if _create_and_bind_arqs() failed, but I think we just only clean it at L847 if the _create_and_bind_arqs() failed [3]. If we add "compute_utils.delete_arqs_if_needed(context, instance)"[2] in _cleanup_when_reschedule_fails()[1], once we call _cleanup_when_reschedule_fails() it will be cleanup arqs with dp name in instance's flavor, I think that will increase Nova's call to Cyborg (by | |
| 01:58:30 | brinzhang_ | cyborgclinet), right? | |
| 01:58:43 | brinzhang_ | [1]https://opendev.org/openstack/nova/src/branch/master/nova/conductor/manager.py#L582 | |
| 01:58:51 | brinzhang_ | [2]https://opendev.org/openstack/nova/src/branch/master/nova/conductor/manager.py#L598 | |
| 01:58:57 | brinzhang_ | [3]https://opendev.org/openstack/nova/src/branch/master/nova/conductor/manager.py#L848-L849 | |
| 01:59:16 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add new default roles in remaining servers policies https://review.opendev.org/720116 | |
| 02:00:24 | brinzhang_ | IMO, if I am correct, I think we should move "compute_utils.delete_arqs_if_needed(context, instance)"[2] to the right place, i.e., when _create_and_bind_arqs() failed | |
| 02:05:00 | gmann | johnthetubaguy: i replied on server policies comments and finished the remaining one. please check. my worried is to allow host attr for project that is why i kept them system-admin for now | |
| 02:20:21 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Fix follow up comments on policy work https://review.opendev.org/717835 | |
| 02:23:37 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Use oslo policy flag to disable default change warning instead of all https://review.opendev.org/717884 | |
| 03:10:20 | openstackgerrit | Merged openstack/nova master: Introduce scope_types in quota set Policies https://review.opendev.org/719317 | |
| 03:44:48 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Fix server actions to be system and project scoped https://review.opendev.org/720123 | |
| 05:56:03 | openstackgerrit | Merged openstack/nova master: Add new default roles in server group policies https://review.opendev.org/717175 | |
| 06:35:39 | openstackgerrit | Merged openstack/nova master: Add new default roles in quota sets policies https://review.opendev.org/719361 | |
| 06:45:13 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add doc for policy new defaults https://review.opendev.org/720129 | |
| 07:05:38 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Fix server actions to be system and project scoped https://review.opendev.org/720123 | |
| 07:25:21 | openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add new default roles in remaining servers policies https://review.opendev.org/720116 | |
| 07:37:10 | bauzas | good morning Nova | |
| 07:38:52 | gibi | bauzas: good morning | |
| 07:39:52 | bauzas | gibi: good news, now that I'm done with the last series I had to work on, I'll do upstream bug triage today \o/ | |
| 07:40:37 | bauzas | it's been a while | |
| 07:42:18 | bauzas | gibi: stephenfin: sean-k-mooney: lyarwood: since you reviewed the vGPU multiple types series, you could be interested in that quite easy series for fixing cold migration issues for vGPUs | |
| 07:42:45 | bauzas | tl;dr: that's just adding a new argument to resize virt methods and just calling a new method | |
| 07:42:59 | bauzas | the main review is just for the functional test | |
| 07:43:08 | gibi | bauzas: \o/ | |
| 07:43:43 | gibi | bauzas: hit me with the link | |
| 07:44:01 | bauzas | gibi: https://review.opendev.org/#/c/712741 and below | |
| 07:44:47 | gibi | thanks | |
| 07:44:55 | bauzas | gibi: thanks for you :p | |
| 07:45:18 | gibi | bauzas: I'm looking at https://bugs.launchpad.net/nova/+bug/1868033 so you can skip that | |
| 07:45:18 | openstack | Launchpad bug 1868033 in OpenStack Compute (nova) "Booting instance with pci_device fails during rocky->stein live upgrade" [High,New] | |
| 07:45:27 | bauzas | gibi: ack | |
| 07:48:44 | bauzas | 103 bugs... :( | |
| 07:54:12 | gibi | and I already triaged couple of yesterday :) | |
| 07:54:23 | gibi | at least we won't be bored | |
| 07:59:22 | bauzas | gibi: yeah, my main worries go to find some potential bugs that are Ussuri regressions | |
| 08:10:10 | bauzas | gibi: heh, from what I can see, nothing really changes from my day-to-day internal bug triage https://bugs.launchpad.net/nova/+bug/1777608 | |
| 08:10:10 | openstack | Launchpad bug 1777608 in OpenStack Compute (nova) "Nova compute calls plug_vifs unnecessarily for ironic nodes in init_host" [Medium,New] | |
| 08:10:22 | bauzas | gibi: sorry if you feel getting trampled by OSP :p | |
| 08:11:48 | gibi | :) | |
| 08:18:34 | bauzas | lyarwood: looks a valid bug, do you confirm ? | |
| 08:18:38 | bauzas | whoops | |
| 08:18:42 | bauzas | lyarwood: https://bugs.launchpad.net/nova/+bug/1871861 | |
| 08:18:42 | openstack | Launchpad bug 1871861 in OpenStack Compute (nova) "Share the snapshot of volume-backed instance to other tenant, but can not build instance with this image." [Undecided,New] | |
| 08:23:27 | nightmare_unreal | sean-k-mooney: I saw your comments/review here : https://review.opendev.org/#/c/715395/ . I have added my draft, what do you suggest ? | |
| 08:23:37 | nightmare_unreal | Thanks | |
| 08:23:43 | lyarwood | bauzas: looking | |
| 08:23:58 | bauzas | lyarwood: tl;dr: glance allowed to share an image | |
| 08:24:18 | bauzas | lyarwood: another tenant can take this image to build an volume-baked instance | |
| 08:24:30 | bauzas | lyarwood: but this fails miserably | |
| 08:24:44 | lyarwood | bauzas: yeah the issue is it's not an image, it's just a metadata store pointing to the volume snapshot that's in the other tenant still | |
| 08:25:08 | bauzas | but then a cinder bug, correct ? | |
| 08:25:24 | bauzas | we just get the metadata from cinder, nope ? | |
| 08:25:37 | lyarwood | bauzas: yeah it's a cinder feature bug | |
| 08:25:58 | lyarwood | well, between glance and cinder at least | |