Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2020-03-24
14:08:16 dansmith alex_xu: good job catching that
14:10:12 openstackgerrit John Garbutt proposed openstack/nova master: Add logic to enforce local api and db limits https://review.opendev.org/712139
14:10:13 openstackgerrit John Garbutt proposed openstack/nova master: Enforce api and db limits https://review.opendev.org/712142
14:11:14 openstackgerrit Artom Lifshitz proposed openstack/nova master: func tests: move _run_periodics() into base class https://review.opendev.org/714228
14:11:15 openstackgerrit Artom Lifshitz proposed openstack/nova master: Functional tests for NUMA live migration https://review.opendev.org/672595
14:11:15 openstackgerrit Artom Lifshitz proposed openstack/nova master: tests: work around malformed serial XML https://review.opendev.org/705546
14:11:16 openstackgerrit Artom Lifshitz proposed openstack/nova master: NUMA LM: Add func test for bug 1845146 https://review.opendev.org/687404
14:11:17 openstack bug 1845146 in OpenStack Compute (nova) train "NUMA aware live migration failed when vCPU pin set" [High,Fix committed] https://launchpad.net/bugs/1845146 - Assigned to Dan Smith (danms)
14:11:23 artom dansmith, ^^
14:14:14 artom \o/
14:20:58 openstackgerrit Luyao Zhong proposed openstack/nova master: bug-fix: set do_cleanup always True for libvirt driver https://review.opendev.org/714593
14:20:58 nightmare_unreal gibi: what if user pass both instance-id and cell-id
14:20:59 openstackgerrit Luyao Zhong proposed openstack/nova master: Track orphan instances and error migrations in resource tracker https://review.opendev.org/714653
14:20:59 openstackgerrit Luyao Zhong proposed openstack/nova master: support live migration with vpmems https://review.opendev.org/687856
14:30:01 openstackgerrit Huaqiang Wang proposed openstack/nova master: 'cpu_pinning_requested' property could be derived directly from cpu_policy https://review.opendev.org/713353
14:30:01 openstackgerrit Huaqiang Wang proposed openstack/nova master: Take the instance dedicated CPU list from 'cpu_pinning' if possible https://review.opendev.org/713352
14:30:02 openstackgerrit Huaqiang Wang proposed openstack/nova master: Introduce the interface of creating 'MIXED' policy instance through 'PCPU' and 'VCPU' https://review.opendev.org/713355
14:30:02 openstackgerrit Huaqiang Wang proposed openstack/nova master: Introduce 'MIXED' CPU allocation policy for instance https://review.opendev.org/713354
14:30:03 openstackgerrit Huaqiang Wang proposed openstack/nova master: libvirt: support to create instance with dedicated and shared CPUs https://review.opendev.org/714655
14:30:03 openstackgerrit Huaqiang Wang proposed openstack/nova master: metadata: export the vCPU IDs that are pinning on the host CPUs https://review.opendev.org/688936
14:30:04 openstackgerrit Huaqiang Wang proposed openstack/nova master: Refactor the code in checking available host CPUs https://review.opendev.org/714657
14:30:04 openstackgerrit Huaqiang Wang proposed openstack/nova master: Remove 'InstanceNUMACell.cpu_pinning_requested' field https://review.opendev.org/714656
14:30:05 openstackgerrit Huaqiang Wang proposed openstack/nova master: [WIP] An alternative way for keeping instance dedicated CPUs https://review.opendev.org/714658
14:37:25 openstackgerrit Stephen Finucane proposed openstack/nova master: libvirt: Use domain capabilities to get supported device models https://review.opendev.org/666915
14:37:26 openstackgerrit Stephen Finucane proposed openstack/nova master: Add transform_image_metadata request filter https://review.opendev.org/665775
14:40:40 openstackgerrit Kevin Zhao proposed openstack/nova master: Add default cpu model for aarch64 https://review.opendev.org/709494
14:47:11 openstackgerrit Kevin Zhao proposed openstack/nova master: Add default cpu model for aarch64 https://review.opendev.org/709494
14:51:34 gibi nightmare_unreal: good point. You can make the two filters (instance_uuid and cell_id) mutually exclusive
14:52:06 gibi I don't see any reason why the user want to filter for both at the same time
14:52:07 nightmare_unreal yup I was thinking something like that in for loop
14:53:39 nightmare_unreal gibi: I am not sure but can we add/append to cells ? for e.g. if instance_id is provided cells variable will have cell_id of that instance and along with cell_id is also passed, can we just append it in cells. it can make code more simple
14:56:06 huaqiang hi stephenfin，in my new patches, I haven't add a 'cpu_policy' property to 'InstanceNUMATopology' object.
14:56:06 huaqiang hi stephenfin，in my new patches, I haven't add a 'cpu_policy' property to 'InstanceNUMATopology' object.
14:57:21 huaqiang If I do, I have to replace every 'topology.cpu_pinning_requested' line with 'if topology.cpu_policy in "DEDICATED or MIXED"'
14:58:11 huaqiang I think 'topology.cpu_pinning_requested' and 'topology.cpu_sharing_requested' are more concise.
14:59:40 gibi nightmare_unreal: there is not much reason to append to the cells variable if both instance_uuid and cell_id is provided. Because today if the instance_uuid is passed the it means that heal _only_ that instance.
14:59:58 gibi so if you just append the extra cell specified by the cell_id then you still only heal one instance
15:00:12 gibi not one instance + a whole other cell
15:00:21 nightmare_unreal ah yes
15:08:19 stephenfin huaqiang: I don't think that replacement is that much work (I've it done locally and it's quite concise). However, adding 'cpu_policy' to the 'InstanceNUMATopology' object is a lot of work though that can wait, yes.
15:10:15 sean-k-mooney we can infer the policy from the cpu pinning info in anycase
15:11:19 sean-k-mooney the cpu_policy is also somewhat incorrectly stored in the insance numa cell https://github.com/openstack/nova/blob/master/nova/objects/instance_numa.py#L59-L60
15:12:11 gibi stephenfin: I'm +2 on the image-metadata-prefiltering
15:13:00 sean-k-mooney it should be in the InstanceNUMATopology instead bug you could add a property that would return the value form the first cell and in a later version add a cpu_policy field and remove it form teh instance numa cell
15:13:04 stephenfin sean-k-mooney: yeah, I'd a suggestion to duplicate it to the topology and slowly deprecate use of the cell field, but it's not really related to this
15:13:38 huaqiang stephenfin: nice!
15:14:05 sean-k-mooney right its not related to this but we shoudl eventually move it
15:15:52 huaqiang stephenfin, can you share some detail? or you can submit the changes
15:16:05 huaqiang then I do the work under your changes
15:16:14 stephenfin for the replacement?
15:16:19 huaqiang yes
15:16:24 stephenfin sure
15:17:07 sean-k-mooney by the way if you want to start using the value form the isntanceNUMATopology without bumping the verion fo the object you can add this
15:17:10 sean-k-mooney @property
15:17:12 sean-k-mooney def cpu_policy(self):
15:17:14 sean-k-mooney return self.cells[0].cpu_policy if self.cells else "shared"
15:17:37 stephenfin sean-k-mooney: that's what I have
15:17:54 openstackgerrit Stephen Finucane proposed openstack/nova master: Remove future imports https://review.opendev.org/714675
15:17:55 openstackgerrit Stephen Finucane proposed openstack/nova master: Use unittest.mock instead of third party mock https://review.opendev.org/714676
15:17:56 stephenfin albeit with a check to make sure all the cells have the same value and error out if not (it shouldn't ever be triggered)
15:18:04 huaqiang sean-k-mooney,stephenfin: thanks
15:18:06 sean-k-mooney cool
15:18:20 sean-k-mooney stephenfin: ya so that cant happen today unless you messed with the db
15:18:26 sean-k-mooney so i did not bother with the check
15:22:26 openstackgerrit jayaditya gupta proposed openstack/nova master: Support for nova-manage placement heal_allocations --cell https://review.opendev.org/714459
15:22:27 sean-k-mooney ok ill be afk for about an hour or so. bbl
15:31:03 gibi lyarwood, stephenfin: is this a bugfix or a feature? https://review.opendev.org/#/c/696834
15:32:19 stephenfin gibi: It's probably closer to a feature. We couldn't do this before and now we can
15:32:57 stephenfin Should there be a specless blueprint? I didn't think of that
15:34:44 stephenfin alex_xu: Think you could look at this? https://review.opendev.org/#/c/666915/
15:37:24 gibi stephenfin: would be nice to track it somehow. I'm OK to have a specless bp as the impl basically done and only affects the libvirt driver
15:37:42 gibi this is a bit of paperwork I know
15:39:09 openstackgerrit Luigi Toscano proposed openstack/nova stable/pike: Remove exp legacy-tempest-dsvm-full-devstack-plugin-nfs https://review.opendev.org/702061
15:50:42 openstackgerrit Merged openstack/nova-specs master: Move 'live migration with vpmems' spec https://review.opendev.org/708229
16:08:16 tosky oh, the last nova-live-migration fix is finally fine on stable/pike: it just needs another +2 (and a +w)! https://review.opendev.org/#/c/713036/
16:20:27 openstackgerrit Merged openstack/nova master: Enforce os-traits/SUPPORTED_STORAGE_BUSES sync https://review.opendev.org/709233
16:27:23 openstackgerrit John Garbutt proposed openstack/nova master: Update quota_class APIs for db and api limits https://review.opendev.org/712143
16:36:20 dansmith kinda seems like the openstack sdk test is failing a lot
16:36:28 dansmith I didn't see that mentioned on the recent gate bug thread
16:37:03 dansmith it has failed a couple times in a row with a bunch of missing packages
16:38:09 dansmith ah, known mirror failure which might be fixed now
16:43:48 openstackgerrit Stephen Finucane proposed openstack/nova master: hardware: Tweak the 'cpu_realtime_mask' handling slightly https://review.opendev.org/461456
16:43:48 openstackgerrit Stephen Finucane proposed openstack/nova master: hardware: Add validation for 'cpu_realtime_mask' https://review.opendev.org/468203
16:43:49 openstackgerrit Stephen Finucane proposed openstack/nova master: libvirt: Add typing information https://review.opendev.org/714695
16:43:49 openstackgerrit Stephen Finucane proposed openstack/nova master: hardware: Update and correct typing information https://review.opendev.org/714694
16:43:50 openstackgerrit Stephen Finucane proposed openstack/nova master: objects: Replace 'cpu_pinning_requested' helper https://review.opendev.org/714697
16:43:50 openstackgerrit Stephen Finucane proposed openstack/nova master: tests: Split instance NUMA object tests https://review.opendev.org/714696
16:43:51 openstackgerrit Stephen Finucane proposed openstack/nova master: hardware: Remove handling of pre-Train compute nodes https://review.opendev.org/714699
16:43:51 openstackgerrit Stephen Finucane proposed openstack/nova master: hardware: Don't consider overhead CPUs for unpinned instances https://review.opendev.org/714698
16:43:52 openstackgerrit Stephen Finucane proposed openstack/nova master: hardware: Invert order of NUMA topology generation https://review.opendev.org/714701
16:43:52 openstackgerrit Stephen Finucane proposed openstack/nova master: hardware: Rework 'get_realtime_constraint' https://review.opendev.org/714700
16:43:53 openstackgerrit Stephen Finucane proposed openstack/nova master: hardware: Remove '_numa_fit_instance_cell_with_pinning' https://review.opendev.org/714703
16:43:53 openstackgerrit Stephen Finucane proposed openstack/nova master: hardware: 'numa_fit_instance_to_host' requires 'limits' argument https://review.opendev.org/714702
16:43:54 stephenfin huaqiang: ^
16:43:54 openstackgerrit Stephen Finucane proposed openstack/nova master: WIP! objects: Add 'InstanceNUMACell.pcpuset' field https://review.opendev.org/714705
16:43:54 openstackgerrit Stephen Finucane proposed openstack/nova master: WIP! objects: Add 'CPUAllocationPolicy.MIXED' enum https://review.opendev.org/714704
16:43:55 openstackgerrit Stephen Finucane proposed openstack/nova master: WIP! hardware: Start configuring 'InstanceNUMACell.pcpuset' https://review.opendev.org/714707
16:43:55 openstackgerrit Stephen Finucane proposed openstack/nova master: WIP! hardware: Add support for 'hw:cpu_dedicated_mask' extra spec https://review.opendev.org/714706
16:53:48 openstackgerrit John Garbutt proposed openstack/nova master: Update limit APIs https://review.opendev.org/712707
16:53:49 openstackgerrit John Garbutt proposed openstack/nova master: Update quota sets APIs https://review.opendev.org/712749

Earlier   Later