| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2020-04-15 | |||
| 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 | |
| 10:52:47 | brinzhang_ | bauzas: greate, I will check and push a spec for this | |
| 10:53:24 | bauzas | brinzhang_: feel free to close it then and open a blueprint | |
| 10:54:07 | brinzhang_ | bauzas: yeah, we can close it until I open the bp | |
| 10:56:00 | openstack | Launchpad bug 1868033 in OpenStack Compute (nova) "Booting instance with pci_device fails during rocky->stein live upgrade" [High,New] | |
| 10:56:00 | bauzas | gibi: stephenfin: huh, that's a nasty bug https://bugs.launchpad.net/nova/+bug/1868033 | |
| 10:57:43 | stephenfin | ick | |
| 10:58:14 | stephenfin | That's a pretty big issue. I can work on fixing that today. | |
| 10:58:33 | stephenfin | Guess we didn't spot that since we don't do PCI testing in the gate :( | |
| 10:58:38 | gibi | stephenfin: I started looking into that but got distracted (virtual PTG planning) so you can go ahead | |
| 10:58:41 | bauzas | TIL https://www.merriam-webster.com/dictionary/ick | |
| 10:59:18 | bauzas | stephenfin: I'll just throw it to you | |
| 11:00:13 | bauzas | stephenfin: that'd require some PCI grenade testing | |
| 11:00:21 | openstackgerrit | Merged openstack/nova master: fup: Combine SUPPORTED_DEVICE_BUS and SUPPORTED_STORAGE_BUSES https://review.opendev.org/718448 | |
| 11:00:38 | bauzas | wow, this one is an old +Wd ^ | |
| 11:01:04 | lyarwood | finally :d | |
| 11:01:27 | gibi | lyarwood: two to go :) | |
| 11:01:55 | bauzas | lyarwood: I honestly stopped looking at the gate for your series once we merged 2.87 :p | |
| 11:02:06 | bauzas | I spent the whole Friday rechecking | |
| 11:02:13 | 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:50 | bauzas | brinzhang_: ack, feel free to close the bug then with a link pointing to the bp | |
| 11:03:58 | brinzhang_ | bauzas: yeah, I think that we cannot hit new microversion, that just should add the event base on mivroversion 2.82 | |
| 11:04:17 | bauzas | I'm hungry, I can't think | |
| 11:04:55 | brinzhang_ | bauzas: ah, let me check in the future ^^ | |
| 11:27:59 | 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:59 | brinzhang_ | bauzas: you set bug 1872934 to wishlist? IMO, it just a bug, that canbe done with bug fix, right? | |
| 11:49:23 | hrw | https://review.opendev.org/#/c/715979/ - anyone with free few minutes to get aarch64 feature matrix updated? needs +2+w | |
| 11:55:39 | 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:39 | bauzas | brinzhang_: I did set back to Low https://bugs.launchpad.net/nova/+bug/1872934 | |
| 11:55:48 | bauzas | brinzhang_: you can modify your own bugs FWIW | |