Posted	Nick	Remark
#openstack-nova - 2019-06-03
18:00:46	openstackgerrit	Boris Bobrov proposed openstack/nova master: Allow guest devices to include https://review.opendev.org/644564
18:01:47	openstackgerrit	Boris Bobrov proposed openstack/nova master: Detect that SEV is required and enable iommu for devices https://review.opendev.org/644565
18:02:28	openstackgerrit	Boris Bobrov proposed openstack/nova master: Use element when SEV is required https://review.opendev.org/662557
18:02:42	openstackgerrit	Boris Bobrov proposed openstack/nova master: Use element when SEV is required https://review.opendev.org/662557
18:02:54	openstackgerrit	Boris Bobrov proposed openstack/nova master: Enable memory locking if SEV is requested https://review.opendev.org/662558
18:03:40	openstackgerrit	Boris Bobrov proposed openstack/nova master: Add element to guest config for AMD SEV https://review.opendev.org/636318
18:05:58	openstackgerrit	Boris Bobrov proposed openstack/nova master: Allow guest devices to include https://review.opendev.org/644564
18:06:42	openstackgerrit	Boris Bobrov proposed openstack/nova master: Detect that SEV is required and enable iommu for devices https://review.opendev.org/644565
18:15:07	openstackgerrit	Boris Bobrov proposed openstack/nova master: Use element when SEV is required https://review.opendev.org/662557
18:15:36	openstackgerrit	Boris Bobrov proposed openstack/nova master: Enable memory locking if SEV is requested https://review.opendev.org/662558
18:16:26	openstackgerrit	Boris Bobrov proposed openstack/nova master: Enable memory locking if SEV is requested https://review.opendev.org/662558
18:21:52	efried	kashyap: Did you have a spec at some point about "live resize"?
18:22:25	efried	ah, it was claudiub
18:24:35	openstackgerrit	Merged openstack/nova master: Avoid lazy-loading instance.flavor in cold migration https://review.opendev.org/662683
18:38:31	openstackgerrit	Sylvain Bauza proposed openstack/nova master: Pass allocations to virt drivers when resizing https://review.opendev.org/589085
19:08:51	openstackgerrit	Matt Riedemann proposed openstack/nova master: Add heal_instance_mappings command https://review.opendev.org/655908
19:19:38	openstackgerrit	Matt Riedemann proposed openstack/nova master: Handle lazy-load of Migration.cross_cell_move https://review.opendev.org/662833
19:25:29	openstackgerrit	Merged openstack/nova master: Reset the stored logs at each notification test steps https://review.opendev.org/656844
19:25:38	openstackgerrit	Merged openstack/nova master: Remove remaining vestiges of fake_libvirt_utils from unit tests. https://review.opendev.org/643896
19:39:27	openstackgerrit	Matt Riedemann proposed openstack/nova stable/queens: Add regression recreate test for bug 1830747 https://review.opendev.org/662774
19:39:29	openstack	bug 1830747 in OpenStack Compute (nova) "Error 500 trying to migrate an instance after wrong request_spec" [High,In progress] https://launchpad.net/bugs/1830747 - Assigned to Matt Riedemann (mriedem)
19:42:09	openstackgerrit	Merged openstack/nova master: Bump openstackdocstheme to 1.30.0 https://review.opendev.org/662353
19:44:12	openstackgerrit	Merged openstack/nova master: Exclude broken ironicclient versions 2.7.1 https://review.opendev.org/662404
19:53:13	mriedem	cross-cell-resize is out of the runway slot and https://blueprints.launchpad.net/nova/+spec/add-host-and-hypervisor-hostname-flag-to-create-server takes its place (2nd time in i think)
20:48:59	mriedem	melwitt: efried: what do you want to do about a reno for https://review.opendev.org/#/c/579897/6
20:50:09	efried	mriedem: oh, yeah, a reno is a good idea. /me goes and pulls up the reno template...
20:51:11	efried	sheesh, that's not very helpful.
20:51:28	efried	mriedem: 'fixes' is fine I guess.
20:53:07	mriedem	ok might as well get my nits while in there if you're updating
20:54:43	efried	ack
20:54:47	efried	gimme a few
21:31:27	openstackgerrit	Merged openstack/nova master: Noop CantStartEngineError in targets_cell if API DB not configured https://review.opendev.org/581912
22:13:16	openstackgerrit	Merged openstack/nova master: Workaround missing RequestSpec.instance_group.uuid https://review.opendev.org/661786
22:13:30	openstackgerrit	Merged openstack/nova master: Set/get group uuid when transforming RequestSpec to/from filter_properties https://review.opendev.org/662550
22:13:39	openstackgerrit	Merged openstack/nova master: Cleanup migrate flags https://review.opendev.org/651056
22:20:41	openstackgerrit	Eric Fried proposed openstack/nova master: Hide hypervisor id on windows guests https://review.opendev.org/579897
22:20:51	efried	mriedem, melwitt: ^
22:20:58	efried	hopefully I did that right.
22:21:15	mriedem	Sundar: efried: sean-k-mooney: gibi: my first attempt at the nova/cyborg spec https://review.opendev.org/#/c/603955/
22:21:47	efried	thanks mriedem
22:21:55	openstackgerrit	Michael Still proposed openstack/nova master: Remove fake_libvirt_utils users in functional testing. https://review.opendev.org/644793
22:25:01	Sundar	mriedem: Thank you very much. Will respond soon.
23:41:47	openstackgerrit	Dustin Cowles proposed openstack/nova-specs master: Spec: Use OpenStack SDK in Nova https://review.opendev.org/662881
23:59:51	openstackgerrit	Artom Lifshitz proposed openstack/nova master: DNM: Run tempest-full-py3 with q35 machine type https://review.opendev.org/662887
#openstack-nova - 2019-06-04
00:51:13	openstackgerrit	Matt Riedemann proposed openstack/nova master: Hide hypervisor id on windows guests https://review.opendev.org/579897
01:01:09	openstackgerrit	Matt Riedemann proposed openstack/nova stable/stein: Workaround missing RequestSpec.instance_group.uuid https://review.opendev.org/662894
01:03:28	openstackgerrit	Matt Riedemann proposed openstack/nova stable/rocky: Workaround missing RequestSpec.instance_group.uuid https://review.opendev.org/662895
01:32:27	openstackgerrit	Brin Zhang proposed openstack/nova master: Replace the invalid index of nova-rocky releasenote https://review.opendev.org/662897
02:10:03	openstackgerrit	zhaixiaojun proposed openstack/python-novaclient master: Bump openstackdocstheme to 1.30.0 https://review.opendev.org/662905
02:53:32	openstackgerrit	zhaixiaojun proposed openstack/python-novaclient master: Blacklist python-cinderclient 4.0.0 https://review.opendev.org/662912
03:45:34	openstackgerrit	Artom Lifshitz proposed openstack/nova master: DNM: Run tempest-full-py3 with q35 machine type https://review.opendev.org/662887
04:08:33	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: WIP:Introduce scope_types in os-services https://review.opendev.org/645427
05:54:31	openstackgerrit	Merged openstack/nova master: Follow up for counting quota usage from placement https://review.opendev.org/662056
06:36:12	openstackgerrit	Merged openstack/nova stable/stein: xenapi/agent: Change openssl error handling https://review.opendev.org/656304
07:35:37	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add new default roles and mapping in policy base class https://review.opendev.org/645452
07:39:22	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: WIP:Introduce scope_types in os-services https://review.opendev.org/645427
07:39:42	openstackgerrit	Boxiang Zhu proposed openstack/nova master: Validate requested host/node during servers create https://review.opendev.org/661237
07:40:50	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: WIP: Add new default roles in os-services API policies https://review.opendev.org/648480
07:57:40	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: WIP: Add new default roles in os-services API policies https://review.opendev.org/648480
07:58:12	kashyap	efried: Indeed, it was not me (on "live resize"). But I've followed it on-and-off
08:06:27	openstackgerrit	Balazs Gibizer proposed openstack/nova stable/stein: Reset the stored logs at each notification test steps https://review.opendev.org/662965
08:24:17	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: WIP:Introduce scope_types in servers API https://review.opendev.org/662968
08:29:34	johnthetubaguy	gmann: I couldn't get my unit test to do what I expected with this: https://review.opendev.org/#/c/657823/2
08:29:40	johnthetubaguy	I am probably doing something dumb
08:31:12	gmann	johnthetubaguy: ok, i am updating the spec now with more mapping data. I will check your unit test thing after tnat
08:32:04	johnthetubaguy	gmann: sweet, lets refresh the spec first for sure
08:32:12	johnthetubaguy	this is implementation detail
08:33:19	kashyap	johnthetubaguy: In your copious free time, mind having a look at the Secure Boot spec, please? -- https://review.openstack.org/#/c/506720/
08:34:01	johnthetubaguy	its spec review day, so totally will take a look at that
08:34:02	kashyap	Before I respin to rephrase some text, just want to check if there's anything else anyone has. Near as I see, I've addressed all the feedback
08:34:09	kashyap	johnthetubaguy: Most excellent
08:34:27	johnthetubaguy	that and I have customers who will want this
08:36:58	kashyap	johnthetubaguy: Cool!
08:37:41	kashyap	johnthetubaguy: A good chunk of stuff is in Work Items
08:38:05	kashyap	johnthetubaguy: The thid point, that shows the use 'UefiShell.iso' is info-only; no need for us to run any of that.
08:38:42	kashyap	Because, all distributions that matter, ship OVMF "variables" (or "vars") files that have default UEFI keys enrolled.
08:38:52	kashyap	Anyway, it's a detail. We'll get to it when you're reviewing.
08:40:51	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: WIP: Add new default roles in servers API policies https://review.opendev.org/662971
08:42:09	openstackgerrit	Balazs Gibizer proposed openstack/nova master: Change the default of notification_format to unversioned https://review.opendev.org/603079
08:57:10	johnthetubaguy	kashyap: I see what you mean about the work items thing... its normally like three lines
08:57:17	kashyap	:D
08:57:43	kashyap	johnthetubaguy: I wanted to remove some of the background stuff, to reduce text. As most of the stuff is automated for us by libvirt's new interface
08:57:59	kashyap	And I've working with distributions to ensure they're shipping the variables files
09:05:04	johnthetubaguy	kashyap: yeah, I think you could, finding it interesting, but don't feel qualified to review it
09:05:38	kashyap	johnthetubaguy: Okido; I'll leave it as is. I made sure to write clear sentences that add some value :D
09:07:17	johnthetubaguy	kashyap: I am putting some comments on the bits I would trim, like that list of distro details, I think just two distros to illustrate the complexity would do
09:07:41	kashyap	johnthetubaguy: Sure. Comment away
09:07:58	kashyap	johnthetubaguy: Aside: If you want to try a SecureBoot-enabled guest, here's an automated script I wrote for it: https://kashyapc.fedorapeople.org/Create-a-SecureBoot-enabled-VM.bash
09:08:17	johnthetubaguy	the ovmf loader paths in the xml... I thought libvirt did all that for us/
09:08:57	kashyap	johnthetubaguy: Are you referring to point-2 in the Work Items section?
09:09:07	kashyap	johnthetubaguy: Yes, the new libvirt does handle that for us.
09:09:19	johnthetubaguy	yeah, that is the bit
09:09:35	kashyap	I have a TODO to trim that second point to reflect that.
09:19:06	johnthetubaguy	kashyap: so this sounds harsh... but I think you need to turn your spec upside down, given the new libvirt version, and I guess that is what you are thinking too
09:19:35	kashyap	johnthetubaguy: Sorry, what do you mean upside down?
09:19:49	kashyap	johnthetubaguy: You mean, push the content from Work Items a bit above?
09:20:12	kashyap	Noting clearly that the new libvirt (and QEMU, OVMF / EDK2) version handles most of the work for us?
09:20:46	kashyap	(All harsh feedback is welcome :-) After spending so much time on my text, I need other eyes to help edit it.)
09:20:56	kashyap	s/other/others'/
09:21:32	johnthetubaguy	kashyap: trying to work out how to describe what I am thinking :)

Earlier   Later