Posted	Nick	Remark
#openstack-nova - 2020-04-07
17:59:56	gmann	dansmith: i think it is ok with your approach and we update policy doc that for swap + update we only check swap policy which is expected to be superset of 'update'
18:00:25	dansmith	okay
18:00:36	dansmith	will you comment on the patch about that?
18:00:45	gmann	doing,
18:00:53	openstackgerrit	Merged openstack/nova master: Pass the actual target in os-flavor-manage policy https://review.opendev.org/714822
18:01:04	dansmith	thanks
18:10:26	gmann	dansmith: done. added comment on tests cases, few are not needed and does not test much
18:32:26	openstackgerrit	Merged openstack/nova master: Fix resume server policy to be admin_or_owner https://review.opendev.org/717561
18:32:34	openstackgerrit	Merged openstack/nova master: Add test coverage of existing suspend server policies https://review.opendev.org/717554
18:32:42	openstackgerrit	Merged openstack/nova master: Introduce scope_types in suspend server https://review.opendev.org/717582
18:32:50	openstackgerrit	Merged openstack/nova master: Add new default roles in suspend server policies https://review.opendev.org/717583
18:32:57	openstackgerrit	Merged openstack/nova master: Introduce scope_types in limits policy https://review.opendev.org/715680
18:33:05	openstackgerrit	Merged openstack/nova master: Add new default roles in limits policies https://review.opendev.org/715760
18:33:11	openstackgerrit	Merged openstack/nova master: Add test coverage of existing server group policies https://review.opendev.org/717173
18:39:57	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: DNM testing enforce new defaults flag https://review.opendev.org/717945
18:40:11	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: DNM testing enforce new defaults flag https://review.opendev.org/717945
18:52:38	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add test coverage of existing server topology policies https://review.opendev.org/717524
18:53:05	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in server topology https://review.opendev.org/717584
18:53:19	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add new default roles in server topology policies https://review.opendev.org/717585
19:03:53	openstackgerrit	Merged openstack/nova master: Introduce scope_types in server group policy https://review.opendev.org/717174
19:04:00	openstackgerrit	Merged openstack/nova stable/queens: Functional test for UnexpectedDeletingTaskStateError https://review.opendev.org/715405
19:29:23	openstackgerrit	Merged openstack/nova stable/queens: Unplug VIFs as part of cleanup of networks https://review.opendev.org/715406
19:40:16	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add new default roles in server topology policies https://review.opendev.org/717585
19:43:20	melwitt	gmann: hey, what were you meaning here? https://review.opendev.org/#/c/716809/1/tempest/scenario/test_server_advanced_ops.py are you saying that the test involves passing a specific network during server create? and that getting the 409 multiple networks error means somehow the list_networks is returning None?
19:43:56	melwitt	oops, I should have written that in -qa
19:45:15	gmann	melwitt: yeah. that is one possibility otherwise scenario base tests always send the network in create request
19:45:47	gmann	it was API tests where we reply on tenants network created during setup of credential.
19:45:59	gmann	s/reply/rely
19:46:12	melwitt	ok
20:32:03	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: DNM testing enforce new defaults flag https://review.opendev.org/717945
20:40:34	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add new default roles in server group policies https://review.opendev.org/717175
20:50:45	openstackgerrit	Sylvain Bauza proposed openstack/nova master: Support different vGPU types per pGPU https://review.opendev.org/715490
20:50:45	openstackgerrit	Sylvain Bauza proposed openstack/nova master: Provide the parent pGPU when creating a new vGPU https://review.opendev.org/715489
20:50:46	openstackgerrit	Sylvain Bauza proposed openstack/nova master: (WIP) Functional test with pGPUs https://review.opendev.org/717975
22:09:00	openstackgerrit	sean mooney proposed openstack/nova master: cyborg evacuate support https://review.opendev.org/715326
23:01:09	openstackgerrit	Sylvain Bauza proposed openstack/nova master: (WIP) Functional test with pGPUs https://review.opendev.org/717975
23:10:48	openstackgerrit	Sylvain Bauza proposed openstack/nova master: WIP: Functional test with pGPUs https://review.opendev.org/717975
#openstack-nova - 2020-04-08
00:49:38	openstackgerrit	Arthur Dayne proposed openstack/nova master: libvirt:driver:Disallow AIO=native when 'O_DIRECT' is not available https://review.opendev.org/682772
00:57:30	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Pass the actual target in server group policy https://review.opendev.org/717176
01:28:33	brinzhang_	gmann:https://review.opendev.org/#/c/711194 this patch, I understand damsmith's approach is ok for you, right?
01:29:23	brinzhang_	gmann: https://review.opendev.org/#/c/711194/14/nova/tests/unit/policies/test_volumes.py@224 this case I should move to class VolumeAttachScopeTypePolicyTest ?
01:30:26	gmann	brinzhang_: yes, with mentioning the same in policy description doc
01:30:52	gmann	brinzhang_: VolumeAttachScopeTypePolicyTest should run all tests in is its parent class
01:31:14	gmann	you can update in same class which should run in scope test class also
01:32:50	brinzhang_	gmann: you mean, keep that case in parent class and copy it to the class VolumeAttachScopeTypePolicyTest?
01:33:23	gmann	brinzhang_: no, i mean only update the tests with comments not the location of tests
01:36:41	brinzhang_	gmann: add "new 'update' policy about 'for swap + update' request (which is possible only >2.85) only <swap policy> is checked. We expect <swap policy> to be always superset of this policy permission." to comment this test?
01:38:35	gmann	brinzhang_: that is to update in 'update' policy doc
01:38:44	gmann	policy descritpnio
01:45:22	brinzhang_	gmann: two things, 1) add above description to https://review.opendev.org/#/c/711194/14/nova/policies/volumes_attachments.py@71 and 2):I should add comments on the test case, such as: """ to be always superset of update API policy permission"""
01:51:24	gmann	brinzhang_: only doc is fine. tests case is ok with verification only
02:03:46	openstackgerrit	Huaqiang Wang proposed openstack/nova master: Introduce 'pcpuset' field for InstanceNUMACell https://review.opendev.org/714658
02:03:47	openstackgerrit	Huaqiang Wang proposed openstack/nova master: hardware: Add support for 'hw:cpu_dedicated_mask' extra spec https://review.opendev.org/714706
02:03:47	openstackgerrit	Huaqiang Wang proposed openstack/nova master: Introduce the 'CPUAllocationPolicy.MIXED' enum https://review.opendev.org/716267
02:03:48	openstackgerrit	Huaqiang Wang proposed openstack/nova master: Calculate the pinned and unpinned CPU usage for mixed instance https://review.opendev.org/716269
02:03:48	openstackgerrit	Huaqiang Wang proposed openstack/nova master: computer: validate cpu pinning configuration for mixed instance https://review.opendev.org/716268
02:03:49	openstackgerrit	Huaqiang Wang proposed openstack/nova master: Setup 'mixed' instance through 'PCPU' and 'VCPU' resource https://review.opendev.org/713355
02:03:49	openstackgerrit	Huaqiang Wang proposed openstack/nova master: mark mixed policy as ready, bump compute service version https://review.opendev.org/716270
02:03:50	openstackgerrit	Huaqiang Wang proposed openstack/nova master: libvirt/hardware: refactor numa_get_constraints https://review.opendev.org/718294
02:03:50	openstackgerrit	Huaqiang Wang proposed openstack/nova master: metadata: export the vCPU IDs that are pinning on the host CPUs https://review.opendev.org/688936
02:05:01	openstackgerrit	Brin Zhang proposed openstack/nova master: Separate update and swap volume policies https://review.opendev.org/711194
02:05:15	brinzhang_	gmann: done, pls review again, thanks
02:06:10	openstackgerrit	Brin Zhang proposed openstack/nova master: Separate update and swap volume policies https://review.opendev.org/711194
02:18:57	openstackgerrit	Huaqiang Wang proposed openstack/nova master: mark mixed policy as ready, bump compute service version https://review.opendev.org/716271
02:18:58	openstackgerrit	Huaqiang Wang proposed openstack/nova master: Setup 'mixed' instance through 'PCPU' and 'VCPU' resource https://review.opendev.org/713355
02:18:58	openstackgerrit	Huaqiang Wang proposed openstack/nova master: libvirt/hardware: refactor numa_get_constraints https://review.opendev.org/718294
02:18:59	openstackgerrit	Huaqiang Wang proposed openstack/nova master: metadata: export the vCPU IDs that are pinning on the host CPUs https://review.opendev.org/688936
02:21:20	openstackgerrit	Ghanshyam Mann proposed openstack/nova master: Add new default roles in server group policies https://review.opendev.org/717175
02:25:40	openstackgerrit	Huaqiang Wang proposed openstack/nova master: computer: validate cpu pinning configuration for mixed instance https://review.opendev.org/716268
02:25:41	openstackgerrit	Huaqiang Wang proposed openstack/nova master: mark mixed policy as ready, bump compute service version https://review.opendev.org/716271
02:25:41	openstackgerrit	Huaqiang Wang proposed openstack/nova master: Calculate the pinned and unpinned CPU usage for mixed instance https://review.opendev.org/716269
02:25:42	openstackgerrit	Huaqiang Wang proposed openstack/nova master: Setup 'mixed' instance through 'PCPU' and 'VCPU' resource https://review.opendev.org/713355
02:25:42	openstackgerrit	Huaqiang Wang proposed openstack/nova master: libvirt/hardware: refactor numa_get_constraints https://review.opendev.org/718294
02:25:43	openstackgerrit	Huaqiang Wang proposed openstack/nova master: metadata: export the vCPU IDs that are pinning on the host CPUs https://review.opendev.org/688936
02:25:57	openstackgerrit	Brin Zhang proposed openstack/nova master: Separate update and swap volume policies https://review.opendev.org/711194
03:20:43	openstackgerrit	melanie witt proposed openstack/nova master: Reset the cell cache for database access in Service https://review.opendev.org/717662
03:34:33	openstackgerrit	Merged openstack/python-novaclient master: Microversion 2.83 - Add more filters for the nova list command https://review.opendev.org/713089
04:10:35	openstackgerrit	Brin Zhang proposed openstack/python-novaclient master: Microversion 2.84 - action event fault details https://review.opendev.org/714561
04:10:55	openstackgerrit	Brin Zhang proposed openstack/python-novaclient master: Microversion 2.85: Change volume-update CLI https://review.opendev.org/712651
04:11:13	openstackgerrit	Brin Zhang proposed openstack/python-novaclient master: Microversion 2.86 - Extra spec validation https://review.opendev.org/718057
04:11:30	openstackgerrit	Brin Zhang proposed openstack/python-novaclient master: Microversion 2.87 - Stable device boot from volume rescue https://review.opendev.org/714956
04:17:36	brinzhang_	alex_xu, gmann, gibi: I rebased the novalient merge conflict patch https://review.opendev.org/#/c/714561/9, need +A again, thanks
04:52:35	openstackgerrit	Qiu Fossen proposed openstack/nova master: The instance is volume backed and power state is PAUSED，shelve the instance failed https://review.opendev.org/711609
04:52:35	openstackgerrit	Qiu Fossen proposed openstack/nova master: The instance is volume backed and power state is PAUSED，shelve the instance failed https://review.opendev.org/711609
06:22:53	huaqiang	a
06:45:37	openstackgerrit	Merged openstack/python-novaclient master: Microversion 2.84 - action event fault details https://review.opendev.org/714561
06:55:04	bauzas	good morning Nova
07:17:31	alex_xu	gibi: would you like to take a look those patches https://review.opendev.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/use-pcpu-and-vcpu-in-one-instance, the refactor patches from stephenfin looks good now
07:37:56	gibi	stephenfin: did you see that https://review.opendev.org/#/c/713089/ merged last night?
07:38:18	gibi	stephenfin: do you think we should do something about it?
07:40:58	brinzhang	gibi: although we merged that patch, but the config_drive issue also exist, what do you want to plan to do?
07:42:54	gibi	brinzhang: there is a Nova API bug regarding config_drive filtering, that I don't want to solve this week due to FF and due to the fact that it would need a bigger API change to make it consistent across the API. Regarding the client side, the client now works as a pass through which is not wrong but stephenfin argued for a bit nicer CLI interface
07:43:40	gibi	alex_xu: I will try but no promisses
07:44:02	alex_xu	gibi: thanks
07:45:14	brinzhang	gibi: ok, I left some comment in bug 1871409, and I looked your chat records, yes, it's a bigger change
07:45:15	openstack	bug 1871409 in OpenStack Compute (nova) "cannot filter the list servers to show only those that does not have config drive" [Medium,In progress] https://launchpad.net/bugs/1871409 - Assigned to Balazs Gibizer (balazs-gibizer)
07:45:40	brinzhang	gibi: we are decide to completed this in gmann's next cleanup api?
07:45:53	gibi	brinzhang: no, we only added it to the list
07:47:02	brinzhang	gibi: ok, thanks, got it, if need my help that I can.
07:47:50	gibi	brinzhang: thanks. I would not rush on that as config_driver filtering is broken at least in the last 3 years and nobody filed a bug

Earlier   Later