Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2020-04-07
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
07:48:16 gibi so we will take the time to figure out how we want to fix it
07:49:38 brinzhang_ gibi: Yes, we rarely use config_drive to filter servers too.
07:54:33 brinzhang_ gibi: the bp/destroy-instance-with-datavolume separate patch updated and the novaclient also update, if you have time today, pls review again, I am sorry to add your work, thanks.
08:00:13 gibi brinzhang_: I have comment about the unit test https://review.opendev.org/#/c/711194
08:01:11 gibi now I will look at the client patch
08:01:24 brinzhang_ gibi: thanks, will update
08:05:09 brinzhang_ gibi: the test case that I missed the api_version=2.85, and I will add d-o-t to the request body, and set 2.85 in request
08:09:12 bauzas gibi: morning, FWIW I worked last night on the functional test, even if it's still a WIP, we verify whether traits work for two pGPUs https://review.opendev.org/#/c/717975/8/nova/tests/functional/libvirt/test_vgpu.py@214
08:09:29 bauzas and Zuul says +1 :p
08:10:30 bauzas (WIP because I need to update some tests for the fakelibvirt)
08:12:35 gibi bauzas: morning. ack
08:16:02 openstackgerrit Brin Zhang proposed openstack/nova master: Separate update and swap volume policies https://review.opendev.org/711194
08:16:13 brinzhang_ gibi: the test update done, thanks
08:19:22 gibi brinzhang_: could you quickly fix this nit? then I will +2 https://review.opendev.org/#/c/711194/18/nova/tests/unit/policies/test_volumes.py@199
08:19:40 bauzas stephenfin: fwiw, left comments on https://review.opendev.org/#/c/708436/ but I'm mostly +2
08:20:34 gibi brinzhang_: two nits actually

Earlier   Later