Wednesday, 2020-07-29

*** rcernin has quit IRC00:05
*** brinzhang_ has joined #openstack-nova00:14
*** k_mouza has joined #openstack-nova00:15
*** brinzhang has quit IRC00:17
*** k_mouza has quit IRC00:20
*** nweinber has quit IRC00:22
*** rcernin has joined #openstack-nova00:31
*** zzzeek has quit IRC00:57
*** dtantsur|afk has quit IRC00:57
*** owalsh_ has joined #openstack-nova01:00
*** zzzeek has joined #openstack-nova01:01
*** dtantsur|afk has joined #openstack-nova01:01
*** alex_xu has quit IRC01:02
*** gyee has quit IRC01:02
*** owalsh has quit IRC01:04
*** tkajinam has quit IRC01:06
*** tkajinam has joined #openstack-nova01:06
*** ociuhandu has joined #openstack-nova01:17
*** ociuhandu has quit IRC01:21
*** k_mouza has joined #openstack-nova01:31
*** tkajinam has quit IRC01:33
*** tkajinam has joined #openstack-nova01:34
*** k_mouza has quit IRC01:36
*** ociuhandu has joined #openstack-nova01:43
*** ociuhandu has quit IRC01:48
openstackgerritTony Su proposed openstack/nova master: Provider Config File: YAML file loading and schema validation  https://review.opendev.org/67334101:56
*** yaawang has quit IRC02:21
*** yaawang has joined #openstack-nova02:22
*** artom has quit IRC02:35
*** k_mouza has joined #openstack-nova02:45
*** k_mouza has quit IRC02:50
*** ociuhandu has joined #openstack-nova02:53
*** ociuhandu has quit IRC02:57
*** sapd1 has joined #openstack-nova03:14
*** k_mouza has joined #openstack-nova03:27
*** k_mouza has quit IRC03:31
*** psachin has joined #openstack-nova03:38
*** mkrai has joined #openstack-nova03:44
*** evrardjp has quit IRC04:33
*** evrardjp has joined #openstack-nova04:33
*** ratailor has joined #openstack-nova04:34
*** udesale has joined #openstack-nova04:45
*** sapd1 has quit IRC04:45
*** xiaolin has quit IRC04:58
*** tetsuro has joined #openstack-nova05:25
*** tetsuro has quit IRC05:25
*** tetsuro has joined #openstack-nova05:26
*** tetsuro has quit IRC05:27
*** links has joined #openstack-nova05:46
*** slaweq has joined #openstack-nova05:52
*** yaawang has quit IRC05:52
*** yaawang has joined #openstack-nova05:53
*** ociuhandu has joined #openstack-nova05:59
*** slaweq has quit IRC06:02
*** mkrai has quit IRC06:10
*** mkrai has joined #openstack-nova06:15
*** brtknr has quit IRC06:17
*** vishalmanchanda has joined #openstack-nova06:40
*** mkrai has quit IRC06:40
*** whoami-rajat has quit IRC06:41
*** whoami-rajat has joined #openstack-nova06:41
*** yaawang has quit IRC06:42
*** dklyle has quit IRC06:42
*** yaawang has joined #openstack-nova06:42
*** mkrai has joined #openstack-nova06:55
*** slaweq has joined #openstack-nova06:57
*** brinzhang0 has joined #openstack-nova07:02
*** brinzhang_ has quit IRC07:05
*** tesseract has joined #openstack-nova07:11
*** mkrai has quit IRC07:15
*** bhagyashris is now known as bhagyashris|lunc07:22
*** brinzhang_ has joined #openstack-nova07:22
*** xek_ has joined #openstack-nova07:24
*** brinzhang0 has quit IRC07:25
*** mkrai has joined #openstack-nova07:26
*** yonglihe has joined #openstack-nova07:31
*** tosky has joined #openstack-nova07:39
*** ralonsoh has joined #openstack-nova07:40
*** brinzhang0 has joined #openstack-nova07:45
*** brinzhang_ has quit IRC07:48
*** dtantsur|afk is now known as dtantsur07:53
*** mkrai has quit IRC07:55
tony_sustephenfin: really appreciate your detailed review and I uploaded the latest patch reflecting the latest comments.08:01
*** evrardjp has quit IRC08:06
*** evrardjp has joined #openstack-nova08:08
*** mkrai has joined #openstack-nova08:12
*** maciejjozefczyk has joined #openstack-nova08:14
*** sapd1 has joined #openstack-nova08:22
*** derekh has joined #openstack-nova08:23
*** bhagyashris|lunc is now known as bhagyashris08:40
*** jsuchome has joined #openstack-nova08:43
*** brtknr has joined #openstack-nova08:46
*** ociuhandu has quit IRC08:56
*** dtantsur is now known as dtantsur|brb08:58
*** xiaolin has joined #openstack-nova09:08
*** martinkennelly has joined #openstack-nova09:10
*** jangutter_ has joined #openstack-nova09:11
*** rcernin has quit IRC09:12
*** jangutter has quit IRC09:14
lyarwoodstephenfin: https://review.opendev.org/#/q/topic:bug/1889108 updated for your reviewing pleasure btw09:15
stephenfinta09:15
stephenfinmelwitt, dansmith: Haven't investigated yet, but nova-ceph-multistore is failing again on some jobs. Could be nothing but just FYI https://review.opendev.org/#/c/673341/5409:21
* lyarwood clicks09:24
lyarwoodhttps://zuul.opendev.org/t/openstack/build/0709ecfb37c54cb8813afb42aefe9097/log/controller/logs/screen-g-api.txt#1228609:25
lyarwoodrbd.OSError: [errno 116] error removing image09:25
lyarwood[errno 2] error removing snapshot b'snap' from b'39db07f4-d56b-47f2-b7c2-81d3d4e8707e' Snapshot does not exist.09:26
lyarwoodthat's a better error09:26
*** k_mouza has joined #openstack-nova09:29
stephenfinlyarwood: done; some comments on the last one but other straight +2s09:32
stephenfinswot09:32
*** ociuhandu has joined #openstack-nova09:33
lyarwoodhttps://github.com/openstack/glance_store/blob/100fe260bfb6698795ae5d6fce6aeadecbed1772/glance_store/_drivers/rbd.py#L385-L407 - odd, that should create a snapshot09:34
lyarwoodstephenfin: ta09:34
lyarwoodyeah I'll address that now09:36
*** ociuhandu has quit IRC09:37
*** ociuhandu has joined #openstack-nova09:42
*** sapd1 has quit IRC09:57
*** tkajinam has quit IRC10:01
openstackgerritLee Yarwood proposed openstack/nova master: func: Add live migration rollback volume attachment tests  https://review.opendev.org/74353410:02
openstackgerritLee Yarwood proposed openstack/nova master: Add regression tests for bug #1889108  https://review.opendev.org/74328910:02
openstackbug 1889108 in OpenStack Compute (nova) "failures during driver.pre_live_migration remove source attachments during rollback" [High,In progress] https://launchpad.net/bugs/1889108 - Assigned to Lee Yarwood (lyarwood)10:02
openstackgerritLee Yarwood proposed openstack/nova master: compute: Don't delete the original attachment during pre LM rollback  https://review.opendev.org/74331910:02
openstackgerritLee Yarwood proposed openstack/nova master: func: Add CinderFixture to _IntegratedTestBase  https://review.opendev.org/74353510:02
openstackgerritLee Yarwood proposed openstack/nova master: func: Introduce a server_final_status kwarg to _IntegratedTestBase.live_migrate  https://review.opendev.org/74371010:02
openstackgerritLee Yarwood proposed openstack/nova master: func: Introduce a server_final_status kwarg to InstanceHelperMixin._live_migrate  https://review.opendev.org/74371010:03
lyarwoodstephenfin: ^ thanks again10:03
*** yaawang has quit IRC10:06
openstackgerritLee Yarwood proposed openstack/nova master: func: Introduce a server_final_status kwarg to InstanceHelperMixin._live_migrate  https://review.opendev.org/74371010:06
* lyarwood -> coffee10:06
*** yaawang has joined #openstack-nova10:07
sean-k-mooneyanyone know if you can rebuild or resize vms in a shelved state?10:34
sean-k-mooneyif the answer was yes for both that would be really helpful.10:35
openstackgerritLee Yarwood proposed openstack/nova master: block_device: Use initialize APIs to refresh when reported as idempotent  https://review.opendev.org/72076910:49
*** mgariepy has quit IRC11:09
sean-k-mooneythat would be a no :(11:11
*** dtantsur|brb is now known as dtantsur11:26
*** xek_ has quit IRC11:27
*** markvoelker has joined #openstack-nova11:40
*** raildo has joined #openstack-nova11:44
*** markvoelker has quit IRC11:47
openstackgerritStephen Finucane proposed openstack/nova master: objects: Don't use generic 'Field' container  https://review.opendev.org/73823911:57
openstackgerritStephen Finucane proposed openstack/nova master: objects: Remove unnecessary type aliases, exceptions  https://review.opendev.org/73824011:57
openstackgerritStephen Finucane proposed openstack/nova master: objects: Use imports instead of type aliases  https://review.opendev.org/73801811:58
openstackgerritStephen Finucane proposed openstack/nova master: objects: Remove wrappers around ovo mixins  https://review.opendev.org/73801911:58
openstackgerritStephen Finucane proposed openstack/nova master: WIP: objects: Add type hints  https://review.opendev.org/73802011:58
*** artom has joined #openstack-nova12:03
openstackgerritLee Yarwood proposed openstack/nova master: func: Introduce a server_expected_state kwarg to InstanceHelperMixin._live_migrate  https://review.opendev.org/74371012:16
*** udesale_ has joined #openstack-nova12:19
lyarwoodDoes anyone have any ideas why this https://review.opendev.org/#/c/731506/ never ends up in the gate?12:20
lyarwoodI can't rebase it, it isn't in conflict etc12:20
*** udesale has quit IRC12:21
*** mgariepy has joined #openstack-nova12:23
stephenfinlyarwood: possibly a bug in Gerrit? There is no Depends-On but it was based on a patch that has since been abandoned12:25
openstackgerritStephen Finucane proposed openstack/nova master: compute: As cellsv1 is no more fold _detach_volume back into detach_volume  https://review.opendev.org/73150612:25
stephenfinthat should do the trick12:25
stephenfinlyarwood: Can you look at https://review.opendev.org/#/c/731506/ today?12:26
stephenfinAlso bauzas, perhaps? https://review.opendev.org/#/c/731506/12:26
* stephenfin would ask gibi but he's not around of course12:26
*** xek has joined #openstack-nova12:27
lyarwoodlooking12:27
lyarwoodstephenfin: wait, that's the link to my change12:29
lyarwoodstephenfin: which review did you want me to look at12:29
stephenfindoh12:29
stephenfinhttps://review.opendev.org/#/c/743568/12:29
lyarwoodstephenfin: and re https://review.opendev.org/731506 yeah I missed that it was still pointing at that change sorry12:29
stephenfinall good12:30
lyarwoodstephenfin: done, LGTM, I guess func tests wouldn't make sense here given libvirt is acting up right?12:36
stephenfinlyarwood: yes, that's my thinking too12:36
*** bbowen has quit IRC12:38
*** bbowen has joined #openstack-nova12:39
*** lpetrut has joined #openstack-nova12:44
*** markvoelker has joined #openstack-nova12:48
ralonsohstephenfin, hi! I you have some time: https://review.opendev.org/#/c/739961/12:49
*** ratailor has quit IRC12:50
stephenfinralonsoh: Could we get some context in the commit message?12:52
stephenfini.e. what it's for and why it's needed12:52
ralonsohstephenfin, ok, I'll add this12:53
stephenfinthanks :)12:53
*** nweinber has joined #openstack-nova12:57
openstackgerritRodolfo Alonso Hernandez proposed openstack/os-traits master: Add NUMA affinity policy traits  https://review.opendev.org/73996113:00
aarentsHi nova,13:14
aarentslyarwood : When you will have time for this https://review.opendev.org/#/c/696084/4 , I split the change with another: https://review.opendev.org/#/c/743537/ and added a tempest test to reproduce the issue: https://review.opendev.org/#/c/743708/1 (failed as expected in tempest-slow-py3 jobs)13:14
openstackgerritTakashi Natsume proposed openstack/python-novaclient stable/ussuri: Add a cleanup for a server in a functional test  https://review.opendev.org/74375613:17
openstackgerritTakashi Natsume proposed openstack/python-novaclient stable/train: Add a cleanup for a server in a functional test  https://review.opendev.org/74375713:19
*** dave-mccowan has joined #openstack-nova13:29
*** xek has quit IRC13:30
openstackgerritLee Yarwood proposed openstack/nova master: Use absolute path during qemu img rebase  https://review.opendev.org/73924613:42
lyarwood^ from gibi LGTM now if any other cores have time today13:43
*** jangutter_ is now known as jangutter13:53
*** jangutter_ has joined #openstack-nova13:54
*** jangutter has quit IRC13:57
*** yaawang has quit IRC14:09
*** xek has joined #openstack-nova14:09
*** yaawang has joined #openstack-nova14:10
*** mriedem has joined #openstack-nova14:11
*** brinzhang_ has joined #openstack-nova14:17
*** brinzhang0 has quit IRC14:20
*** dklyle has joined #openstack-nova14:29
*** links has quit IRC14:36
*** mlavalle has joined #openstack-nova14:40
*** rcernin has joined #openstack-nova14:41
*** lpetrut has quit IRC14:43
*** spatel has joined #openstack-nova14:45
*** jangutter_ is now known as jangutter14:48
*** lbragstad_ has joined #openstack-nova14:51
*** rcernin_ has joined #openstack-nova14:51
openstackgerritMerged openstack/nova master: Use compression by default for 'SshDriver'  https://review.opendev.org/68439314:52
*** dklyle has quit IRC14:53
*** rcernin_ has quit IRC14:56
*** rcernin has quit IRC14:58
*** irclogbot_3 has quit IRC14:58
*** zer0c00l has quit IRC14:58
*** lbragstad has quit IRC14:58
*** irclogbot_1 has joined #openstack-nova14:59
*** dklyle has joined #openstack-nova15:01
bauzasstephenfin: sorry, I was on PTO this morning15:02
*** brinzhang0 has joined #openstack-nova15:04
*** links has joined #openstack-nova15:06
*** brinzhang_ has quit IRC15:06
*** links has quit IRC15:07
*** brinzhang_ has joined #openstack-nova15:08
*** hoonetorg has quit IRC15:09
*** brinzhang_ has quit IRC15:09
*** brinzhang_ has joined #openstack-nova15:10
*** brinzhang0 has quit IRC15:12
*** sapd1 has joined #openstack-nova15:13
stephenfinbauzas: np. Any chance you could look at https://review.opendev.org/743568 today?15:15
bauzasack15:16
stephenfinthanks :)15:16
*** hoonetorg has joined #openstack-nova15:21
*** nweinber has quit IRC15:24
*** spatel has quit IRC15:28
*** mgariepy has quit IRC15:40
*** mkrai has quit IRC15:43
*** mkrai_ has joined #openstack-nova15:43
*** nweinber has joined #openstack-nova15:51
*** xek has quit IRC16:05
openstackgerritLee Yarwood proposed openstack/nova master: compute: Validate a BDMs disk_bus when provided  https://review.opendev.org/72484216:05
lyarwoodstephenfin: ^ can you +W that again when you get a chance?16:06
lyarwoodhttps://review.opendev.org/#/c/729596/ and this please16:06
*** xiaolin has quit IRC16:06
lyarwoodmelwitt: https://review.opendev.org/#/q/topic:bug/1889108 - would you mind taking a look at this series?16:07
*** k_mouza has quit IRC16:09
*** brinzhang0 has joined #openstack-nova16:10
*** brinzhang_ has quit IRC16:13
*** markvoelker has quit IRC16:13
openstackgerritSylvain Bauza proposed openstack/nova-specs master: Routed network scheduling spec  https://review.opendev.org/73370316:16
bauzasstephenfin: you may be interested in ^16:16
stephenfinbauzas: Opened in a tab. I'll review that first thing in the morning.16:16
bauzas++16:16
bauzasstephenfin: I guess we couldn't catch https://bugs.launchpad.net/nova/+bug/1889257 on a functional test ?16:17
openstackLaunchpad bug 1889257 in OpenStack Compute (nova) "Live migration of realtime instances is broken" [Medium,In progress] - Assigned to Stephen Finucane (stephenfinucane)16:17
bauzasstephenfin: I need to jump straight off the keyboard but I can review decently https://review.opendev.org/#/c/743568/ later tonight16:18
bauzasto be fair, the bug isn't trivial to understand and there are implications on the way we generate the XML16:18
bauzasfortunately, the XML implications are limited to the attribute that is bugged16:19
bauzasstephenfin: but I guess non-realtime users would be impacted by this change too, right?16:20
*** ociuhandu_ has joined #openstack-nova16:21
lyarwoodbauzas: I asked about func tests earlier but we'd have to mock out the way libvirt is rewriting the xml etc, we could but I don't think it's required to land this tbh.16:23
lyarwoodbauzas: and AFAIK it only impacts realtime LM16:23
*** ociuhandu has quit IRC16:23
*** udesale_ has quit IRC16:24
*** tesseract has quit IRC16:25
lyarwoodhuh we don't have a regression keyword in launchpad for nova, that might be useful.16:26
bauzaswe have but this is release-specific16:26
bauzaslyarwood: so, I need to be absolutely sure that vcpusched is only used by the realtime users16:27
*** ociuhandu_ has quit IRC16:27
lyarwoodbauzas: https://github.com/openstack/nova/blob/49c3ac7dfa7aca5504f9d19958ff82a40e47f5a1/nova/virt/libvirt/driver.py#L5111-L511716:28
*** gyee has joined #openstack-nova16:28
*** k_mouza has joined #openstack-nova16:28
bauzaslyarwood: gosh lord, thank you16:28
lyarwoodbauzas: https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L8493-L849816:28
lyarwoodartom: ^ did I miss anything btw? ./me thinks you know more about this16:29
bauzaslyarwood: Saint Github tells me that the only reference of LibvirtConfigGuestCPUTuneVCPUSched is https://github.com/openstack/nova/blob/49c3ac7dfa7aca5504f9d19958ff82a40e47f5a1/nova/virt/libvirt/driver.py#L511316:30
bauzasthat's enough for me16:30
*** k_mouza has quit IRC16:33
*** ociuhandu has joined #openstack-nova16:35
*** ociuhandu has quit IRC16:41
melwittlyarwood: ack will do16:43
lyarwoodthanks16:45
*** mkrai_ has quit IRC16:46
*** k_mouza has joined #openstack-nova16:46
*** mlavalle has quit IRC16:46
*** mlavalle has joined #openstack-nova16:48
*** k_mouza has quit IRC16:50
*** psachin has quit IRC16:53
*** k_mouza has joined #openstack-nova16:57
openstackgerritMerged openstack/python-novaclient stable/ussuri: Switch legacy Zuul jobs to native Zuul v3 jobs  https://review.opendev.org/74048816:58
openstackgerritTakashi Natsume proposed openstack/python-novaclient stable/ussuri: Add a cleanup for a server in a functional test  https://review.opendev.org/74375617:00
*** derekh has quit IRC17:00
*** k_mouza has quit IRC17:01
*** mgariepy has joined #openstack-nova17:05
openstackgerritStephen Finucane proposed openstack/nova stable/ussuri: Handle multiple 'vcpusched' elements during live migrate  https://review.opendev.org/74380417:06
openstackgerritStephen Finucane proposed openstack/nova stable/train: Handle multiple 'vcpusched' elements during live migrate  https://review.opendev.org/74380517:10
*** k_mouza has joined #openstack-nova17:15
*** k_mouza has quit IRC17:19
*** dtantsur is now known as dtantsur|afk17:21
*** sapd1 has quit IRC17:26
*** k_mouza has joined #openstack-nova17:35
*** k_mouza has quit IRC17:39
*** artom has quit IRC17:45
*** artom has joined #openstack-nova17:46
*** artom has quit IRC17:52
*** k_mouza has joined #openstack-nova17:54
*** k_mouza has quit IRC17:58
*** artom has joined #openstack-nova18:07
*** k_mouza has joined #openstack-nova18:10
*** k_mouza has quit IRC18:14
*** jsuchome has quit IRC18:15
*** maciejjozefczyk has quit IRC18:24
*** vishalmanchanda has quit IRC18:29
*** brinzhang_ has joined #openstack-nova18:30
*** k_mouza has joined #openstack-nova18:31
*** artom has quit IRC18:31
*** brinzhang0 has quit IRC18:33
*** k_mouza has quit IRC18:39
*** artom has joined #openstack-nova18:48
*** artom has quit IRC18:48
*** artom has joined #openstack-nova18:48
*** ralonsoh has quit IRC18:53
*** lbragstad_ is now known as lbragstad19:01
*** zer0c00l has joined #openstack-nova19:05
*** zer0c00l has quit IRC19:06
*** efried has quit IRC19:07
*** nweinber has quit IRC19:10
*** efried has joined #openstack-nova19:13
*** zer0c00l has joined #openstack-nova19:13
*** k_mouza has joined #openstack-nova19:22
*** k_mouza has quit IRC19:27
openstackgerritMerged openstack/nova master: Handle multiple 'vcpusched' elements during live migrate  https://review.opendev.org/74356819:31
*** k_mouza has joined #openstack-nova19:33
*** k_mouza has quit IRC19:37
*** k_mouza has joined #openstack-nova19:42
openstackgerritSean McGinnis proposed openstack/nova master: Drop use of legacy encryptor specifiers  https://review.opendev.org/74383419:45
*** bbowen has quit IRC19:46
*** k_mouza has quit IRC19:47
*** xek has joined #openstack-nova19:49
*** k_mouza has joined #openstack-nova20:05
*** yaawang has quit IRC20:07
*** yaawang has joined #openstack-nova20:07
*** k_mouza has quit IRC20:09
*** ociuhandu has joined #openstack-nova20:39
*** ociuhandu has quit IRC20:43
*** martinkennelly has quit IRC20:50
*** slaweq has quit IRC21:07
*** xek has quit IRC21:10
*** raildo has quit IRC21:26
*** bbowen has joined #openstack-nova21:33
openstackgerritmelanie witt proposed openstack/nova master: WIP Add option to sync instance power states during shutdown  https://review.opendev.org/74384821:37
*** artom has quit IRC22:04
openstackgerritMerged openstack/nova stable/train: objects: Update keypairs when saving an instance  https://review.opendev.org/74270922:31
openstackgerritMerged openstack/nova master: Replace assertItemsEqual with assertCountEqual  https://review.opendev.org/74057522:31
*** rcernin_ has joined #openstack-nova22:35
*** rcernin_ has quit IRC22:48
*** rcernin has joined #openstack-nova22:48
*** tkajinam has joined #openstack-nova22:53
*** ociuhandu has joined #openstack-nova23:01
*** ociuhandu has quit IRC23:06
*** tosky has quit IRC23:21
*** mriedem has left #openstack-nova23:29

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!