Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2020-04-07
17:52:43 gmann if we keep default same as old 'update' policy then it is ok
17:53:47 dansmith .../
17:53:49 gmann dansmith: without overeride policy they get permission automatically but if overriding the swap policy then even no-deprecation is issue
17:54:06 dansmith I don't understand what you're saying
17:54:45 dansmith if the policy defaults are kept, then if I roll out ussuri, users who should have swap permissions will lose that ability with microversion 2.85 until I grant them update permission
17:55:05 dansmith but they would continue to be able to do it with 2.1
17:55:16 dansmith however, with what I have, they will see no change in behavior
17:55:43 gmann with defaults they are always have update permission as it is subset of swap default.
17:55:59 gmann with defaults, there is no issue on your proposal
17:56:37 gmann i am thinking if uses has override the old swap policy and want to do the same for new update policy
17:57:07 gmann if we consider either case default or override - swap is always supeset then it is ok
17:58:28 gmann but yes, we can not handle all cases in this case
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

Earlier   Later