Earlier  
Posted Nick Remark
#openstack-nova - 2018-01-19
23:28:28 openstackgerrit Eric Fried proposed openstack/nova master: Fix nits in update_provider_tree series https://review.openstack.org/531260
23:28:28 openstackgerrit Eric Fried proposed openstack/nova master: WIP: Use update_provider_tree from resource tracker https://review.openstack.org/520246
23:28:29 openstackgerrit Eric Fried proposed openstack/nova master: Move refresh time from report client to prov tree https://review.openstack.org/535517
23:50:14 Spazmotic Hrm.
23:50:40 fried_rice Do tell
23:50:49 Spazmotic Just reading over what leaky said
23:51:07 fried_rice Yeah, I still think there's a hole there.
23:51:16 fried_rice I meant to put it on the PTG agenda, but I think I forgot.
23:51:20 fried_rice ...
23:51:59 Spazmotic https://github.com/openstack/nova/blob/master/nova/api/openstack/placement/handlers/allocation.py#L335-L343
23:52:24 Spazmotic This is what I see.. It's if the set allocations POST call doesn't receive any data, it pulls all the consumers allocations that exist and runs allocation.used = 0
23:52:28 fried_rice Spazmotic You going to be at the PTG?
23:52:44 Spazmotic Nah i'll be in Korea and honestly i think i'm too new to the scene to cause a fuss :D
23:53:08 Spazmotic I will re-read the code tonight, maybe I just don't understand it properly
23:55:33 fried_rice Spazmotic Oh, and the other thing was a way to add/update/delete individual pieces of allocation rather than the whole payload.
23:55:42 fried_rice Spazmotic I'm not sure there's really a good use case for that.
23:55:54 fried_rice ...IF we have proper concurrency management
23:56:25 fried_rice The amount of data just isn't that onerous. And you'd want to have the existing allocation before you tried sending an update anyway. If for no other reason than to have the current generation :)
23:56:48 Spazmotic Hmm. I guess I can understand that to ensure no double allocations.
23:57:59 Spazmotic I think the only use case that I could think of immediately was the ability to manage the allocations of a resource provider without having to touch multi-tenant full libraries of allocations, which just feels a bit top heavy for that ability. But without a redesign of how taht works it seems unavoidable for the immediate future.
#openstack-nova - 2018-01-20
00:00:46 Spazmotic I will head to the gym and think on it. Poor brain.
00:00:58 fried_rice Spazmotic You don't have to muck with allocations for anything other than the single consumer you're dealing with.
00:01:32 Spazmotic Well my brain was on something like data-loss scenarios for resource providers and there is no current way to clear out a resource provider and avoid the stuck allocations.
00:01:55 fried_rice Data loss like how?
00:01:57 Spazmotic For something like large cinder hosts later, or even large compute nodes, the ability to remove those resource allocations to free up the usages for the tenant
00:02:16 Spazmotic Well data_loss is a bit of a specific term, but even a compute_node going down with no intention to ever return
00:02:25 Spazmotic Or going down to be headshot and coming back up empty and fresh
00:02:45 fried_rice Okay, yes, one can imagine some kind of healing periodic task that goes through all consumers in placement and makes sure they still exist in the world. If not, delete their allocations.
00:03:17 fried_rice But there's nothing about the API, current or imagined, that's going to help with the intricacies of that scenario.
00:04:15 Spazmotic I'm not sure if a self heal would be able to detect it really, the only idea that my brain could grasp at is a manual call to empty a reserve host. The only way I could see a self heal identifying that is if you set a timeout for a RP to come back online or wipe it out, which seems dangerous :D
00:05:21 fried_rice Right you are. No universally good solution there.
00:05:58 Spazmotic Thanks for the morning discussion man.. i gotta get moving and get to the gym
00:25:41 Spazmotic changing the resource values of a single allocation.
00:25:41 Spazmotic oh.. and resizing allocations would be a nice usecase for that ability as well I think. I think right now you're correct that the amount of allocations would not be too bad to manage, but I was more thinking in a future of Placements being its own product. You could easilly start to see tenant allocations pushing 10,000+ which seems a bit heavy for every time small allocation changes would need to be made, like
00:25:53 Spazmotic whew... alright that's enough placement insanity i need some wine this weekend to make myself stop thinking hehehe
00:28:22 fried_rice Spazmotic Drinking sounds like a good plan. But I am curious to know what kind of scenario would have a *single* consumer with tens of thousands of allocations.
00:30:50 fried_rice I need to bail now too, but I'm interested in continuing this conversation. Have a good weekend.
00:57:38 alex_xu sean-k-mooney: yea, I assume granular request needs more magic sql to be done, that is hard to finish in this release. so supporting single request group first
02:08:46 openstackgerrit Brianna Poulos proposed openstack/nova master: Add trusted_image_certificates to REST API https://review.openstack.org/486204
02:08:46 openstackgerrit Brianna Poulos proposed openstack/nova master: Add trusted_certs object https://review.openstack.org/489408
02:48:02 openstackgerrit Merged openstack/nova master: Pass multiattach flag to reserve_block_device_name https://review.openstack.org/531244
02:48:10 openstackgerrit Merged openstack/nova master: ProviderTree.data => ProviderData https://review.openstack.org/533244
02:48:20 openstackgerrit Merged openstack/nova master: SchedulerReportClient.set_traits_for_provider https://review.openstack.org/532564
04:08:41 Spazmotic Will try to spend tonight learning more about semaphores for concurrency avoidance..
04:14:31 Spazmotic oslo hurts me
13:45:11 Spazmotic Busy day.. no coding today.. talk you all tomorrow
14:48:28 efried mgoddard set_traits_for_provider has merged FYI
17:44:09 openstackgerrit Michael Bayer proposed openstack/nova master: Generalize DB conf group copying https://review.openstack.org/484908
20:47:44 openstackgerrit Brianna Poulos proposed openstack/nova master: Add trusted_image_certificates to REST API https://review.openstack.org/486204
22:10:44 openstackgerrit Merged openstack/nova master: Updated from global requirements https://review.openstack.org/535919
22:27:38 openstackgerrit Merged openstack/nova master: Track associated sharing RPs in report client https://review.openstack.org/526539
#openstack-nova - 2018-01-21
06:58:31 openstackgerrit Alex Xu proposed openstack/nova master: placement: using the dict format for the allocations https://review.openstack.org/536083
07:33:41 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Fix 'all_tenants' & 'all_projects' type in api-ref https://review.openstack.org/522918
07:53:15 openstackgerrit Alex Xu proposed openstack/nova master: placement: support traits in allocation candidates API https://review.openstack.org/535642
07:53:16 openstackgerrit Alex Xu proposed openstack/nova master: placement: enable required traits from the flavor extra specs https://review.openstack.org/536085
07:53:16 openstackgerrit Alex Xu proposed openstack/nova master: placement: using the dict format for the allocations https://review.openstack.org/536083
09:17:16 mgoddard efried: good work, thanks for the heads up
09:43:05 ameeda Morning :)
11:38:54 openstackgerrit Lee Yarwood proposed openstack/nova master: WIP libvirt: Reintroduce volume based LM tests https://review.openstack.org/536105
14:28:55 Spazmotic efried, I suddenly realized where my logic was failing and got a much better grasp on the consumer situation.. was thinking of it like tenant. Feel much better about the current situation, but i'm reading over your suggested fix and munching on the idea for the concurrency issues. We can talk this week if you have time :)
16:51:44 efried Spazmotic Roger that.
17:34:11 mriedem bauzas: you were +2 on this novaclient change before a rebase https://review.openstack.org/#/c/528128/ - can you +W at your earliest convenience? gracias.
19:58:33 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add support for Windows network commands https://review.openstack.org/487405
22:39:10 openstackgerrit Lee Yarwood proposed openstack/nova master: Enable test_volume_backed_live_migration in tempest https://review.openstack.org/528104
#openstack-nova - 2018-01-22
01:33:59 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Fix 500 in test_resize_server_negative_invalid_state https://review.openstack.org/531117
01:34:22 openstackgerrit Yikun Jiang (Kero) proposed openstack/nova master: Add index(instance_uuid, updated_at) on instance_actions table https://review.openstack.org/530429
02:08:15 openstackgerrit jichenjc proposed openstack/nova-specs master: add z/VM Driver in Rocky https://review.openstack.org/536186
02:14:40 openstackgerrit int32bit proposed openstack/nova master: Fix wrong disk size for sparse flat files https://review.openstack.org/396873
02:23:14 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in delete (3) https://review.openstack.org/526557
02:34:46 openstackgerrit Ya Lian Pan proposed openstack/nova master: z/VM Driver: Spawn and destroy function of z/VM driver https://review.openstack.org/527658
02:34:46 openstackgerrit Ya Lian Pan proposed openstack/nova master: z/VM Driver: Initial change set of z/VM driver https://review.openstack.org/523387
02:52:35 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Parameter verification for servers.inc https://review.openstack.org/528201
02:52:58 openstackgerrit Takashi NATSUME proposed openstack/nova master: api-ref: Example verification for servers.inc https://review.openstack.org/529520
03:17:28 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in post https://review.openstack.org/526823
03:17:49 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in put (1) https://review.openstack.org/531258
03:18:06 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in put (2) https://review.openstack.org/531266
03:45:47 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in put (3) https://review.openstack.org/531280
04:46:51 vivsoni hi team, i am having master setup created 2 days back, 'nova net-list' command not found
04:47:09 vivsoni error: argument <subcommand>: invalid choice: u'net-list'
04:47:19 vivsoni is this command deprecated ?
04:51:36 takashin vivsoni: 'nova net-list' has been removed in Pike. See https://docs.openstack.org/releasenotes/python-novaclient/pike.html#id10 .
05:01:24 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in get https://review.openstack.org/531299
05:01:31 vivsoni takashin: thanks :)
05:02:01 jianghuaw vivsoni, now you can use "openstack network list" instead.
05:04:12 openstackgerrit Takashi NATSUME proposed openstack/python-novaclient master: Add CLI to show instance usage audit logs https://review.openstack.org/409413
05:15:48 openstackgerrit Deepak Mourya proposed openstack/nova master: Handle TZ change in iso8601 >=1.12.0 https://review.openstack.org/535700
05:20:26 vivsoni jianghuaw: ohk.. i was searching for that only. Thanks.. also it would be great if you can share link from which in future i can refer to
05:24:48 jianghuaw vivsoni, the commands are included in openstack client. See here: https://docs.openstack.org/python-openstackclient/latest/cli/command-objects/network.html
05:25:33 vivsoni jianghuaw: Thanks !!!
05:38:23 openstackgerrit Merged openstack/nova master: Fix instance evacuation with PCI devices https://review.openstack.org/484381
05:38:32 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in put (1) https://review.openstack.org/531258
06:21:34 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in put (2) https://review.openstack.org/531266
06:29:56 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in put (3) https://review.openstack.org/531280
06:30:17 openstackgerrit Takashi NATSUME proposed openstack/nova master: [placement] Add sending global request ID in get https://review.openstack.org/531299
06:32:13 openstackgerrit Takashi NATSUME proposed openstack/nova master: [cellv2] Improve getting BDMs in multiple cells https://review.openstack.org/521400
07:13:59 openstackgerrit shuangyang.qian proposed openstack/nova master: Fix the metadata re to match the Chinese https://review.openstack.org/536236
08:00:56 openstackgerrit shuangyang.qian proposed openstack/nova master: Fix the wrong user_id record in the log https://review.openstack.org/536243
08:39:54 gibi good morning nova
08:40:16 Spazmotic mornin gibi
08:41:57 Spazmotic UK Slowly waking up and my 3rd shift US butt over here just sitting over here all lonely :D

Earlier   Later