| *** threestrands has joined #openstack-nova | 00:00 | |
| *** JamesBenson has joined #openstack-nova | 00:06 | |
| *** tetsuro has joined #openstack-nova | 00:10 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing remaining servers policies https://review.opendev.org/720104 | 00:14 |
|---|---|---|
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in remaining servers Policies https://review.opendev.org/720106 | 00:30 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in servers Policies https://review.opendev.org/662968 | 00:34 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add new default roles in servers policies https://review.opendev.org/662971 | 00:34 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing server attributes policies https://review.opendev.org/719726 | 00:34 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in servers attributes Policies https://review.opendev.org/719729 | 00:35 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add new default roles in servers attributes policies https://review.opendev.org/719730 | 00:44 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing remaining servers policies https://review.opendev.org/720104 | 00:45 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in remaining servers Policies https://review.opendev.org/720106 | 00:45 |
| *** amodi has joined #openstack-nova | 00:52 | |
| *** ttsiouts has joined #openstack-nova | 01:00 | |
| 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:05 |
| 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:16 |
| *** sapd1 has joined #openstack-nova | 01:30 | |
| *** dlbewley has quit IRC | 01:32 | |
| *** ttsiouts has quit IRC | 01:34 | |
| openstackgerrit | zhufl proposed openstack/nova master: Add missing white spaces between words in log messages https://review.opendev.org/720112 | 01:34 |
| *** sapd1 has quit IRC | 01:36 | |
| *** lbragstad has quit IRC | 01:38 | |
| *** sapd1 has joined #openstack-nova | 01:49 | |
| *** ociuhandu has joined #openstack-nova | 01:56 | |
| brinzhang_ | gibi, sean-k-monney, dansmith: another thing maybe need to concern, please check, see below: | 01:58 |
| brinzhang_ | While we build_instances() we should cleanup arqs if the _create_and_bind_arqs() failed. | 01:58 |
| 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 |
| brinzhang_ | cyborgclinet), right? | 01:58 |
| brinzhang_ | [1]https://opendev.org/openstack/nova/src/branch/master/nova/conductor/manager.py#L582 | 01:58 |
| brinzhang_ | [2]https://opendev.org/openstack/nova/src/branch/master/nova/conductor/manager.py#L598 | 01:58 |
| brinzhang_ | [3]https://opendev.org/openstack/nova/src/branch/master/nova/conductor/manager.py#L848-L849 | 01:58 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add new default roles in remaining servers policies https://review.opendev.org/720116 | 01:59 |
| 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:00 |
| *** ttsiouts has joined #openstack-nova | 02:03 | |
| 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:05 |
| *** mkrai has joined #openstack-nova | 02:07 | |
| *** ttsiouts has quit IRC | 02:07 | |
| *** ociuhandu has quit IRC | 02:13 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Fix follow up comments on policy work https://review.opendev.org/717835 | 02:20 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Use oslo policy flag to disable default change warning instead of all https://review.opendev.org/717884 | 02:23 |
| *** tetsuro has quit IRC | 02:29 | |
| *** artom has quit IRC | 02:32 | |
| *** tetsuro has joined #openstack-nova | 02:32 | |
| *** ttsiouts has joined #openstack-nova | 02:36 | |
| openstackgerrit | Merged openstack/nova master: Introduce scope_types in quota set Policies https://review.opendev.org/719317 | 03:10 |
| *** ttsiouts has quit IRC | 03:10 | |
| *** mkrai has quit IRC | 03:23 | |
| *** mkrai has joined #openstack-nova | 03:24 | |
| *** hamzy_ has joined #openstack-nova | 03:29 | |
| *** hamzy has quit IRC | 03:31 | |
| *** psachin has joined #openstack-nova | 03:33 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Fix server actions to be system and project scoped https://review.opendev.org/720123 | 03:44 |
| *** mkrai has quit IRC | 04:05 | |
| *** mkrai_ has joined #openstack-nova | 04:06 | |
| *** ttsiouts has joined #openstack-nova | 04:12 | |
| *** gyee has quit IRC | 04:13 | |
| *** mkrai has joined #openstack-nova | 04:14 | |
| *** ociuhandu has joined #openstack-nova | 04:15 | |
| *** mkrai_ has quit IRC | 04:17 | |
| *** KeithMnemonic has quit IRC | 04:30 | |
| *** ociuhandu has quit IRC | 04:33 | |
| *** evrardjp has quit IRC | 04:37 | |
| *** evrardjp has joined #openstack-nova | 04:37 | |
| *** ttsiouts has quit IRC | 04:46 | |
| *** dklyle has quit IRC | 04:56 | |
| *** mkrai has quit IRC | 05:01 | |
| *** mkrai has joined #openstack-nova | 05:06 | |
| *** ttsiouts has joined #openstack-nova | 05:22 | |
| *** ratailor has joined #openstack-nova | 05:23 | |
| *** ttsiouts has quit IRC | 05:27 | |
| *** udesale has joined #openstack-nova | 05:37 | |
| *** ociuhandu has joined #openstack-nova | 05:39 | |
| *** vishalmanchanda has joined #openstack-nova | 05:40 | |
| *** sapd1 has quit IRC | 05:41 | |
| *** tobberydberg has joined #openstack-nova | 05:46 | |
| *** xek has joined #openstack-nova | 05:50 | |
| *** ociuhandu has quit IRC | 05:51 | |
| *** threestrands has quit IRC | 05:54 | |
| *** ttsiouts has joined #openstack-nova | 05:55 | |
| *** sapd1 has joined #openstack-nova | 05:55 | |
| openstackgerrit | Merged openstack/nova master: Add new default roles in server group policies https://review.opendev.org/717175 | 05:56 |
| *** ratailor has quit IRC | 06:00 | |
| *** ratailor has joined #openstack-nova | 06:12 | |
| *** ociuhandu has joined #openstack-nova | 06:26 | |
| *** ttsiouts has quit IRC | 06:29 | |
| openstackgerrit | Merged openstack/nova master: Add new default roles in quota sets policies https://review.opendev.org/719361 | 06:35 |
| *** slaweq_ has joined #openstack-nova | 06:37 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add doc for policy new defaults https://review.opendev.org/720129 | 06:45 |
| *** dpawlik has joined #openstack-nova | 06:49 | |
| *** ttsiouts has joined #openstack-nova | 06:51 | |
| *** ttsiouts has quit IRC | 06:51 | |
| *** ttsiouts has joined #openstack-nova | 06:52 | |
| *** nightmare_unreal has joined #openstack-nova | 06:53 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Fix server actions to be system and project scoped https://review.opendev.org/720123 | 07:05 |
| *** maciejjozefczyk has joined #openstack-nova | 07:06 | |
| *** lpetrut has joined #openstack-nova | 07:07 | |
| *** ccamacho has joined #openstack-nova | 07:11 | |
| *** ralonsoh has joined #openstack-nova | 07:14 | |
| *** tesseract has joined #openstack-nova | 07:16 | |
| *** tosky has joined #openstack-nova | 07:23 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add new default roles in remaining servers policies https://review.opendev.org/720116 | 07:25 |
| *** rpittau|afk is now known as rpittau | 07:28 | |
| *** iurygregory has quit IRC | 07:31 | |
| *** iurygregory has joined #openstack-nova | 07:32 | |
| bauzas | good morning Nova | 07:37 |
| gibi | bauzas: good morning | 07:38 |
| 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:39 |
| bauzas | it's been a while | 07:40 |
| *** ociuhandu has quit IRC | 07:40 | |
| *** ociuhandu has joined #openstack-nova | 07:41 | |
| 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 |
| bauzas | tl;dr: that's just adding a new argument to resize virt methods and just calling a new method | 07:42 |
| bauzas | the main review is just for the functional test | 07:42 |
| gibi | bauzas: \o/ | 07:43 |
| gibi | bauzas: hit me with the link | 07:43 |
| bauzas | gibi: https://review.opendev.org/#/c/712741 and below | 07:44 |
| gibi | thanks | 07:44 |
| * bauzas goes to launchpad | 07:44 | |
| bauzas | gibi: thanks for you :p | 07:44 |
| gibi | bauzas: I'm looking at https://bugs.launchpad.net/nova/+bug/1868033 so you can skip that | 07:45 |
| openstack | Launchpad bug 1868033 in OpenStack Compute (nova) "Booting instance with pci_device fails during rocky->stein live upgrade" [High,New] | 07:45 |
| bauzas | gibi: ack | 07:45 |
| *** ociuhandu has quit IRC | 07:46 | |
| bauzas | 103 bugs... :( | 07:48 |
| gibi | and I already triaged couple of yesterday :) | 07:54 |
| gibi | at least we won't be bored | 07:54 |
| *** links has joined #openstack-nova | 07:56 | |
| bauzas | gibi: yeah, my main worries go to find some potential bugs that are Ussuri regressions | 07:59 |
| 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 |
| openstack | Launchpad bug 1777608 in OpenStack Compute (nova) "Nova compute calls plug_vifs unnecessarily for ironic nodes in init_host" [Medium,New] | 08:10 |
| *** slaweq_ is now known as slaweq | 08:10 | |
| bauzas | gibi: sorry if you feel getting trampled by OSP :p | 08:10 |
| gibi | :) | 08:11 |
| *** ttsiouts has quit IRC | 08:13 | |
| bauzas | lyarwood: looks a valid bug, do you confirm ? | 08:18 |
| bauzas | whoops | 08:18 |
| bauzas | lyarwood: https://bugs.launchpad.net/nova/+bug/1871861 | 08:18 |
| 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:18 |
| *** ttsiouts has joined #openstack-nova | 08:19 | |
| *** ccamacho has quit IRC | 08:20 | |
| *** tkajinam has quit IRC | 08:21 | |
| *** ociuhandu has joined #openstack-nova | 08:22 | |
| 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 |
| nightmare_unreal | Thanks | 08:23 |
| lyarwood | bauzas: looking | 08:23 |
| bauzas | lyarwood: tl;dr: glance allowed to share an image | 08:23 |
| bauzas | lyarwood: another tenant can take this image to build an volume-baked instance | 08:24 |
| bauzas | lyarwood: but this fails miserably | 08:24 |
| 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:24 |
| bauzas | but then a cinder bug, correct ? | 08:25 |
| bauzas | we just get the metadata from cinder, nope ? | 08:25 |
| lyarwood | bauzas: yeah it's a cinder feature bug | 08:25 |
| * bauzas moves it to cinder | 08:25 | |
| lyarwood | well, between glance and cinder at least | 08:25 |
| bauzas | lyarwood: thanks, I'm done with this bug | 08:25 |
| bauzas | we're talking about upstream bug triage | 08:26 |
| bauzas | I just want the cinder folks to look at it | 08:26 |
| lyarwood | tbh there might be a way of forcing the data to actually end up in glance | 08:26 |
| lyarwood | ack yeah | 08:26 |
| lyarwood | I'll take a look quickly but I'll leave it to them if I can't find anything | 08:26 |
| * lyarwood -> coffee | 08:27 | |
| *** ociuhandu has quit IRC | 08:29 | |
| *** ociuhandu has joined #openstack-nova | 08:29 | |
| johnthetubaguy | lyarwood: do cinder do volume snapshot sharing between tenants? | 08:34 |
| johnthetubaguy | (or get glance to copy it, I guess) | 08:34 |
| bauzas | johnthetubaguy: I feel this bug will potentially require a cross-project effort in order to get fixed | 08:36 |
| bauzas | (but there could be duplicates) | 08:36 |
| johnthetubaguy | bauzas: sounds like it | 08:36 |
| *** avolkov has joined #openstack-nova | 08:37 | |
| johnthetubaguy | I mean, I actually don't think nova should do anything here, but I would say that :P | 08:37 |
| lyarwood | johnthetubaguy: yeah I was thinking you could force the upload into glance to workaround | 08:37 |
| * lyarwood looks at the API docs now | 08:37 | |
| johnthetubaguy | lyarwood: I was thinking glance forces an upload into cinder for the additional project, but either way I guess | 08:38 |
| lyarwood | that could also work, at least c-api/c-vol can offload things to the backends that way | 08:38 |
| *** ccamacho has joined #openstack-nova | 08:40 | |
| *** ccamacho has quit IRC | 08:40 | |
| bauzas | johnthetubaguy: feel free to add nova as a related project to the bug then ;) | 08:41 |
| bauzas | johnthetubaguy: I only marked cinder, but obviously glance and maybe nova could be added | 08:41 |
| *** ratailor has quit IRC | 08:41 | |
| *** ratailor has joined #openstack-nova | 08:42 | |
| *** ccamacho has joined #openstack-nova | 08:45 | |
| *** tetsuro has quit IRC | 08:45 | |
| *** dtantsur|afk is now known as dtantsur | 08:49 | |
| *** martinkennelly has joined #openstack-nova | 08:56 | |
| bauzas | gibi: I've seen you started to look at https://bugs.launchpad.net/nova/+bug/1871583 | 08:58 |
| openstack | Launchpad bug 1871583 in OpenStack Compute (nova) "Migrate the instance, the destination host is not specified. When the "destination host" == " source host" , will set the instance's state error." [Undecided,New] - Assigned to Qiu Fossen (fossen123) | 08:58 |
| bauzas | gibi: I'm confused with the exception | 08:59 |
| bauzas | gibi: so, libvirt doesn't support resizing to the same host | 09:01 |
| bauzas | hence the exception | 09:01 |
| bauzas | anyway, nevermind | 09:02 |
| bauzas | I think what to triage | 09:02 |
| bauzas | what/how | 09:02 |
| *** sapd1 has quit IRC | 09:04 | |
| slaweq | gibi: hi | 09:06 |
| slaweq | gibi: I was hoping that when https://review.opendev.org/#/c/717662/ will be merged, we will be good with grenade jobs finally | 09:07 |
| slaweq | gibi: but I saw job from yesterday which failed again https://7ad29d1b700c1da60ae0-1bae5319fe4594ade335a46ad1c3bcc9.ssl.cf2.rackcdn.com/717083/5/check/neutron-grenade-multinode/2be9497/logs/grenade.sh.txt | 09:07 |
| slaweq | gibi: can You maybe take a look if that is some new issue or still the same? | 09:07 |
| slaweq | thx in advance | 09:07 |
| hrw | https://review.opendev.org/#/c/712607/ has two +2, has +W, has +1 from zuul. needs recheck to go to gate? | 09:14 |
| *** sapd1 has joined #openstack-nova | 09:16 | |
| *** martinkennelly has quit IRC | 09:16 | |
| bauzas | hrw: looking | 09:16 |
| hrw | bauzas: thx | 09:17 |
| hrw | bauzas: I am looking at rebasing https://review.opendev.org/#/c/714439/ on top of 712607 and https://review.opendev.org/#/c/714311/ but without changing them again. | 09:18 |
| *** sapd1 has quit IRC | 09:21 | |
| bauzas | hrw: okay, so two things | 09:21 |
| bauzas | hrw: yes you need to recheck 712607 | 09:22 |
| bauzas | hrw: and if you rebase 714439 on top of ^, then 712607 will also be checked | 09:22 |
| bauzas | hrw: do you know http://zuul.openstack.org/status ? | 09:23 |
| hrw | bauzas: yes. I even wrote userscript for it | 09:23 |
| hrw | just not familar with nova jobs | 09:23 |
| bauzas | all cool then, you can see what goes into th check pipeline | 09:23 |
| bauzas | the* | 09:23 |
| bauzas | hrw: nothing really here is related to nova jobs, it's just zuul ordering | 09:24 |
| bauzas | (and gerrit) | 09:24 |
| hrw | bauzas: can you look at https://review.opendev.org/#/c/714425/ too? simple 'check for SEV only on x86' patch | 09:24 |
| bauzas | hrw: sure, i can this week | 09:24 |
| hrw | thanks | 09:24 |
| hrw | I have 22 open patches now. feels far too many ;D | 09:25 |
| *** ccamacho has quit IRC | 09:44 | |
| hrw | bauzas: thanks for SEV review. will look at improving | 09:45 |
| bauzas | np, ping me once you're done and i'll be happy to look at it again | 09:46 |
| hrw | left comment | 09:48 |
| *** artom has joined #openstack-nova | 09:51 | |
| bauzas | can someone remind me what's the current status with service tokens ? https://bugs.launchpad.net/nova/+bug/1870558 | 09:57 |
| openstack | Launchpad bug 1870558 in OpenStack Compute (nova) "Server's host not changed but actually on dest node after live-migrating" [Undecided,New] | 09:57 |
| bauzas | nevermind | 10:03 |
| hrw | bauzas: test_host checks what host can, test_driver checks what guest can, right? | 10:05 |
| stephenfin | gmann: Few questions on https://review.opendev.org/#/c/662968/ | 10:05 |
| bauzas | hrw: kind of | 10:05 |
| *** ccamacho has joined #openstack-nova | 10:05 | |
| bauzas | hrw: test_driver just tests the driver.py module | 10:05 |
| bauzas | which is basically the libvirt wrapper | 10:06 |
| bauzas | hrw: and test_host.py does verify libvirt.host.py module which is more the interface for https://libvirt.org/drvnodedev.html | 10:06 |
| hrw | ok | 10:07 |
| *** ttsiouts has quit IRC | 10:07 | |
| bauzas | and https://libvirt.org/format.html | 10:07 |
| hrw | so in test_host I should check 'if host arch != x86 then no sev' like | 10:07 |
| bauzas | yup, see the example I gave in the comment | 10:08 |
| hrw | I have that on gvim now. right with test_driver (where I already played with tests) | 10:10 |
| *** ttsiouts has joined #openstack-nova | 10:11 | |
| hrw | bauzas: is https://paste.centos.org/view/e56be492 looking good? | 10:17 |
| hrw | no, it is rather nto | 10:17 |
| hrw | should call _set_amd_sev_support() | 10:20 |
| *** rpittau is now known as rpittau|bbl | 10:23 | |
| openstackgerrit | Merged openstack/nova master: Add new default roles in flavor extra specs policies https://review.opendev.org/719381 | 10:28 |
| openstackgerrit | Marcin Juszkiewicz proposed openstack/nova master: libvirt: check for AMD SEV only on x86-64 https://review.opendev.org/714425 | 10:31 |
| hrw | bauzas: good to have so many tests to take ideas from ;d | 10:32 |
| *** sapd1 has joined #openstack-nova | 10:33 | |
| *** sapd1 has quit IRC | 10:37 | |
| *** sapd1 has joined #openstack-nova | 10:38 | |
| * bauzas officially creates the cyborg tag for https://bugs.launchpad.net/nova/+bug/1872934 | 10:40 | |
| openstack | Launchpad bug 1872934 in OpenStack Compute (nova) "Accurately clean up ARQs resources during build_instances() in conductor" [Undecided,New] - Assigned to Brin Zhang (zhangbailin) | 10:40 |
| bauzas | brinzhang_: ^ | 10:40 |
| brinzhang_ | bauzas: hi, I post this in IRC, you can see above | 10:41 |
| bauzas | brinzhang_: my point is just to tell you that there is a new 'cyborg' tag you can use for triaging bugs that relate to Cyborg | 10:42 |
| brinzhang_ | bauzas: thanks check this bug ^^ | 10:42 |
| brinzhang_ | bauzas: got it, will add this tag if there is the same issue :) | 10:42 |
| bauzas | brinzhang_: if you create a bug for cyborg, just triage it at the same time so it will help the untriaged bugs number to not raise | 10:42 |
| brinzhang_ | bauzas: yeah, good idea, I will take this way, thanks. | 10:44 |
| bauzas | gosh, I was missing LP timeout issues... | 10:44 |
| brinzhang_ | bauzas: but sorry Cyborg dont use launchpad (use storyboard), this makes me disheartenment | 10:45 |
| brinzhang_ | ... | 10:45 |
| *** sapd1 has quit IRC | 10:45 | |
| bauzas | brinzhang_: you lucky folks... | 10:45 |
| bauzas | brinzhang_: yet another cyborg-related bug FYK https://bugs.launchpad.net/nova/+bug/1865549 | 10:47 |
| openstack | Launchpad bug 1865549 in OpenStack Compute (nova) " Define Cyborg ARQ binding notification event." [Undecided,New] | 10:47 |
| openstackgerrit | Merged openstack/nova master: Pass the actual target in flavor extra specs policy https://review.opendev.org/719603 | 10:48 |
| openstackgerrit | Merged openstack/nova stable/train: Use placement stable version for functional job https://review.opendev.org/719120 | 10:48 |
| brinzhang_ | bauzas: ack, I will check this tomorrow | 10:48 |
| bauzas | just sayin' | 10:49 |
| *** derekh has joined #openstack-nova | 10:49 | |
| bauzas | aaaand cyborg got promoted as an 'official' LP tag :) | 10:49 |
| brinzhang_ | this bug looks like a BP, right? | 10:50 |
| *** sapd1 has joined #openstack-nova | 10:50 | |
| bauzas | brinzhang_: which one ? | 10:50 |
| brinzhang_ | bug 1865549 | 10:50 |
| openstack | bug 1865549 in OpenStack Compute (nova) " Define Cyborg ARQ binding notification event." [Low,Confirmed] https://launchpad.net/bugs/1865549 | 10:50 |
| brinzhang_ | looks like we should add an event for cyborg notification | 10:51 |
| bauzas | yeah, agreed, just marked it as Wishlist | 10:51 |
| brinzhang_ | I was not looked into, I have a meeting now | 10:51 |
| brinzhang_ | bauzas: greate, I will check and push a spec for this | 10:52 |
| bauzas | brinzhang_: feel free to close it then and open a blueprint | 10:53 |
| brinzhang_ | bauzas: yeah, we can close it until I open the bp | 10:54 |
| *** sapd1 has quit IRC | 10:55 | |
| *** sapd1 has joined #openstack-nova | 10:55 | |
| bauzas | gibi: stephenfin: huh, that's a nasty bug https://bugs.launchpad.net/nova/+bug/1868033 | 10:56 |
| openstack | Launchpad bug 1868033 in OpenStack Compute (nova) "Booting instance with pci_device fails during rocky->stein live upgrade" [High,New] | 10:56 |
| stephenfin | ick | 10:57 |
| stephenfin | That's a pretty big issue. I can work on fixing that today. | 10:58 |
| stephenfin | Guess we didn't spot that since we don't do PCI testing in the gate :( | 10:58 |
| gibi | stephenfin: I started looking into that but got distracted (virtual PTG planning) so you can go ahead | 10:58 |
| bauzas | TIL https://www.merriam-webster.com/dictionary/ick | 10:58 |
| bauzas | stephenfin: I'll just throw it to you | 10:59 |
| bauzas | stephenfin: that'd require some PCI grenade testing | 11:00 |
| openstackgerrit | Merged openstack/nova master: fup: Combine SUPPORTED_DEVICE_BUS and SUPPORTED_STORAGE_BUSES https://review.opendev.org/718448 | 11:00 |
| bauzas | wow, this one is an old +Wd ^ | 11:00 |
| lyarwood | finally :d | 11:01 |
| *** sapd1 has quit IRC | 11:01 | |
| gibi | lyarwood: two to go :) | 11:01 |
| bauzas | lyarwood: I honestly stopped looking at the gate for your series once we merged 2.87 :p | 11:01 |
| bauzas | I spent the whole Friday rechecking | 11:02 |
| brinzhang_ | bauzas: I register a bp, but I think that should push a spec to show more details, https://blueprints.launchpad.net/nova/+spec/define-cyborg-arq-binding-notification-event | 11:02 |
| bauzas | brinzhang_: ack, feel free to close the bug then with a link pointing to the bp | 11:02 |
| brinzhang_ | bauzas: yeah, I think that we cannot hit new microversion, that just should add the event base on mivroversion 2.82 | 11:03 |
| bauzas | I'm hungry, I can't think | 11:04 |
| * bauzas goes awol | 11:04 | |
| brinzhang_ | bauzas: ah, let me check in the future ^^ | 11:04 |
| *** sapd1 has joined #openstack-nova | 11:15 | |
| *** kplant has quit IRC | 11:27 | |
| *** kevinbenton has quit IRC | 11:27 | |
| *** kevinbenton has joined #openstack-nova | 11:27 | |
| brinzhang_ | bauzas: you set bug 1872934 to wishlist? IMO, it just a bug, that canbe done with bug fix, right? | 11:27 |
| openstack | bug 1872934 in OpenStack Compute (nova) "Accurately clean up ARQs resources during build_instances() in conductor" [Wishlist,Confirmed] https://launchpad.net/bugs/1872934 - Assigned to Brin Zhang (zhangbailin) | 11:27 |
| *** iurygregory is now known as iurygregory_afk | 11:31 | |
| *** sylar has joined #openstack-nova | 11:33 | |
| *** sylar is now known as kplant | 11:34 | |
| *** sapd1 has quit IRC | 11:37 | |
| *** ociuhandu has quit IRC | 11:42 | |
| hrw | https://review.opendev.org/#/c/715979/ - anyone with free few minutes to get aarch64 feature matrix updated? needs +2+w | 11:49 |
| *** sapd1 has joined #openstack-nova | 11:54 | |
| bauzas | brinzhang_: I did set back to Low https://bugs.launchpad.net/nova/+bug/1872934 | 11:55 |
| openstack | Launchpad bug 1872934 in OpenStack Compute (nova) "Accurately clean up ARQs resources during build_instances() in conductor" [Low,Confirmed] - Assigned to Brin Zhang (zhangbailin) | 11:55 |
| bauzas | brinzhang_: you can modify your own bugs FWIW | 11:55 |
| brinzhang_ | bauzas: I can not change the importance of the bug, so pin to you | 11:56 |
| brinzhang_ | bauzas: thanks ^ | 11:56 |
| bauzas | brinzhang_: you can add yourself to https://launchpad.net/~nova-bugs to triage bugs | 11:56 |
| bauzas | it's an open list | 11:57 |
| brinzhang_ | bauzas: ack, done, thanks | 11:58 |
| *** ttsiouts has quit IRC | 11:59 | |
| bauzas | brinzhang_: can I add you as a owner for 'cyborg' bugs ? (ie. within https://wiki.openstack.org/wiki/Nova/BugTriage#Tag_Owner_List ) | 12:02 |
| bauzas | brinzhang_: nothing really important, but in case people would like to discuss bugs, you could be pinged | 12:03 |
| bauzas | and sometimes if we see a lot of related bugs, we could ping you too | 12:03 |
| brinzhang_ | bauzas: you can, I also can send these bug to cyborg team ^ | 12:04 |
| *** udesale_ has joined #openstack-nova | 12:04 | |
| bauzas | that's not mean that you should fix all of the bugs, just be a liaison for the cyborg team | 12:04 |
| bauzas | brinzhang_: who is now the new Cyborg PTL ? | 12:04 |
| brinzhang_ | bauzas: understand :P | 12:05 |
| brinzhang_ | bauzas: Yumeng Bao <yumeng_bao@yahoo.com> | 12:05 |
| bauzas | found it https://governance.openstack.org/election/results/victoria/ptl.html | 12:05 |
| bauzas | kk | 12:05 |
| *** abhishek has joined #openstack-nova | 12:06 | |
| *** udesale has quit IRC | 12:06 | |
| bauzas | brinzhang_: done, thanks! https://wiki.openstack.org/wiki/Nova/BugTriage#Tag_Owner_List | 12:08 |
| brinzhang_ | ack | 12:08 |
| brinzhang_ | bauzas: can you change brinzhang_ to brinzhang ? | 12:09 |
| bauzas | sure | 12:09 |
| bauzas | done. | 12:10 |
| brinzhang_ | thanks | 12:10 |
| brinzhang_ | bauzas: good morning, hope have a good day ^, I will go home | 12:12 |
| bauzas | ++ | 12:12 |
| bauzas | (2pm14 atm, so I'm done with the morning :p | 12:12 |
| brinzhang_ | ah, good afternoon. I am in evening :) | 12:13 |
| brinzhang_ | +8 for UTC time | 12:13 |
| *** brinzhang_ has quit IRC | 12:14 | |
| *** abhishek has quit IRC | 12:15 | |
| gibi | slaweq, melwitt: I put https://bugs.launchpad.net/nova/+bug/1844929 back to Confirmed as the issue is still visible 5 days after the fix https://review.opendev.org/#/c/717662/ was merged | 12:16 |
| openstack | Launchpad bug 1844929 in OpenStack Compute (nova) train "grenade jobs failing due to "Timed out waiting for response from cell" in scheduler" [High,In progress] - Assigned to melanie witt (melwitt) | 12:16 |
| *** factor has joined #openstack-nova | 12:16 | |
| gibi | slaweq: thanks for the headsup | 12:16 |
| sean-k-mooney | gibi: (efried just tagging you as an fyi) regarding the provider yaml feature https://review.opendev.org/#/q/topic:bp/provider-config-file+(status:open+OR+status:merged) | 12:18 |
| sean-k-mooney | i could proably finish that off early next cycle as it really just need a rebase and maybe some test fix ups | 12:19 |
| sean-k-mooney | but im wondering is this is still a feature that people are interested in | 12:20 |
| *** ttsiouts has joined #openstack-nova | 12:20 | |
| sean-k-mooney | the primary customer of this was intel for enabling RDT | 12:20 |
| sean-k-mooney | so maybe alex_xu has input ^ | 12:20 |
| gibi | sean-k-mooney: I think it is worth to ask it on the ML and if nobody shows up then we can simply stop | 12:20 |
| gibi | yeah, or alex_xu knows more | 12:21 |
| gibi | sean-k-mooney: personally I (i.e. my employer) have no priority about that feature | 12:21 |
| gibi | it is pretty close so I don't have problem reviewing it next cycle but I totally understand if you need to downprioritize it from your side | 12:22 |
| sean-k-mooney | from a redhat persective the only request we have are from intel in relation to support of RDT in our product. but we dont nessiscarly have customer of OSP asking for it. | 12:22 |
| sean-k-mooney | gibi: ya the main thing i have to do other then rebase was to make the testing consitent | 12:23 |
| sean-k-mooney | you asked to either use the yaml file or python but not do positive test in yaml and negitive in python | 12:24 |
| sean-k-mooney | i had already adressed the other feedback but i also dont want to reporpose specs/feature if noone will end up using it | 12:24 |
| gibi | sean-k-mooney: totally agree with you. I suggest to wait from alex_xu's response about it | 12:26 |
| bauzas | gibi: fwiw, I'm about to confirm a gate issue https://bugs.launchpad.net/nova/+bug/1863889 | 12:26 |
| openstack | Launchpad bug 1863889 in OpenStack Compute (nova) "Revert resize problem in neutron-tempest-dvr-ha-multinode-full" [Medium,Confirmed] | 12:26 |
| gibi | until that it is yet another unfinished feature | 12:26 |
| bauzas | no need to keep it 'new' | 12:26 |
| gibi | bauzas: I've just confirmed it | 12:26 |
| bauzas | ah lol | 12:26 |
| gibi | and tagged | 12:26 |
| bauzas | we're fighting over the same list :p | 12:26 |
| gibi | bauzas: I will go and look at the bug list from the top to avoid such conflict | 12:26 |
| bauzas | take the semaphore if you want :p | 12:27 |
| *** mriedem has joined #openstack-nova | 12:27 | |
| bauzas | gibi: I personnally look at this query ordered by age https://bugs.launchpad.net/nova/+bugs?search=Search&field.status=New&orderby=-datecreated&start=0 | 12:27 |
| gibi | I think the list is long enough that we can eat it from two different direction | 12:27 |
| bauzas | gibi: I could also briefly look at the 'in-progress' recent bugs that haven't been triaged yet | 12:28 |
| gibi | do you walk that list from the top? | 12:28 |
| bauzas | gibi: I do, because $regression check | 12:28 |
| gibi | ok, then I will go from the bottom | 12:28 |
| bauzas | old bugs that are more than 1 year old aren't really important so close to RC1 | 12:28 |
| bauzas | gibi: well, I stopped triaging for a while and I'm seeing the same bugs | 12:28 |
| bauzas | good luck with that | 12:28 |
| bauzas | gibi: no, please, keep looking at the top list, I'll do something else with bugs that is important close to RC1 | 12:29 |
| bauzas | gibi: like i said, I've stopped a couple of times some bugs that were in progress and were crucial because they were fixing a regression | 12:30 |
| bauzas | we were just missing them since someone proposed a patch and didn't yell it | 12:30 |
| gibi | bauzas: OK. I will look at the top for the next 30 minutes, then I have to jump on a call | 12:30 |
| gibi | I will let you know when I released the semaphor | 12:30 |
| bauzas | as soon as someone proposes a patch, then the related bugs disappear from our radara | 12:30 |
| bauzas | radar* | 12:31 |
| *** ociuhandu has joined #openstack-nova | 12:31 | |
| gibi | bauzas: yes, good point. If you have a good query for those bugs then please share | 12:31 |
| * gibi is noob in launchpad | 12:31 | |
| bauzas | gibi: easy peasy, sec | 12:31 |
| *** abhishek has joined #openstack-nova | 12:31 | |
| *** ociuhandu has quit IRC | 12:32 | |
| *** ociuhandu has joined #openstack-nova | 12:32 | |
| bauzas | gibi: https://frama.link/Z-EVGtyx | 12:34 |
| bauzas | enjoy the list, 112 potential regressions :p | 12:34 |
| bauzas | maybe 20 if we only consider the last 6 months | 12:35 |
| bauzas | those bugs are in general written by developers who found a bug | 12:36 |
| bauzas | whether the issue is new (hence a regression) or always existing, that's the crux of the problem :) | 12:36 |
| bauzas | gibi: like, first case https://bugs.launchpad.net/nova/+bug/1872385 | 12:38 |
| openstack | Launchpad bug 1872385 in OpenStack Compute (nova) "delete sub resource provider in tree" [Undecided,In progress] - Assigned to Wenping Song (wenping1) | 12:38 |
| bauzas | yay... | 12:39 |
| *** belmoreira has joined #openstack-nova | 12:39 | |
| *** belmoreira has quit IRC | 12:39 | |
| *** artom has quit IRC | 12:39 | |
| bauzas | gibi: we don't have yet ussuri-rc-potential tag ? | 12:40 |
| sean-k-mooney | not offically but im already using it | 12:40 |
| sean-k-mooney | i was going to go update the tags | 12:40 |
| sean-k-mooney | sorry not the rc on i am using the backport potential one | 12:41 |
| bauzas | sean-k-mooney: just saying https://bugs.launchpad.net/nova/+bugs?field.tag=ussuri-rc-potential | 12:41 |
| bauzas | sean-k-mooney: that's different | 12:41 |
| sean-k-mooney | yes it is | 12:41 |
| bauzas | sean-k-mooney: you shouldn't use this tag (backport) yet | 12:41 |
| sean-k-mooney | the cyborg bug is not an rc candiate | 12:41 |
| sean-k-mooney | so i notice that both were missing | 12:41 |
| bauzas | sean-k-mooney: sorry I don't get you ? | 12:44 |
| *** nweinber has joined #openstack-nova | 12:44 | |
| gibi | bauzas: I did not needed the tag yet | 12:44 |
| gibi | bauzas: I thought it is created when first needed | 12:45 |
| sean-k-mooney | from https://bugs.launchpad.net/nova/+manage-official-tags | 12:45 |
| gibi | bauzas: is there another way to pre-create a tag? | 12:45 |
| sean-k-mooney | i have moved ussuri-backport-potential to offical | 12:45 |
| sean-k-mooney | but i cant add ussuri-rc-potential for some reason | 12:45 |
| bauzas | gibi: no, you just use it on purpose but then you can promote it 'official' | 12:46 |
| gibi | me neither | 12:46 |
| bauzas | gibi: it will then raise the visibility | 12:46 |
| gibi | OK | 12:46 |
| bauzas | hold on, doing it for one bug | 12:46 |
| bauzas | gibi: there https://bugs.launchpad.net/nova/+bugs?field.tag=ussuri-rc-potential | 12:47 |
| gibi | thanks | 12:47 |
| bauzas | gibi: look at the right panels, you can see the list of tags | 12:47 |
| bauzas | and there is one link for 'edit official tags' | 12:47 |
| slaweq | gibi: thx a lot for checking this grenade issue | 12:47 |
| sean-k-mooney | https://bugs.launchpad.net/nova/+manage-official-tags | 12:47 |
| bauzas | which leads to https://bugs.launchpad.net/nova/+manage-official-tags | 12:47 |
| bauzas | jinx | 12:48 |
| gibi | OK, thanks I made it official | 12:48 |
| bauzas | and then, I'm adding ussuri-rc-potential to the left list | 12:48 |
| gibi | slaweq: I hope melwitt has some future ideas. she looked a lot at this bug | 12:49 |
| *** rpittau|bbl is now known as rpittau | 12:49 | |
| bauzas | sean-k-mooney: gibi: the ussuri-backport-potential is mostly used for knowing whether it's a good backport candidate after GA | 12:49 |
| sean-k-mooney | bauzas: yep i know | 12:50 |
| bauzas | but we should iterate over the list of ussuri-rc-potential before and after RC1 | 12:50 |
| bauzas | until we get 0 candidates | 12:50 |
| sean-k-mooney | bauzas: i wanted to also flag that this was not intended to be fixed in the rc | 12:50 |
| bauzas | the -backport tag ? yes | 12:50 |
| bauzas | was that a question ? | 12:51 |
| bauzas | like I said earlier "(14:49:59) bauzas: sean-k-mooney: gibi: the ussuri-backport-potential is mostly used for knowing whether it's a good backport candidate after GA" | 12:51 |
| sean-k-mooney | yep im going to be reuploading it later today but i do not intend the patch to merge untill after ussuri is released then ill backport it | 12:51 |
| bauzas | *after GA* | 12:51 |
| *** ttsiouts has quit IRC | 12:53 | |
| *** KeithMnemonic has joined #openstack-nova | 12:54 | |
| bauzas | sean-k-mooney: a cyborg patch you mean ? I'm confused again :) | 12:55 |
| gibi | bauzas: I'm releasing the bug lock for now, need to jump on a call | 12:57 |
| bauzas | gibi: ack | 12:58 |
| * bauzas wonders if he can do upstream bug triage while listening to some mandatory e-training course | 12:58 | |
| *** ratailor has quit IRC | 13:04 | |
| gibi | sean-k-mooney: do you have a feeling about https://bugs.launchpad.net/nova/+bug/1863038 ? For me it seems that the routed network feautre was never considered ironic in the first place. | 13:08 |
| openstack | Launchpad bug 1863038 in OpenStack Compute (nova) "Nova need to consider ironic node as 'host'" [Undecided,New] | 13:08 |
| bauzas | routed networks aren't a thing yet, right? | 13:10 |
| *** ttsiouts has joined #openstack-nova | 13:13 | |
| gmann | stephenfin: chekcing.. | 13:13 |
| *** lbragstad has joined #openstack-nova | 13:14 | |
| gibi | bauzas: it works, in a narrow case | 13:16 |
| bauzas | kk | 13:16 |
| gibi | bauzas: when a port is pre-create with ip_allocation=deferred config | 13:16 |
| bauzas | I missed that then | 13:16 |
| gibi | for that case there was no need for nova change | 13:16 |
| gibi | iff every network has a segment for every compute | 13:17 |
| gibi | but overall you are right there is no real, good, tested routed net support in nova | 13:18 |
| gmann | stephenfin: replied. please check | 13:22 |
| gmann | stephenfin: https://review.opendev.org/#/c/662968/9 | 13:22 |
| bauzas | gibi: ahah, now I understand why I never heard of this as 'implemented' | 13:23 |
| bauzas | it was all proxied to neutron then | 13:23 |
| bauzas | gotcha | 13:23 |
| gibi | sean-k-mooney: bauzas: we will talk about route net support in nova during the PTG so this ironic case might be also a question for that discussion | 13:24 |
| *** artom has joined #openstack-nova | 13:25 | |
| sean-k-mooney | sorry was distracted ill read scrollback | 13:25 |
| sean-k-mooney | bauzas: well we jsut dont have schduling support | 13:28 |
| sean-k-mooney | you can sort of force it to work by mapping AZs to routed segments but that is a pain and a hack | 13:29 |
| sean-k-mooney | if you map an AZ to a segment then the fact we dont live migrate across AZs by default make it work fine provided you use ip_allocaiton=deffered | 13:29 |
| efried | sean-k-mooney, gibi: I sure would like to see the provider yaml feature land. It's something we've talked about for years and years, and it's the first step toward being able to do PCI passthrough whitelisting+configuration properly. If you're still planning to represent PCI devices as child providers _ever_, you're going to want it or something like it. And the current work is *so* close. | 13:30 |
| efried | tbc, my position on this is not related to employer, but to what I see as the right direction for nova | 13:31 |
| sean-k-mooney | efried: yep althouhg i dont really like how we are doing to modeling if we are going to use it for pci if im being totally honest | 13:31 |
| sean-k-mooney | that said its not awful either | 13:31 |
| sean-k-mooney | its just missing the corralation to the actul devices | 13:32 |
| sean-k-mooney | which could be added in a later schema version | 13:32 |
| efried | oh, as written it's not ready for PCI yet, for sure. | 13:32 |
| efried | we dialed back to the bare bones to get a mvp that would fit in a cycle | 13:33 |
| sean-k-mooney | efried: my alternitve spec for this feature start with pci modeling as the MVP with what the current feature does as out of scope | 13:33 |
| *** liuyulong has joined #openstack-nova | 13:33 | |
| sean-k-mooney | well kind of | 13:34 |
| sean-k-mooney | anyway i do think its worth completing give how close it is | 13:34 |
| sean-k-mooney | efried: would you remind removing your -2 on the bottom patch and maybe gibi could add his | 13:34 |
| sean-k-mooney | so that we dont need to find you to do that when we get to victoria | 13:35 |
| bauzas | stephenfin: i'd appreciate some reviews of https://review.opendev.org/#/c/712741/2 btw. | 13:35 |
| efried | oh, I have a -2? Fixing... | 13:35 |
| sean-k-mooney | efried: it was a procedual one | 13:35 |
| bauzas | efried: my only concers with the providers YAML file is that it makes the DSL so placement-specific... | 13:36 |
| bauzas | I'd have appreciate some kind of abstraction | 13:36 |
| sean-k-mooney | well its a yaml file for modeling things in placement | 13:36 |
| sean-k-mooney | but that is a fair observation | 13:36 |
| openstackgerrit | Marcin Juszkiewicz proposed openstack/nova master: libvirt: check for AMD SEV only on x86-64 https://review.opendev.org/714425 | 13:36 |
| sean-k-mooney | efried: https://review.opendev.org/#/c/673341/42 | 13:37 |
| *** rcernin has quit IRC | 13:37 | |
| bauzas | sean-k-mooney: I know | 13:37 |
| hrw | fixed pep8 | 13:37 |
| bauzas | sean-k-mooney: and I reviewed the spec, I think i even left a +2 on it | 13:37 |
| bauzas | sean-k-mooney: but my concern remains | 13:37 |
| efried | sean-k-mooney, gibi: removed the -2 | 13:37 |
| gibi | efried: tanks | 13:37 |
| gibi | thanks | 13:38 |
| bauzas | i think we mostly have a consensus about to leaking all the request group syntax into our flavor extraspecs | 13:38 |
| bauzas | about not* | 13:38 |
| bauzas | I'd appreciate if we could do the same for inventories | 13:38 |
| sean-k-mooney | gibi: want to replace it with a procedual one since we are past code freeze and this need to be reporposed | 13:38 |
| sean-k-mooney | or we can just leave it since we all know its for Victoria at this point | 13:38 |
| efried | bauzas: well, it's not really that provider yaml is using a placement abstraction. It's that we designed placement with an abstraction and are using that same abstraction for other things (including cyborg and bw). Also worth noting that the *provider* yaml is completely targeted at *providers* which are a placement thing... | 13:39 |
| sean-k-mooney | bauzas: well i want to remove the request groups form the flavor extra specs entirely | 13:39 |
| bauzas | efried: I don't disagree hence my +2 on the spec | 13:39 |
| efried | there's no request group syntax in provider yaml. That wouldn't make sense. | 13:39 |
| bauzas | efried: I'm just saying we need to keep it crystal-clear for ops | 13:39 |
| efried | ++ | 13:39 |
| sean-k-mooney | efried: ya i think bauzas was just concerned about how the resouce would be consumed | 13:40 |
| sean-k-mooney | they may be tempeted to use the request group syntax in the falvor | 13:40 |
| bauzas | efried: the request group syntax was an example about how things could become difficult if we expose them thru extrasepcs | 13:40 |
| bauzas | sean-k-mooney: correct | 13:41 |
| gibi | sean-k-mooney: I will simply leave it as is, I trust the cores that they will not approve it | 13:41 |
| efried | Yes, I agree it would be nice if we could close the door on direct rg syntax in extra specs. (Recall I advocated hard for using flavor-ese rather than placement-ese wherever possible. Dan pushed back at least wrt non-granular and I capitulated; but granular is still worth fighting for imo.) | 13:42 |
| gibi | bauzas: re-acquire the bug triage lock for bugs in New state | 13:42 |
| bauzas | gibi: I had to jump on a mandatory training course I have to do by today EOB | 13:43 |
| bauzas | about 'open hybrid cloud' | 13:43 |
| bauzas | where I'm teached about what is 'cloud' and 'cloud computing' | 13:43 |
| * bauzas loves his job | 13:43 | |
| gibi | I'm sorry for you | 13:43 |
| sean-k-mooney | for what its worth this was my proposal for how to do this which had much less of a direct use of placmenet semantics https://review.opendev.org/#/c/603805/2/specs/stein/approved/device-discovery-policy-lib.rst | 13:44 |
| bauzas | gibi: honestly, it's not really for today, I'm exagerating | 13:44 |
| sean-k-mooney | still used resouce classes and trait but that was it | 13:44 |
| bauzas | gibi: but I don't like having paperwork like this to slip over time | 13:44 |
| *** Liang__ has joined #openstack-nova | 13:53 | |
| *** abhishek has quit IRC | 13:53 | |
| *** Liang__ is now known as LiangFang | 13:54 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in servers attributes Policies https://review.opendev.org/719729 | 13:54 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add new default roles in servers attributes policies https://review.opendev.org/719730 | 13:57 |
| *** ttsiouts has quit IRC | 14:07 | |
| *** ttsiouts has joined #openstack-nova | 14:09 | |
| gibi | nova API experts I need help what would be the expected behavior for the quering GET /v2.1 and GET /v2.1/ without a token. As per https://bugs.launchpad.net/nova/+bug/1862477 | 14:10 |
| openstack | Launchpad bug 1862477 in OpenStack Compute (nova) "API returns 401 for /v2.1 (when no auth provided)" [Undecided,Confirmed] | 14:10 |
| gibi | alex_xu, gmann, dansmith ^^ | 14:11 |
| bauzas | gibi: AFAIK you get the routes | 14:11 |
| bauzas | gibi: lemme find the right thing | 14:11 |
| gibi | I think we have an inconsistency between /v2.1 and / and between with token and without token | 14:12 |
| gibi | see my comment in the review | 14:12 |
| bauzas | gibi: yeah you're right with the triage I think | 14:12 |
| gibi | in the bug | 14:12 |
| bauzas | actually sec | 14:12 |
| gibi | bauzas: is this something we broke recently? | 14:12 |
| bauzas | gibi: no I don't think | 14:13 |
| dansmith | I think efried and mordred just changed something about this, no? | 14:13 |
| bauzas | gibi: I just wonder whether we should return 401 by design or no | 14:13 |
| dansmith | fwiw, I've never understood why we should return *anything* to an unauthorized client | 14:13 |
| bauzas | but I tend to say we should accept the root page even for the /v2.1 document | 14:13 |
| efried | dansmith: yes, we recently merged a couple of fixes related to getting version discovery docs unauthenticated. Finding... | 14:14 |
| bauzas | / et /v2.1 are two different resources in my mind | 14:14 |
| efried | dansmith: https://review.opendev.org/#/c/685181/ | 14:14 |
| dansmith | gibi: ^ | 14:14 |
| gibi | thanks | 14:14 |
| bauzas | / is the root page, I'm okay with not asking to be auth'd to see it | 14:14 |
| efried | The versioned discovery docs also need to not require auth. | 14:15 |
| bauzas | /v2.1 is an endpoint, I could understand we would require to be auth'd | 14:15 |
| gibi | hold on, that patch is newer than my devstack | 14:15 |
| sean-k-mooney | bauzas: i dont think we should assume that all endpoint need auth | 14:15 |
| bauzas | I don't know what to say | 14:16 |
| bauzas | it's just a design decision | 14:16 |
| sean-k-mooney | bauzas: we should contol it via policy | 14:16 |
| sean-k-mooney | and i think the default policy for /2.1 should be no auth | 14:16 |
| sean-k-mooney | for version discovery to work | 14:16 |
| bauzas | I think I entered a rathole, STEP BACK, STEP BACK !!! | 14:16 |
| gibi | I will test those queries with the above patch applied | 14:16 |
| efried | Oh, yeah, https://bugs.launchpad.net/nova/+bug/1862477 should be resolved via the above patch. | 14:17 |
| openstack | Launchpad bug 1862477 in OpenStack Compute (nova) "API returns 401 for /v2.1 (when no auth provided)" [Undecided,Confirmed] | 14:17 |
| bauzas | sean-k-mooney: yeah you make sense | 14:17 |
| efried | ...as well as the two bugs tagged in the commit message. | 14:17 |
| bauzas | gibi: mark it as duplicate of https://bugs.launchpad.net/nova/+bug/1862477 AFAICS | 14:17 |
| bauzas | gibi: want me to do it ? | 14:18 |
| gibi | bauzas, efried: thanks. I will quickly test it then mark it dublicate | 14:18 |
| bauzas | ack | 14:18 |
| *** iokiwi has quit IRC | 14:19 | |
| bauzas | gmann: that's a very old bug, right ? https://bugs.launchpad.net/nova/+bug/1871665 | 14:20 |
| openstack | Launchpad bug 1871665 in OpenStack Compute (nova) "servers actions (many) API policy is allowed for everyone even policy defaults is admin_or_owner " [Undecided,In progress] - Assigned to Ghanshyam Mann (ghanshyammann) | 14:20 |
| bauzas | gmann: I never remember this worked once | 14:20 |
| *** iokiwi has joined #openstack-nova | 14:20 | |
| bauzas | (that surprised me like 5 years ago when I was trying to understand how oslo.policy was in use with nova) | 14:20 |
| sean-k-mooney | gibi: regarding https://bugs.launchpad.net/nova/+bug/1863038 nova should be seting the instanace.host to the ironic node uuid right and we should be setting the instance.host in the neutron port binding profie to | 14:21 |
| openstack | Launchpad bug 1863038 in OpenStack Compute (nova) "Nova need to consider ironic node as 'host'" [Wishlist,Confirmed] | 14:21 |
| sean-k-mooney | i can go check that but i would be suprised if we are pasing the comput node host instead of the hypervior host in the call to neutron | 14:22 |
| gibi | I also would need to read the code to be sure about what we set node or host | 14:24 |
| *** mkrai has quit IRC | 14:26 | |
| sean-k-mooney | looking at teh migration code we are using instance.host im looking now at some of the other code paths but we appear to be using instance.host in most case if not all | 14:27 |
| sean-k-mooney | gibi: i guess allocate for instance delegates to the driver https://github.com/openstack/nova/blob/01da968978db8c01920616fe594bea6eda2cba57/nova/compute/manager.py#L1724 | 14:29 |
| sean-k-mooney | which for ironic retuns none https://github.com/openstack/nova/blob/01da968978db8c01920616fe594bea6eda2cba57/nova/virt/ironic/driver.py#L1795 | 14:30 |
| gmann | bauzas: correct, almost all of our admin_or_owner has this issue. founded and fixed in policy defaults work | 14:30 |
| bauzas | gmann: I just rechecked the patch | 14:32 |
| sean-k-mooney | gibi: they do the vif attach here https://github.com/openstack/nova/blob/01da968978db8c01920616fe594bea6eda2cba57/nova/virt/ironic/driver.py#L1545-L1546 | 14:33 |
| gmann | thanks | 14:33 |
| sean-k-mooney | also using the uuid so this fells like either an ironic bug or not a bug at all that said i have not traced all the codepaths so there still could be a nova bug but i doubt it. | 14:34 |
| gibi | sean-k-mooney: thanks for the analysis. I deferr this to the PTG discussion. It is clearly not a regression to deal with now | 14:35 |
| sean-k-mooney | gibi: well im not sure the bug report is valid | 14:35 |
| *** dklyle has joined #openstack-nova | 14:35 | |
| sean-k-mooney | e.g. i dont know if we actully are using the host it looks like we shoudl be useing the ironic node uuid | 14:36 |
| *** ociuhandu has quit IRC | 14:36 | |
| *** mkrai has joined #openstack-nova | 14:37 | |
| gibi | sean-k-mooney: I think the reporter also said that for ironic case neutron gets the node uuid and try to use that as a host for the nova host aggregate API | 14:37 |
| sean-k-mooney | well there is some weirdnes with host aggreates and ironic node | 14:39 |
| sean-k-mooney | its not clear to me if we expect each compute serivce to be a member of the host aggreate or each compute/ironic node | 14:40 |
| sean-k-mooney | i think in the ironic case we would want each fo the ironic nodes to be set as member of the hsot aggreate | 14:41 |
| sean-k-mooney | since they can move between compute services | 14:41 |
| sean-k-mooney | they code they linked too however is querying placment | 14:41 |
| sean-k-mooney | https://github.com/openstack/neutron/blob/master/neutron/services/segments/plugin.py#L253 | 14:41 |
| *** mkrai_ has joined #openstack-nova | 14:42 | |
| *** mkrai_ has quit IRC | 14:42 | |
| *** mkrai_ has joined #openstack-nova | 14:42 | |
| *** mkrai has quit IRC | 14:42 | |
| gibi | sean-k-mooney: I think neutron creates a host aggregate that is then mapped to placement aggregate by nova and then neutron puts the IP RP to into that placement aggregate | 14:42 |
| gibi | or something like that (it is pretty complicate) | 14:43 |
| sean-k-mooney | gibi: neutron should be creating both a RP and hostaggreate makeing the RP a sharing resouce provider of ips | 14:43 |
| *** tkajinam has joined #openstack-nova | 14:43 | |
| hrw | bauzas: https://review.opendev.org/714425 (AMD SEV check) updated with tests | 14:43 |
| *** ociuhandu has joined #openstack-nova | 14:43 | |
| sean-k-mooney | hrw: that looks like an optimiation | 14:44 |
| sean-k-mooney | is this breaking something if so you shoudl file a bug | 14:44 |
| gibi | sean-k-mooney: so for ironic neturon gets the node uuid for libvirt neutron gets the hostname and the neutron in both case use the given id as a member of an aggregate but in nova host aggregates works on compute host level not on compute node level | 14:47 |
| *** mlavalle has joined #openstack-nova | 14:47 | |
| sean-k-mooney | gibi: well this is all handeld at the compute manager level | 14:48 |
| sean-k-mooney | for everythign other then ironic it gets the instance.host | 14:48 |
| gibi | yepp | 14:48 |
| gibi | this is the key difference | 14:48 |
| sean-k-mooney | for ironic it gets the node uuid | 14:48 |
| sean-k-mooney | the instance.host shoudl be the node uuid | 14:48 |
| sean-k-mooney | in the ironic case | 14:49 |
| sean-k-mooney | so it shoudl be the same | 14:49 |
| gibi | yepp, but I don't think you can add a node to a nova host aggregate | 14:49 |
| gibi | you can only add a host there | 14:49 |
| sean-k-mooney | isnint instance.host always set to the hypervior hostname | 14:50 |
| sean-k-mooney | so i think the neutron code only ever works in that case because for non ironic host the hypervior hostname and compute service host are the same | 14:50 |
| gibi | yepp, that could be the case | 14:50 |
| hrw | sean-k-mooney: it does not break anything. just looks weird when on aarch64 host I see checks of x86 features in log | 14:51 |
| sean-k-mooney | hrw: its not an x86 feature | 14:51 |
| sean-k-mooney | its not part of the instuction set | 14:51 |
| sean-k-mooney | its an AMD extention | 14:51 |
| hrw | sean-k-mooney: AMD SVE is not aarch64 feature as well ;D | 14:51 |
| *** sapd1 has quit IRC | 14:52 | |
| sean-k-mooney | yes but its incrrect to call it x86 | 14:52 |
| hrw | sean-k-mooney: when you run 'lscpu' it is listed in features. along with 'I can run KVM' which is not part of ISA iirc too | 14:52 |
| sean-k-mooney | sure but the feature are not instuctionsets | 14:53 |
| *** sapd1 has joined #openstack-nova | 14:53 | |
| sean-k-mooney | several of the security flaws are listed as "features" too | 14:53 |
| sean-k-mooney | anyway if you file a bug for that patch im +1 on it | 14:54 |
| *** mkrai_ has quit IRC | 14:54 | |
| hrw | sean-k-mooney: looking at history of amd64/x86-64/x64 it is sometimes hard to notice who copied who ;D | 14:54 |
| *** mkrai_ has joined #openstack-nova | 14:54 | |
| *** dtantsur is now known as dtantsur|brb | 14:55 | |
| *** spsurya_ has joined #openstack-nova | 14:55 | |
| *** mkrai_ has quit IRC | 14:55 | |
| sean-k-mooney | well that is irrelevent to this conversation but x64 was itainiaum amd64 is the 64 bit extentison amd developed and x86_64 is the name for x86 with the amd64 extnetions intel licened form amd | 14:56 |
| hrw | sean-k-mooney: ok | 14:56 |
| hrw | ia64 was itanium. x64 is ms name for x86-64/amd64 | 14:56 |
| sean-k-mooney | ah yep correct | 14:57 |
| hrw | my previous (x86-64) laptop has 'Intel Itanium Inside' sticker on it - for pure fun of it | 14:58 |
| *** priteau has joined #openstack-nova | 15:00 | |
| melwitt | gibi, slaweq: we need to merge the train backport in order to see the bug disappear because grenade runs train first, then upgrades to ussuri https://review.opendev.org/718934 | 15:02 |
| gibi | melwitt: bahh, you are correct. /o\ putting the bug back to FixReleased state | 15:03 |
| *** gyee has joined #openstack-nova | 15:04 | |
| melwitt | and that patch was stuck behind https://review.opendev.org/719120 but that merged, so it is good to review now | 15:04 |
| gibi | elod, lyarwood: could you please look at ^^ ? | 15:04 |
| *** lpetrut has quit IRC | 15:05 | |
| openstackgerrit | Marcin Juszkiewicz proposed openstack/nova master: libvirt: check for AMD SEV only on x86-64 https://review.opendev.org/714425 | 15:07 |
| hrw | sean-k-mooney: bug created, linked | 15:07 |
| elod | gibi melwitt : this one, right? https://review.opendev.org/#/c/718934/ anyway, looking | 15:08 |
| gibi | elod: yepp, that one, thanks | 15:09 |
| *** tbachman has quit IRC | 15:11 | |
| *** tbachman has joined #openstack-nova | 15:12 | |
| lyarwood | gibi / melwitt ; ack done | 15:14 |
| gibi | lyarwood: thnks | 15:16 |
| *** ttsiouts has quit IRC | 15:22 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova stable/stein: Use placement stable version for functional job https://review.opendev.org/719121 | 15:27 |
| *** ttsiouts has joined #openstack-nova | 15:29 | |
| gibi | bauzas: releasing bug triage lock as I'm leaving for today | 15:34 |
| bauzas | gibi: ack, will continue then | 15:34 |
| bauzas | gibi: I triaged a couple of in-progress, nothing crucial | 15:35 |
| melwitt | stephenfin: fyi brief discussion from yesterday on the possibility of bringing back the ability to show user_data in the cli with a switch http://eavesdrop.openstack.org/irclogs/%23openstack-nova/%23openstack-nova.2020-04-14.log.html#t2020-04-14T22:59:39 | 15:35 |
| gibi | bauzas: thanks. nothing RC blocking on my side either | 15:35 |
| *** iurygregory_afk is now known as iurygregory | 15:41 | |
| *** LiangFang has quit IRC | 15:43 | |
| *** sapd1 has quit IRC | 15:49 | |
| *** sapd1 has joined #openstack-nova | 15:49 | |
| *** tkajinam has quit IRC | 15:53 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add new default roles in servers attributes policies https://review.opendev.org/719730 | 15:54 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing remaining servers policies https://review.opendev.org/720104 | 15:54 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in remaining servers Policies https://review.opendev.org/720106 | 15:54 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add new default roles in remaining servers policies https://review.opendev.org/720116 | 15:54 |
| *** maciejjozefczyk has quit IRC | 16:09 | |
| *** dtantsur|brb is now known as dtantsur | 16:11 | |
| *** priteau has quit IRC | 16:12 | |
| openstackgerrit | Merged openstack/nova master: fup: Remove the use of the term `unstable rescue` INFO logs https://review.opendev.org/718451 | 16:16 |
| openstackgerrit | Merged openstack/nova master: fup: Add missing docstrings from get_rescue_device|bus diskinfo funcs https://review.opendev.org/718464 | 16:16 |
| *** psachin has quit IRC | 16:17 | |
| *** ociuhandu has quit IRC | 16:20 | |
| openstackgerrit | Lee Yarwood proposed openstack/nova stable/train: libvirt: Calculate disk_over_committed for raw instances https://review.opendev.org/718964 | 16:24 |
| gmann | melwitt: lyarwood stein backport is ready now (train is merges) - https://review.opendev.org/#/c/719121/ | 16:28 |
| openstackgerrit | Merged openstack/nova master: libvirt: Change UEFI check to handle AArch64 better https://review.opendev.org/714311 | 16:29 |
| melwitt | gmann: I thought you were gonna re-cherry-pick that one from the train change and remove the conflict note from the commit message? | 16:29 |
| *** yaawang has joined #openstack-nova | 16:32 | |
| hrw | yay ;D | 16:32 |
| gmann | melwitt: there was conflict form train also due to deps url | 16:32 |
| melwitt | gmann: haha oh | 16:33 |
| gmann | due to git.openstack.org and opendev.org | 16:33 |
| *** rpittau is now known as rpittau|afk | 16:33 | |
| melwitt | nevermind then | 16:33 |
| *** yaawang_ has quit IRC | 16:34 | |
| *** ociuhandu has joined #openstack-nova | 16:34 | |
| *** maciejjozefczyk has joined #openstack-nova | 16:35 | |
| hrw | does https://review.opendev.org/#/c/712607/ needs another 7h recheck? | 16:35 |
| *** evrardjp has quit IRC | 16:37 | |
| *** evrardjp has joined #openstack-nova | 16:37 | |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: fup: Fix [workarounds]/rbd_volume_local_attach config docs https://review.opendev.org/718100 | 16:38 |
| lyarwood | stephenfin: ^ no idea how that ended up in merge conflict, can you +W that again if it's still okay? | 16:39 |
| *** maciejjozefczyk has quit IRC | 16:41 | |
| sean-k-mooney | huh i taught that already merged honestly | 16:41 |
| openstackgerrit | Marcin Juszkiewicz proposed openstack/nova master: [WIP] CI: add tempest-integrated-compute-aarch64 job https://review.opendev.org/714439 | 16:42 |
| sean-k-mooney | oh it did this is just a fix up for the config | 16:42 |
| *** udesale_ has quit IRC | 16:49 | |
| stephenfin | melwitt: I saw that, but wasn't sure what the conclusion was | 16:49 |
| melwitt | stephenfin: no conclusion yet, wanted to see what you thought about it | 16:50 |
| melwitt | since you had removed the user_data, what you thought about adding it back but by flag only | 16:50 |
| stephenfin | Realistically I don't think we'd ever want that information included in raw format as part of the show command, but we clearly want it somehow. How about a new command? | 16:51 |
| *** dtantsur is now known as dtantsur|afk | 16:53 | |
| melwitt | I guess that seems heavier than what I was imagining. to just have it be like nova show --include-user-data or something like that | 16:54 |
| stephenfin | Hmm, so it's both the 'nova boot' and 'nova show' commands that are affected | 16:54 |
| melwitt | and rebuild apparently | 16:54 |
| stephenfin | yup | 16:54 |
| melwitt | it's not something "commonly" used but it's useful if an end user wants to see, did I pass in the user_data I expected, or what did I pass in user_data earlier | 16:55 |
| sean-k-mooney | melwitt: can you see that via the metadata api | 16:56 |
| melwitt | anyway, I wanted to run it by you in case you're strongly against putting it back via --<option> | 16:56 |
| melwitt | sean-k-mooney: I think you could but that if you're config drive only, you wouldn't be able to | 16:57 |
| sean-k-mooney | oh we dont store the user data in the db do we | 16:57 |
| melwitt | we do | 16:58 |
| sean-k-mooney | sorry im thinking of injected files | 16:58 |
| sean-k-mooney | those are the ones that are only in the config drive | 16:58 |
| melwitt | yeah, sorry, I mean like if you're not running the metadata service. and are config drive only deployment | 16:59 |
| melwitt | (example: verizon media) | 16:59 |
| sean-k-mooney | ya if you dont have metadata api then you would have to check the config drive unless we dumped it as part of nova show | 16:59 |
| stephenfin | and we don't provide a way to configure output or output formats either. Drat | 16:59 |
| sean-k-mooney | you mean in nova client | 17:00 |
| stephenfin | yup | 17:00 |
| sean-k-mooney | then we cant alter the format without potentally breaking people | 17:00 |
| *** ttsiouts has quit IRC | 17:00 | |
| stephenfin | Yes, that's exactly what's happening here :) | 17:01 |
| *** ociuhandu has quit IRC | 17:01 | |
| stephenfin | At least it was a major version bump, but still | 17:01 |
| sean-k-mooney | of nova client right | 17:01 |
| *** ttsiouts has joined #openstack-nova | 17:01 | |
| sean-k-mooney | i assume we are talking about the inclution of OS-EXT-SRV-ATTR:user_data in the nova show respocne right | 17:02 |
| sean-k-mooney | is there a patch i can look at | 17:02 |
| melwitt | https://opendev.org/openstack/python-novaclient/commit/03dca4bc823c82054869dfaf6925d5e1e068ac51 | 17:02 |
| stephenfin | https://review.opendev.org/#/c/708850/ | 17:02 |
| stephenfin | yeah | 17:02 |
| *** ociuhandu has joined #openstack-nova | 17:02 | |
| sean-k-mooney | so melwitt you were suggesting adding an option to include it again | 17:04 |
| sean-k-mooney | if we did i would proably do something like --user-data=<format> | 17:04 |
| *** priteau has joined #openstack-nova | 17:05 | |
| melwitt | yeah, some way to opt-in to it for end user verification purposes | 17:05 |
| sean-k-mooney | possibley with --user-data=None used to not print it and restore the previous behavior of printing by default | 17:05 |
| stephenfin | melwitt: I won't block the '--include-user-data' option, but I'd much rather we had a more generic '--verbose' or '--json' option that just showed the plain old response from the API or a new command to retrieve that field | 17:06 |
| sean-k-mooney | if we want to maintain backward compatablity | 17:06 |
| melwitt | tbc, I think not printing by default makes sense, I just think it would be ideal to have a way to opt-in if you want to verify something | 17:06 |
| stephenfin | Yup, I get that :) | 17:07 |
| * stephenfin checks was OSC does | 17:07 | |
| melwitt | stephenfin: oh, I see what you mean now | 17:07 |
| stephenfin | *what | 17:07 |
| sean-k-mooney | so --user-data=raw|json|plain | 17:07 |
| sean-k-mooney | where raw would give you hte base64 encoded string | 17:07 |
| *** ociuhandu has quit IRC | 17:08 | |
| stephenfin | sean-k-mooney: no, 'nova boot --json ...' | 17:08 |
| stephenfin | where the output is the raw JSON blob returned by the API rather than the pretty tables we do by default | 17:08 |
| sean-k-mooney | wait is that a thing | 17:08 |
| stephenfin | it is for OSC but not for novaclient | 17:08 |
| stephenfin | it's also a lot more work... | 17:08 |
| sean-k-mooney | no for osc we dont have --json we have -f json | 17:08 |
| sean-k-mooney | unless --json has been added recently and is an alias | 17:09 |
| stephenfin | yeah, sorry, I mean we have JSON-style formatting | 17:09 |
| sean-k-mooney | i use the format specifers for osc the whole time | 17:09 |
| stephenfin | different arg, as you say | 17:09 |
| sean-k-mooney | ok well that normally applies to all the output | 17:09 |
| *** ociuhandu has joined #openstack-nova | 17:09 | |
| stephenfin | yeah | 17:09 |
| sean-k-mooney | i was suggesting having a way to contol the output/encoding of just that filed | 17:10 |
| stephenfin | it's arbitrary data - there's nothing to really encoding | 17:10 |
| stephenfin | maybe escape, but not encode | 17:10 |
| sean-k-mooney | well its typically in cloud-init or could-config format or ignition format | 17:11 |
| stephenfin | yeah, 'zactly | 17:11 |
| sean-k-mooney | all of which are json or yaml based and human readable | 17:11 |
| stephenfin | Anyway, I'm suggesting that including non-human readable data in a response designed for humans, like the 'nova show' output is supposed to be, _probably_ isn't the right thing to do and it would be nicer if we had a specific machine-readable format for that | 17:12 |
| sean-k-mooney | stephenfin: do we have the --long optin in nova client | 17:12 |
| sean-k-mooney | we could just use that | 17:12 |
| sean-k-mooney | is it --long or --wide | 17:12 |
| sean-k-mooney | there is one arg you can pass that shows addtional columns | 17:12 |
| stephenfin | you have --width | 17:12 |
| stephenfin | but I don't see anything else | 17:13 |
| *** ociuhandu has quit IRC | 17:14 | |
| stephenfin | melwitt: Can I check out how tough a format argument would be to implement tomorrow morning and come back to you? If it's too much, I'd be okay with a '--verbose' or specific '--include-user-data' argument | 17:15 |
| melwitt | stephenfin: yeah, it's not urgent. just wanted to see what you thought, no worry | 17:16 |
| melwitt | whatever it is, I hope we can backport it to ussuri but not sure whether that's gonna fly | 17:16 |
| sean-k-mooney | stephenfin: i was thinking of openstack hypervisor list --long | 17:16 |
| sean-k-mooney | but that is specific to just that command | 17:17 |
| *** ttsiouts has quit IRC | 17:17 | |
| stephenfin | Yeah, it's weird having release stable branches for a client | 17:17 |
| melwitt | yeah, it's a pain | 17:17 |
| stephenfin | but it is what it is | 17:17 |
| *** derekh has quit IRC | 17:17 | |
| stephenfin | would any opt be backportable? | 17:18 |
| stephenfin | lyarwood: hit https://review.opendev.org/#/c/718100/ btw | 17:18 |
| sean-k-mooney | stephenfin: well you could arge your change was a regression since you broke backwards comatiablity without any way to opt in | 17:18 |
| lyarwood | stephenfin: thanks | 17:18 |
| stephenfin | sean-k-mooney: but we indicated that with a major version bump | 17:19 |
| stephenfin | so per Sem-Ver, we're golden | 17:19 |
| stephenfin | though per the "be nice to users" rule, maybe not so much :) | 17:19 |
| sean-k-mooney | true but its not nessisarly resonable to expect them to be able to pin the version to an older one since nova clint is ill advisidly used as part of sdks and other tools | 17:19 |
| sean-k-mooney | anyway whats done and relased is done | 17:20 |
| sean-k-mooney | stephenfin: this converation reminds me why i avoid nova clinet in the first place | 17:21 |
| stephenfin | yeah, OSC FTW | 17:21 |
| sean-k-mooney | i only ever use it if osc cant do the thing | 17:21 |
| *** priteau has quit IRC | 17:26 | |
| *** sapd1 has quit IRC | 17:30 | |
| openstackgerrit | Sylvain Bauza proposed openstack/nova master: FUP: doc: explaining that resize works for vGPUs https://review.opendev.org/720250 | 17:34 |
| *** tesseract has quit IRC | 17:34 | |
| *** ttsiouts has joined #openstack-nova | 17:34 | |
| *** dpawlik has quit IRC | 17:48 | |
| *** ociuhandu has joined #openstack-nova | 17:52 | |
| *** nightmare_unreal has quit IRC | 17:52 | |
| *** ralonsoh has quit IRC | 17:55 | |
| *** links has quit IRC | 18:01 | |
| *** bbowen has quit IRC | 18:03 | |
| *** ttsiouts has quit IRC | 18:09 | |
| *** bbowen has joined #openstack-nova | 18:12 | |
| *** ociuhandu has quit IRC | 18:15 | |
| *** ociuhandu has joined #openstack-nova | 18:22 | |
| *** ociuhandu has quit IRC | 18:38 | |
| *** ttsiouts has joined #openstack-nova | 18:42 | |
| *** ttsiouts has quit IRC | 18:48 | |
| *** hongbin has joined #openstack-nova | 19:01 | |
| *** factor has quit IRC | 19:14 | |
| *** factor has joined #openstack-nova | 19:14 | |
| *** ttsiouts has joined #openstack-nova | 19:23 | |
| *** ociuhandu has joined #openstack-nova | 19:23 | |
| *** ociuhandu has quit IRC | 19:29 | |
| *** ttsiouts has quit IRC | 19:34 | |
| *** ttsiouts has joined #openstack-nova | 19:34 | |
| *** factor has quit IRC | 19:36 | |
| *** factor has joined #openstack-nova | 19:43 | |
| *** ttsiouts has quit IRC | 19:49 | |
| *** vishalmanchanda has quit IRC | 19:54 | |
| *** ociuhandu has joined #openstack-nova | 20:04 | |
| *** slaweq has quit IRC | 20:08 | |
| *** slaweq has joined #openstack-nova | 20:08 | |
| *** ociuhandu has quit IRC | 20:14 | |
| *** ttsiouts has joined #openstack-nova | 20:23 | |
| *** ttsiouts has quit IRC | 20:28 | |
| *** slaweq has quit IRC | 20:28 | |
| *** prometheanfire has joined #openstack-nova | 20:35 | |
| *** damien_r has quit IRC | 20:41 | |
| *** ttsiouts has joined #openstack-nova | 20:58 | |
| *** ociuhandu has joined #openstack-nova | 21:02 | |
| *** ociuhandu has quit IRC | 21:09 | |
| *** nweinber has quit IRC | 21:15 | |
| openstackgerrit | Ivaylo Mitev proposed openstack/nova master: Faults from cell DB missing in GET /servers/detail https://review.opendev.org/699176 | 21:26 |
| openstackgerrit | Merged openstack/nova master: fup: Fix [workarounds]/rbd_volume_local_attach config docs https://review.opendev.org/718100 | 21:31 |
| *** ttsiouts has quit IRC | 21:32 | |
| *** ociuhandu has joined #openstack-nova | 21:57 | |
| *** mriedem has left #openstack-nova | 21:59 | |
| *** ociuhandu has quit IRC | 22:07 | |
| *** ociuhandu has joined #openstack-nova | 22:08 | |
| *** ccamacho has quit IRC | 22:11 | |
| *** eharney has quit IRC | 22:11 | |
| *** ociuhandu has quit IRC | 22:13 | |
| *** sangeet has joined #openstack-nova | 22:16 | |
| *** prometheanfire has quit IRC | 22:17 | |
| sangeet | What is the best way to power off the servers. "Openstack server stop" is not a graceful shutdown. | 22:19 |
| sangeet | I am using Stein | 22:19 |
| *** ttsiouts has joined #openstack-nova | 22:38 | |
| *** prometheanfire has joined #openstack-nova | 22:39 | |
| sangeet | Want to shut down the VM gracefully so when it is stared again, we are back to full working mode. | 22:43 |
| *** rcernin has joined #openstack-nova | 22:45 | |
| *** hongbin has quit IRC | 22:53 | |
| melwitt | sangeet: graceful shutdown is default behavior but there is a timeout used for how long to wait for the guest to shutdown gracefully, defaulting to 60s https://docs.openstack.org/nova/stein/configuration/config.html#DEFAULT.shutdown_timeout | 22:56 |
| *** tkajinam has joined #openstack-nova | 22:59 | |
| kplant | i think acpid is required in the guest too | 23:00 |
| kplant | iirc | 23:00 |
| *** hongbin has joined #openstack-nova | 23:01 | |
| *** ttsiouts has quit IRC | 23:12 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Add docs and releasenotes for BP policy-defaults-refresh https://review.opendev.org/720129 | 23:14 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Fix follow up comments on policy work https://review.opendev.org/717835 | 23:16 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Fix follow up comments on policy work https://review.opendev.org/717835 | 23:22 |
| *** sangeet has quit IRC | 23:33 | |
| *** ociuhandu has joined #openstack-nova | 23:37 | |
| *** tosky has quit IRC | 23:39 | |
| *** ttsiouts has joined #openstack-nova | 23:40 | |
| *** ttsiouts has quit IRC | 23:45 | |
| *** mlavalle has quit IRC | 23:47 | |
| *** ociuhandu has quit IRC | 23:49 | |
| *** ttsiouts has joined #openstack-nova | 23:54 | |
| *** ttsiouts has quit IRC | 23:58 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!