Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-06-22
18:50:45 artom I suppose
18:51:00 dansmith no?
18:51:09 artom It wouldn't hurt
18:51:14 dansmith this strikes me as a thing you're going to want to have broad agreement on
18:51:25 artom I feel like having concrete code to strawman might be better
18:51:48 dansmith well, as long as you're okay with throwing it away, and make it clear it's for discussion then that's fine with me
18:52:30 artom Heh, well, maybe I'm not?
18:52:48 dansmith ...
18:53:03 dansmith well, just trying to help.. do whichever you think will be most successful :)
18:53:50 artom dansmith, right, sorry, didn't mean to come off as flippant
18:54:03 artom This discussion is definitely helpful and your time is appreciated :)
18:54:32 dansmith for me as well
18:54:47 dansmith now I have all my -1s lined up and it's not even noon :P
19:27:14 smcginnis I seem to recall hearing at one point that the version of libvirt-python used was dependent on the version of libvirt itself on the host. Can anyone confirm that?
20:14:19 openstackgerrit Merged openstack/nova stable/queens: [Stable Only] Add amd-ssbd and amd-no-ssb CPU flags https://review.openstack.org/576270
20:14:27 openstackgerrit Merged openstack/nova master: Fix bug to api-ref https://review.openstack.org/571375
20:24:49 openstackgerrit Dan Smith proposed openstack/nova stable/pike: [Stable Only] Add amd-ssbd and amd-no-ssb CPU flags https://review.openstack.org/577548
21:28:58 openstackgerrit Merged openstack/nova master: Fix nits from change Ia7cf4414feb335b3c2e863b4c8b4ff559b275c34 https://review.openstack.org/577314
21:41:16 openstackgerrit Merged openstack/nova master: libvirt: Log breadcrumb for known encryption bug https://review.openstack.org/577164
#openstack-nova - 2018-06-23
08:13:56 openstackgerrit jiang wei proposed openstack/nova master: Add action initiator attribute to the instance payload https://review.openstack.org/536243
14:10:37 openstackgerrit Merged openstack/nova master: Fix bug to filter_scheduler https://review.openstack.org/571998
#openstack-nova - 2018-06-24
01:52:57 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Use ThreadPoolExecutor for max_concurrent_live_migrations https://review.openstack.org/563505
02:42:17 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Use ThreadPoolExecutor for max_concurrent_live_migrations https://review.openstack.org/563505
16:44:30 openstackgerrit Lei Zhang proposed openstack/nova master: Add method to get cpu traits https://review.openstack.org/560317
#openstack-nova - 2018-06-25
02:20:16 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Change the anti-affinity Filter to adapt to new policy https://review.openstack.org/571166
02:20:17 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Adapt _validate_instance_group_policy to new policy model https://review.openstack.org/571465
02:20:18 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Microversion 2.64 - Use new format policy in server group https://review.openstack.org/567534
02:21:29 openstackgerrit Vu Cong Tuan proposed openstack/os-vif master: Add py36 testenv https://review.openstack.org/577686
02:59:53 openstackgerrit Merged openstack/nova stable/queens: Add policy rule to block image-backed servers with 0 root disk flavor https://review.openstack.org/563692
02:59:58 openstackgerrit Merged openstack/nova stable/pike: Fixed auto-convergence option name in doc https://review.openstack.org/576282
04:13:14 openstackgerrit Steve Kowalik proposed openstack/nova master: Switch to oslo_messaging.ConfFixture.transport_url https://review.openstack.org/573046
04:53:21 openstackgerrit Merged openstack/nova stable/pike: [Stable Only] Add amd-ssbd and amd-no-ssb CPU flags https://review.openstack.org/577548
05:11:35 openstackgerrit Hironori Shiina proposed openstack/nova master: Ironic: Check a last error of power action https://review.openstack.org/470754
05:55:34 openstackgerrit Deepak Mourya proposed openstack/nova master: Handle CannotDeleteParentResourceProvider to 409 Conflict https://review.openstack.org/577726
06:03:13 openstackgerrit Deepak Mourya proposed openstack/nova master: Handle CannotDeleteParentResourceProvider to 409 Conflict https://review.openstack.org/577726
06:12:32 gibi good morning nova
06:23:18 openstackgerrit Merged openstack/nova stable/pike: Change consecutive build failure limit to a weigher https://review.openstack.org/573248
07:12:58 openstackgerrit Chason Chan proposed openstack/nova master: [WIP] Fix the duplicated config options of api_database and placement_database https://review.openstack.org/577023
07:13:39 openstackgerrit Gergely Csatari proposed openstack/nova master: Adding oslo.service to doc requirements https://review.openstack.org/577095
07:29:46 openstackgerrit Deepak Mourya proposed openstack/nova master: Handle CannotDeleteParentResourceProvider to 409 Conflict https://review.openstack.org/577726
07:38:44 openstackgerrit Hironori Shiina proposed openstack/nova master: ironic: Proxy a few attributes of block device mapping https://review.openstack.org/554093
09:00:06 openstackgerrit Zhenyu Zheng proposed openstack/nova master: Compute: add support to abort queued live migration https://review.openstack.org/568542
09:11:31 openstackgerrit Lee Yarwood proposed openstack/nova stable/queens: libvirt: Log breadcrumb for known encryption bug https://review.openstack.org/577757
09:23:11 bauzas gibi: stephenfin: easy fix for reviews that'd help operators https://review.openstack.org/#/c/577424/
09:36:35 openstackgerrit Surya Seetharaman proposed openstack/nova master: Make nova list and migration-list ignore down cells https://review.openstack.org/575734
09:39:31 gibi bauzas: looking
10:41:36 openstackgerrit Chris Dent proposed openstack/nova master: Test alloc_cands with indirectly sharing RPs https://review.openstack.org/519601
11:00:42 openstackgerrit sahid proposed openstack/nova master: hardware: fix memory check when hugepages requested https://review.openstack.org/532168
12:09:28 openstackgerrit Kashyap Chamarthy proposed openstack/nova master: conf: libvirt: Make `/dev/urandom` the default for 'rng_dev_path' https://review.openstack.org/577385
12:28:59 mriedem lyarwood: please kind sir https://review.openstack.org/#/c/571320/
12:32:11 openstackgerrit Chason Chan proposed openstack/nova master: Fix the duplicated config options of api_database and placement_database https://review.openstack.org/577023
12:47:57 openstackgerrit Deepak Mourya proposed openstack/nova master: Handle CannotDeleteParentResourceProvider to 409 Conflict https://review.openstack.org/577726
12:49:51 lyarwood mriedem: ack, would you mind taking a look at this today if you have time - https://review.openstack.org/#/q/topic:bug/1764883+status:open
12:51:38 mriedem oh boy i think dan was looking at that last week
12:53:26 lyarwood mriedem: yup, addressed his nits but I think he still had concerns about how useful this actually is, if you agree I'll abandon and move on.
12:56:42 efried ō/
12:58:11 mriedem lyarwood: you're referring to this? https://review.openstack.org/#/c/562284/3/nova/compute/manager.py@1133
12:59:37 lyarwood mriedem: yeah, L1133
13:01:41 mriedem "For one thing, we'll fail to find the migration in the evacuation code in conductor" - why?
13:02:48 mriedem i must be missing something
13:03:57 openstackgerrit Surya Seetharaman proposed openstack/nova master: Make nova list and migration-list ignore down cells https://review.openstack.org/575734
13:05:14 openstackgerrit Deepak Mourya proposed openstack/nova master: Handle CannotDeleteParentResourceProvider to 409 Conflict https://review.openstack.org/577726
13:10:22 kashyap stephenfin: Is someone already converting away more of those config help texts into multi-line strings? The single-quotes thing is super tiring.
13:10:48 kashyap (It's somewhat 'noisy' to do those, but clearly useful in the long run.)
13:10:48 stephenfin kashyap: What's this is relation to?
13:11:07 kashyap stephenfin: Err, sorry, was assuming context. This one: https://review.openstack.org/#/c/577385/
13:11:51 kashyap In relation to the comment you made, which I addressed. But there are lot of other configs that don't use the multi-line string approach.
13:11:57 stephenfin No one is bulk converting them, no. However, changing them as we go (as you've done there) seems like a viable option to me
13:12:10 kashyap Yeah, I think that's the best too.
13:12:49 stephenfin I'm pretty sure mriedem, supreme hater of "noise" patches, would prefer that approach too :P
13:13:05 kashyap Yeah, I wasn't vonluteering to produce that noise either :-)
13:13:07 mriedem ``noise``
13:13:25 mriedem lyarwood: comments on those patches
13:13:28 mriedem for the evac thing
13:13:31 lyarwood mriedem: thanks
13:14:04 sahid melwitt, mriedem, do you think we could consider https://review.openstack.org/#/c/553072/ for stable?
13:17:53 mriedem sahid: i wouldn't block it, even though it's enabling something that wasn't on stable for certain types of vifs, but i guess that could be argued as a bug
13:18:09 mriedem might need a release note
13:33:51 openstackgerrit Merged openstack/nova stable/queens: Be graceful about vif plugging in early ironic driver startup https://review.openstack.org/576948
14:39:39 openstackgerrit Merged openstack/nova stable/pike: Ensure resource class cache when listing usages https://review.openstack.org/574327
15:05:25 bauzas dansmith: mriedem: cdent: efried: I have a question about allocations for a resize/cold migration
15:05:47 bauzas dansmith: mriedem: cdent: efried: for the moment, we don't pass the allocations to the driver when calling resize
15:05:58 bauzas but we do it for rebuild
15:06:46 bauzas so, for the moment, we have a problem with vGPUs like said in https://docs.openstack.org/nova/latest/admin/virtual-gpu.html#caveats
15:07:36 mriedem so pass allocations to the driver on resize
15:07:51 efried yeah, that seems like it was probably just an oversight.
15:08:01 bauzas efried: so, my question is : can I call self.reportclient.get_allocations_for_consumer() for getting all the allocations, even the migration one ?
15:08:09 dansmith I assume anything like this will eventually need allocations yeah
15:08:28 mriedem bauzas: the allocations for the source host moved to the migration object
15:08:31 efried bauzas: As long as you know the migration UUID, yeah.
15:08:34 mriedem you want the allocations for the dest host, which are on the instance
15:09:15 mriedem are you properly freeing vgpus from the source host when confirm_resize is called?
15:09:19 bauzas mriedem: so, tbc, when calling self.reportclient.get_allocations_for_consumer() in the compute service for resize_instance(), it would give me the migration UUID allocation ?
15:09:35 mriedem bauzas: it depends on what you need and where you call it
15:09:42 bauzas I see
15:09:50 mriedem if you're on the dest host and need the vgpus for some kind of claim on the dest host, you want to use the instance uuid
15:10:00 bauzas if I'm asking for the migration UUID, it would give me the related alloc
15:10:12 mriedem https://specs.openstack.org/openstack/nova-specs/specs/queens/implemented/migration-allocations.html
15:10:22 mriedem if you're asking for the migration uuid, you get the source host allocations
15:10:22 bauzas if I'm asking for the instance UUID, it's giving me the instance alloc

Earlier   Later