Posted	Nick	Remark
#openstack-nova - 2020-03-27
16:55:26	dansmith	gmann: I didn't update that commit message because I figured we're squashing that after we review the diff, but will -W with that to make it obvious
16:56:03	gmann	dansmith: ohk, did not read your comment.
16:56:39	dansmith	gmann: I also didn't update the api-ref just yet, so just FYI
16:59:53	gmann	got it,
17:11:47	openstackgerrit	Sylvain Bauza proposed openstack/nova master: Provide the parent pGPU when creating a new vGPU https://review.opendev.org/715489
17:11:47	openstackgerrit	Sylvain Bauza proposed openstack/nova master: WIP: Support different vGPU types per pGPU https://review.opendev.org/715490
17:33:33	gmann	dansmith: commented, +1 on adding tag in updatable items
17:37:38	gmann	dansmith: policy patch lgtm with 1 comment on passing project_id as target for admin policy.
17:45:59	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Fix os-ips policy to be admin_or_owner https://review.opendev.org/715496
17:46:32	hansmoleman	so did the PATCH thing get dropped because https://review.opendev.org/#/c/693828/ is still talking about PATCH
17:50:22	dansmith	hansmoleman: I just commented, but as we discussed here and on the last patch, I haven't updated all the meta and docs yet, nor squashed the last patch into the first one as planned because I'm trying to allow for review of just the PATCH->PUT code changes before I go moving everything around
17:52:39	hansmoleman	ok so it's diverged quite a bit from the spec, i suppose in an effort to still try to get something in for ussuri?
17:53:01	hansmoleman	because i was originally advocating doing this with PUT in the spec but not with a swap volume operation, but then that all changed at some point to PATCH
17:53:11	hansmoleman	and now the PATCH thing is getting dropped
17:53:31	dansmith	hansmoleman: we discussed that in IRC yesterday
17:53:45	dansmith	gibi, gmann, myself and brin
17:58:04	sean-k-mooney	dansmith: just an fyi im finishing up for the day shortly. ill continue working on https://review.opendev.org/#/c/715326/ on monday but i think getting rebuild and evacuation working with cyborg should not be too hard
17:58:18	dansmith	sean-k-mooney: sounds good, enjoy the weekend
17:58:29	sean-k-mooney	you too o/
18:04:51	openstackgerrit	Stephen Finucane proposed openstack/nova stable/queens: Functional test for UnexpectedDeletingTaskStateError https://review.opendev.org/715405
18:04:52	openstackgerrit	Stephen Finucane proposed openstack/nova stable/queens: Unplug VIFs as part of cleanup of networks https://review.opendev.org/715406
18:07:12	stephenfin	artom: if you've time to kill before the end of the day, the series ending here continues work on cleaning up a lot of crap from hardware.py https://review.opendev.org/#/c/714703/
18:07:40	artom	stephenfin, tbh probably not today, because kids
18:07:53	stephenfin	fair
18:15:47	artom	stephenfin, but I'll keep that on my review list
18:16:22	artom	stephenfin, and if you're still around, gentle reminder that https://review.opendev.org/#/c/714228/ and the patches on top are still a thing
18:16:59	artom	And if you're not, happy 🍺'ing
19:52:58	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing ips policies https://review.opendev.org/715477
20:01:13	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in os-ips https://review.opendev.org/715529
20:08:23	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing ips policies https://review.opendev.org/715477
20:08:36	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in os-ips https://review.opendev.org/715529
20:32:47	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add new default roles in os-ips policies https://review.opendev.org/715545
23:16:46	openstackgerrit	Merged openstack/nova stable/stein: libvirt: Provide the backing file format when creating qcow2 disks https://review.opendev.org/710790
#openstack-nova - 2020-03-28
00:33:12	gmann	:)
05:16:27	openstackgerrit	Sundar Nadathur proposed openstack/nova master: Enable and use COMPUTE_ACCELERATORS trait. https://review.opendev.org/699554
05:16:27	openstackgerrit	Sundar Nadathur proposed openstack/nova master: Bump compute rpcapi version and reduce Cyborg calls. https://review.opendev.org/704227
05:16:28	openstackgerrit	Sundar Nadathur proposed openstack/nova master: Block unsupported instance operations with accelerators. https://review.opendev.org/674726
05:16:28	openstackgerrit	Sundar Nadathur proposed openstack/nova master: Add cyborg tempest job. https://review.opendev.org/670999
05:25:21	openstackgerrit	Kevin Zhao proposed openstack/nova master: fix ut error on arm64 https://review.opendev.org/713163
06:22:34	sapd1	Hi everyone! Can I test this patch https://review.opendev.org/#/q/topic:bp/virt-bfv-instance-rescue+(status:open+OR+status:merged) right now?
13:48:42	openstackgerrit	Takashi Kajinami proposed openstack/nova master: WIP: Add new option to restrict migration within az https://review.opendev.org/715621
19:07:08	openstackgerrit	Andreas Jaeger proposed openstack/nova-specs master: Update hacking for Python3 https://review.opendev.org/715650
19:13:08	openstackgerrit	Andreas Jaeger proposed openstack/os-vif master: Update hacking for Python3 https://review.opendev.org/715651
20:41:23	openstackgerrit	Victor Coutellier proposed openstack/python-novaclient master: Microversion 2.83 - Add more filters for the nova list command https://review.opendev.org/713089
#openstack-nova - 2020-03-29
01:57:03	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Fix os-ips policy to be admin_or_owner https://review.opendev.org/715496
02:02:56	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing ips policies https://review.opendev.org/715477
02:05:59	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing ips policies https://review.opendev.org/715477
02:09:03	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in os-ips https://review.opendev.org/715529
02:09:18	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add new default roles in os-ips policies https://review.opendev.org/715545
02:42:26	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Correct limits policy check_str https://review.opendev.org/715672
03:54:42	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing limits policies https://review.opendev.org/715674
03:57:20	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing limits policies https://review.opendev.org/715674
04:03:55	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Correct limits policy check_str https://review.opendev.org/715672
04:04:17	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing limits policies https://review.opendev.org/715674
05:23:34	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Combine the limits policies in single place https://review.opendev.org/715678
05:25:10	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Combine the limits policies in single place https://review.opendev.org/715678
05:38:06	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing limits policies https://review.opendev.org/715674
05:47:52	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in limits policy https://review.opendev.org/715680
11:47:09	openstackgerrit	jayaditya gupta proposed openstack/nova master: Support for --overwrite flag for nova-manage placement heal_allocations command Closes-Bug:1868997 https://review.opendev.org/715395
12:02:11	openstackgerrit	jayaditya gupta proposed openstack/nova master: Support for nova-manage placement heal_allocations --cell Closes-bug: #1868531 Change-Id: I98b3280583a6d12461d8aa52e5714d7606b84369 https://review.opendev.org/714459
12:02:13	openstack	bug 1868531 in OpenStack Compute (nova) "nova manage placement doesn't support registration per Cell." [Undecided,New] https://launchpad.net/bugs/1868531 - Assigned to jayaditya gupta (jayssj11)
14:24:00	openstackgerrit	Brin Zhang proposed openstack/nova master: Add instance actions v284 samples test https://review.opendev.org/706251
14:34:06	openstackgerrit	Brin Zhang proposed openstack/python-novaclient master: Microversion 2.84 - action event fault details https://review.opendev.org/714561
21:19:06	openstackgerrit	Mohammed Naser proposed openstack/nova master: DNM: test out running with pep8 inline comments https://review.opendev.org/715728
21:19:07	openstackgerrit	Mohammed Naser proposed openstack/nova master: DNM: this is a failing change for inline pep8 https://review.opendev.org/715729
#openstack-nova - 2020-03-30
02:24:12	openstackgerrit	Merged openstack/nova master: Expose instance action event details out of the API https://review.opendev.org/694430
02:24:59	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add new default roles in limits policies https://review.opendev.org/715760
02:36:36	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Pass the actual target in limits policy https://review.opendev.org/715761
02:48:37	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add new default roles in limits policies https://review.opendev.org/715760
02:50:10	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add new default roles in limits policies https://review.opendev.org/715760
02:50:28	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Pass the actual target in limits policy https://review.opendev.org/715761
02:59:34	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add new default roles in os-agents policies https://review.opendev.org/701648
03:00:30	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Pass the actual target in os-agents policy https://review.opendev.org/701649
03:02:32	openstackgerrit	Brin Zhang proposed openstack/nova master: Add instance actions v284 samples test https://review.opendev.org/706251
03:07:10	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add new default roles in limits policies https://review.opendev.org/715760
03:07:43	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Correct limits policy check_str https://review.opendev.org/715672
03:07:58	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Combine the limits policies in single place https://review.opendev.org/715678
03:08:12	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing limits policies https://review.opendev.org/715674
03:08:26	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in limits policy https://review.opendev.org/715680
03:32:06	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add new default roles in limits policies https://review.opendev.org/715760
03:32:29	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add new default roles in limits policies https://review.opendev.org/715760
03:33:07	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add new default roles in limits policies https://review.opendev.org/715760
03:33:26	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Pass the actual target in limits policy https://review.opendev.org/715761
03:39:52	openstackgerrit	Kevin Zhao proposed openstack/nova master: libvirt: change UEFI check to handle AArch64 better https://review.opendev.org/714311
06:42:10	openstackgerrit	Kevin Zhao proposed openstack/nova master: [DNM] Run tempest jobs on Arm64 needed fix https://review.opendev.org/714564
07:22:28	openstackgerrit	Sylvain Bauza proposed openstack/nova master: FUP for Add a placement audit command https://review.opendev.org/715797
07:23:37	bauzas	stephenfin: morning, when you're up, I just provided a FUP based on your comments for https://review.opendev.org/#/c/670112 : https://review.opendev.org/#/c/715797/1
07:23:50	bauzas	I'd appreciate if you could review those soon
08:41:33	nightmare_unreal	hey like we have assertFlavorMatchesAllocation() do we have opposite of it ?
08:41:44	nightmare_unreal	like where assertflavor not match
08:50:57	openstackgerrit	John Garbutt proposed openstack/nova master: Add stub unified limits driver https://review.opendev.org/712137
09:32:25	luyao	stephenfin: Hi, your comments are addressed. :) https://review.opendev.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/support-live-migration-with-virtual-persistent-memory
09:56:00	openstackgerrit	jayaditya gupta proposed openstack/nova master: Support for --overwrite flag for nova-manage placement heal_allocations command Closes-Bug:#1868997 https://review.opendev.org/715395
10:18:35	openstackgerrit	Sylvain Bauza proposed openstack/nova master: FUP for Add a placement audit command https://review.opendev.org/715797
10:18:55	bauzas	stephenfin: thanks for noting the pep8 issue. fixed ^
10:20:15	openstackgerrit	Andreas Jaeger proposed openstack/nova-specs master: Update hacking for Python3 https://review.opendev.org/715650
10:21:28	openstackgerrit	Andreas Jaeger proposed openstack/os-vif master: Update hacking for Python3 https://review.opendev.org/715651
10:32:34	gibi	bauzas: hi! Does https://review.opendev.org/#/c/715797/ solves the bug 1793569 or we need other things still?
10:32:35	openstack	bug 1793569 in OpenStack Compute (nova) "Add placement audit commands" [Wishlist,In progress] https://launchpad.net/bugs/1793569 - Assigned to Sylvain Bauza (sylvain-bauza)
10:33:08	bauzas	gibi: just a FUP

Earlier   Later