| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-04-15 | |||
| 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 | bauzas | lyarwood: thanks, I'm done with this bug | |
| 08:25:58 | lyarwood | well, between glance and cinder at least | |
| 08:26:14 | bauzas | we're talking about upstream bug triage | |
| 08:26:23 | bauzas | I just want the cinder folks to look at it | |
| 08:26:39 | lyarwood | tbh there might be a way of forcing the data to actually end up in glance | |
| 08:26:46 | lyarwood | ack yeah | |
| 08:26:55 | lyarwood | I'll take a look quickly but I'll leave it to them if I can't find anything | |
| 08:34:09 | johnthetubaguy | lyarwood: do cinder do volume snapshot sharing between tenants? | |
| 08:34:33 | johnthetubaguy | (or get glance to copy it, I guess) | |
| 08:36:43 | bauzas | johnthetubaguy: I feel this bug will potentially require a cross-project effort in order to get fixed | |
| 08:36:52 | bauzas | (but there could be duplicates) | |
| 08:36:55 | johnthetubaguy | bauzas: sounds like it | |
| 08:37:19 | johnthetubaguy | I mean, I actually don't think nova should do anything here, but I would say that :P | |
| 08:37:31 | lyarwood | johnthetubaguy: yeah I was thinking you could force the upload into glance to workaround | |
| 08:38:02 | johnthetubaguy | lyarwood: I was thinking glance forces an upload into cinder for the additional project, but either way I guess | |
| 08:38:57 | lyarwood | that could also work, at least c-api/c-vol can offload things to the backends that way | |
| 08:41:29 | bauzas | johnthetubaguy: feel free to add nova as a related project to the bug then ;) | |
| 08:41:47 | bauzas | johnthetubaguy: I only marked cinder, but obviously glance and maybe nova could be added | |
| 08:58:54 | 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:54 | bauzas | gibi: I've seen you started to look at https://bugs.launchpad.net/nova/+bug/1871583 | |
| 08:59:44 | bauzas | gibi: I'm confused with the exception | |
| 09:01:03 | bauzas | gibi: so, libvirt doesn't support resizing to the same host | |
| 09:01:56 | bauzas | hence the exception | |
| 09:02:12 | bauzas | anyway, nevermind | |
| 09:02:18 | bauzas | I think what to triage | |
| 09:02:22 | bauzas | what/how | |
| 09:06:48 | slaweq | gibi: hi | |
| 09:07: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:29 | 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:50 | slaweq | gibi: can You maybe take a look if that is some new issue or still the same? | |
| 09:07:54 | slaweq | thx in advance | |
| 09:14:48 | hrw | https://review.opendev.org/#/c/712607/ has two +2, has +W, has +1 from zuul. needs recheck to go to gate? | |
| 09:16:55 | bauzas | hrw: looking | |
| 09:17:13 | hrw | bauzas: thx | |
| 09:18:01 | 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:21:50 | bauzas | hrw: okay, so two things | |
| 09:22:02 | bauzas | hrw: yes you need to recheck 712607 | |
| 09:22:31 | bauzas | hrw: and if you rebase 714439 on top of ^, then 712607 will also be checked | |
| 09:23:01 | bauzas | hrw: do you know http://zuul.openstack.org/status ? | |
| 09:23:17 | hrw | bauzas: yes. I even wrote userscript for it | |
| 09:23:33 | hrw | just not familar with nova jobs | |
| 09:23:40 | bauzas | all cool then, you can see what goes into th check pipeline | |
| 09:23:44 | bauzas | the* | |
| 09:24:06 | bauzas | hrw: nothing really here is related to nova jobs, it's just zuul ordering | |
| 09:24:23 | hrw | bauzas: can you look at https://review.opendev.org/#/c/714425/ too? simple 'check for SEV only on x86' patch | |
| 09:24:23 | bauzas | (and gerrit) | |
| 09:24:50 | bauzas | hrw: sure, i can this week | |
| 09:24:56 | hrw | thanks | |
| 09:25:55 | hrw | I have 22 open patches now. feels far too many ;D | |
| 09:45:19 | hrw | bauzas: thanks for SEV review. will look at improving | |
| 09:46:27 | bauzas | np, ping me once you're done and i'll be happy to look at it again | |
| 09:48:46 | hrw | left comment | |
| 09:57:39 | 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:39 | bauzas | can someone remind me what's the current status with service tokens ? https://bugs.launchpad.net/nova/+bug/1870558 | |
| 10:03:26 | bauzas | nevermind | |
| 10:05:05 | hrw | bauzas: test_host checks what host can, test_driver checks what guest can, right? | |
| 10:05:14 | stephenfin | gmann: Few questions on https://review.opendev.org/#/c/662968/ | |
| 10:05:39 | bauzas | hrw: kind of | |
| 10:05:53 | bauzas | hrw: test_driver just tests the driver.py module | |
| 10:06:15 | bauzas | which is basically the libvirt wrapper | |
| 10:06:56 | bauzas | hrw: and test_host.py does verify libvirt.host.py module which is more the interface for https://libvirt.org/drvnodedev.html | |
| 10:07:09 | hrw | ok | |
| 10:07:44 | bauzas | and https://libvirt.org/format.html | |
| 10:07:57 | hrw | so in test_host I should check 'if host arch != x86 then no sev' like | |
| 10:08:57 | bauzas | yup, see the example I gave in the comment | |
| 10:10:01 | hrw | I have that on gvim now. right with test_driver (where I already played with tests) | |
| 10:17:14 | hrw | bauzas: is https://paste.centos.org/view/e56be492 looking good? | |
| 10:17:58 | hrw | no, it is rather nto | |
| 10:20:10 | hrw | should call _set_amd_sev_support() | |
| 10:28:54 | openstackgerrit | Merged openstack/nova master: Add new default roles in flavor extra specs policies https://review.opendev.org/719381 | |
| 10:31:50 | openstackgerrit | Marcin Juszkiewicz proposed openstack/nova master: libvirt: check for AMD SEV only on x86-64 https://review.opendev.org/714425 | |
| 10:32:10 | hrw | bauzas: good to have so many tests to take ideas from ;d | |
| 10:40:17 | bauzas | brinzhang_: ^ | |
| 10:40:17 | 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:41:15 | brinzhang_ | bauzas: hi, I post this in IRC, you can see above | |
| 10:42:05 | 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:06 | brinzhang_ | bauzas: thanks check this bug ^^ | |
| 10:42:52 | brinzhang_ | bauzas: got it, will add this tag if there is the same issue :) | |
| 10:42:57 | 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:44:18 | brinzhang_ | bauzas: yeah, good idea, I will take this way, thanks. | |
| 10:44:35 | bauzas | gosh, I was missing LP timeout issues... | |
| 10:45:19 | brinzhang_ | ... | |
| 10:45:19 | brinzhang_ | bauzas: but sorry Cyborg dont use launchpad (use storyboard), this makes me disheartenment | |
| 10:45:37 | bauzas | brinzhang_: you lucky folks... | |
| 10:47:36 | openstack | Launchpad bug 1865549 in OpenStack Compute (nova) " Define Cyborg ARQ binding notification event." [Undecided,New] | |
| 10:47:36 | bauzas | brinzhang_: yet another cyborg-related bug FYK https://bugs.launchpad.net/nova/+bug/1865549 | |
| 10:48:10 | openstackgerrit | Merged openstack/nova master: Pass the actual target in flavor extra specs policy https://review.opendev.org/719603 | |
| 10:48:20 | openstackgerrit | Merged openstack/nova stable/train: Use placement stable version for functional job https://review.opendev.org/719120 | |
| 10:48:33 | brinzhang_ | bauzas: ack, I will check this tomorrow | |
| 10:49:16 | bauzas | just sayin' | |
| 10:49:56 | bauzas | aaaand cyborg got promoted as an 'official' LP tag :) | |
| 10:50:02 | brinzhang_ | this bug looks like a BP, right? | |
| 10:50:37 | bauzas | brinzhang_: which one ? | |
| 10:50:53 | openstack | bug 1865549 in OpenStack Compute (nova) " Define Cyborg ARQ binding notification event." [Low,Confirmed] https://launchpad.net/bugs/1865549 | |
| 10:50:53 | brinzhang_ | bug 1865549 | |
| 10:51:21 | brinzhang_ | looks like we should add an event for cyborg notification | |
| 10:51:53 | bauzas | yeah, agreed, just marked it as Wishlist | |
| 10:51:54 | brinzhang_ | I was not looked into, I have a meeting now | |