Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-05-16
20:14:35 mriedem when he was on the "burn all extension points with fire" crusade
20:16:07 dansmith yeah
20:21:08 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Don't count volume over quota when disabling compute https://review.openstack.org/568953
20:21:10 mriedem dansmith: ok here is the hack a mole WIP ^
20:24:50 dansmith commented
20:25:10 dansmith there are other things, like failing to verify an image signature
20:25:18 dansmith which probably shouldn't count
20:25:28 dansmith since you could probably make that happen externally
20:26:15 mriedem oh there are a buttload of things
20:27:07 mriedem there was one i saw earlier about hitting a compute with device tags but the virt driver doesn't support devices tags (so anything besides libvirt, hyperv and xen)
20:27:20 dansmith hrm yeah
20:27:26 mriedem multiattach is in the same boat
20:27:41 mriedem that's why we added a policy rule on the cinder side to disable being able to make those types of volumes
20:27:53 mriedem if you're purely non-libvirt
20:28:22 mriedem or hell, if you're not using any of the supported volume types for multiattach, which in queens was only 3 out of like the 400 available cinder backends
20:34:07 mriedem mnaser: how much do you hate this? https://review.openstack.org/#/c/568953/ - realizing we should still be doing a volume quota check in conductor, but that could still pass during a race
20:34:36 mriedem dansmith: you didn't pick up on the name of my villian in the commit message
20:35:23 dansmith mriedem: heh, didn't even read the commit message
20:35:35 melwitt I liked it
20:37:24 osnaya Question on openstack snapshot --> Is VMS (Virtual Machine Streaming) implemented in Openstack already? Is it used underneath when a snapshot and volume snapshot created?
20:38:05 osnaya when you take snapshot of a live VM?
20:38:37 osnaya Is VMS (virtual machine streaming) implemented in Pike release?
20:41:15 mriedem jaypipes: if you have the number to Florida Man, i think i might have found him a soul mate across the boarder https://theconcourse.deadspin.com/woman-furiously-shits-on-floor-of-tim-hortons-throws-i-1826082960#_ga=2.237274304.1540633067.1526503254-529658380.1521051074
20:42:20 mriedem osnaya: the libvirt driver supports live snapshot but you have to set a config option to enable it because it's shown to be unreliable at load with older versions of libvirt
20:43:01 mriedem osnaya: https://docs.openstack.org/nova/pike/configuration/config.html#workarounds.disable_libvirt_livesnapshot
20:43:28 mriedem bye!
20:44:56 melwitt mriedem: what do you mean by "we should still be doing a volume quota check in conductor, but that could still pass during a race"? if we had a check in conductor, it could pass during a race?
20:46:12 mriedem melwitt: yes
20:46:26 melwitt oh, because volumes are created by compute. okay
20:46:29 mriedem let's say i have volume quota of 10 (default) and spawn 10 instances in 10 separate requests
20:46:47 mriedem right, not min_count=10 in a single request
20:47:04 mriedem conductor would see usage as 0 for all 10 requests
20:47:08 melwitt yeah, I didn't know or forgot that volumes are created by compute
20:47:18 mriedem yup - we should eventually create them in conductor but...
20:47:27 mriedem just like we should create ports in conductor
20:47:48 melwitt gotcha, yeah
20:47:52 mriedem creating volumes in conductor would also get rid of the cross_az_attach up-call i think
20:48:42 melwitt I wonder why we don't do them even earlier in the api, before conductor, that would make it so we don't get ERROR state for a quota fail
20:49:08 mriedem because if cross_az_attach=False, and boot from volume with an existing volume, conductor can tell the scheduler we need a host in that az or fail; if nova is going to create the volume, we can create the volume in the az of the host that the scheduler picked
20:49:39 mriedem i used to think about creating volumes in the api, not sure why we wouldn't, we already reserve pre-existing volumes in the api
20:49:46 mriedem we don't create ports in the api though
20:49:54 mriedem i think the argument against doing it in the api is it blocks the response
20:50:02 melwitt ah, okay. wonder if we could just volume-update for the az case. but yeah, just thinking
20:50:06 dansmith mriedem: yeah and volume create can be really slow
20:50:13 mriedem volume create is a 202
20:50:26 mriedem so nova-api would volume create, get 202 and continue to conductor
20:50:28 dansmith plus you want to be able to do it closer to scheduling or when you know where the instance is going to be right?
20:50:29 mriedem if we did it in the api
20:50:36 dansmith mriedem: right you could and poll later I guess
20:50:47 mriedem yeah we'd still do the poll for available status in compute i think
20:51:04 mriedem but we could catch volume over quota and az cross attach issues much earlier
20:51:23 mriedem i think we need to do it closer to scheduling to know which az the volume and instance have to be in if cross_az_attach=False
20:52:00 melwitt yeah, I was thinking if we could do a "volume update" to set the az after a volume is created but maybe that's not a thing
20:52:17 mriedem that's not a thing
20:52:23 mriedem that would require a volume migration at least
20:52:25 melwitt guess it wouldn't be if their scheduling works like ours
20:52:27 melwitt yeah
20:52:29 mriedem right
20:53:01 mriedem i've only ever heard of i think 2 people that use cross_az_attach=True, and sam at nectar is always the one to reply to my questions about it in the ML
20:54:55 mriedem reminds me i never finished up the change for https://bugs.launchpad.net/nova/+bug/1694844
20:54:56 openstack Launchpad bug 1694844 in OpenStack Compute (nova) "Boot from volume fails when cross_az_attach=False and volume is provided to nova without an AZ for the instance" [Medium,In progress] - Assigned to Matt Riedemann (mriedem)
21:39:23 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: Resource class set (v1.7) https://review.openstack.org/514644
21:39:24 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: Usages per project and user (v1.8, v1.9) https://review.openstack.org/514646
21:39:25 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: CLI allocation candidates (v1.10) https://review.openstack.org/514647
21:39:26 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: New dict format of allocations (v1.11, v1.12) https://review.openstack.org/542819
21:39:27 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: Transactionally update allocations (v1.13) https://review.openstack.org/546674
21:39:28 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: Add nested resource providers (v1.14) https://review.openstack.org/546675
21:39:29 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: Limit allocation candidates (v1.15, v1.16) https://review.openstack.org/548043
21:39:30 openstackgerrit Matt Riedemann proposed openstack/osc-placement master: Allocation candidates parameter: required (v1.17) https://review.openstack.org/548326
21:41:10 openstackgerrit Matt Riedemann proposed openstack/nova master: Drop API compat handling for old compute error cases https://review.openstack.org/546256
21:41:25 openstackgerrit Vladyslav Drok proposed openstack/nova master: ironic: Report resources as reserved when needed https://review.openstack.org/517921
21:41:32 openstackgerrit Vladyslav Drok proposed openstack/nova master: ironic: Report resources as reserved when needed https://review.openstack.org/517921
21:54:40 openstackgerrit Eric Fried proposed openstack/nova master: Update nova-status and docs for required placement 1.24 https://review.openstack.org/568923
21:54:41 openstackgerrit Eric Fried proposed openstack/nova master: Enforce placement minimum in nova.cmd.status https://review.openstack.org/568965
21:57:57 openstackgerrit Takashi NATSUME proposed openstack/nova-specs master: Abort Cold Migration https://review.openstack.org/334732
22:38:10 openstackgerrit Merged openstack/nova master: Remove deprecated monkey_patch config options https://review.openstack.org/568880
23:30:04 melwitt naichuans: hi, fyi your blueprint vgpu-rocky has been added to a review runway today. please let us know if the next two weeks are not a good time for active review
23:40:21 openstackgerrit Takashi NATSUME proposed openstack/nova master: Fix auth_url example in hypervisor-hyper-v.rst https://review.openstack.org/568983
#openstack-nova - 2018-05-17
00:11:25 openstackgerrit David Bingham proposed openstack/nova master: Adding apply_cells to nova-manage to enable automated cells_v2 configuration. https://review.openstack.org/568987
00:15:30 openstackgerrit Eric Berglund proposed openstack/nova master: PowerVM snapshot cleanup https://review.openstack.org/568988
00:34:44 openstackgerrit David Bingham proposed openstack/nova master: Adding apply_cells to nova-manage to enable automated cells_v2 configuration. https://review.openstack.org/568987
00:35:39 openstackgerrit Takashi NATSUME proposed openstack/nova master: Adds view builders for keypairs controller https://review.openstack.org/347289
00:49:30 openstackgerrit Merged openstack/nova master: Add tests for alloc_cands with member_of https://review.openstack.org/561399
00:52:45 naichuans melwitt: It OK for me to have a review. Thanks you very much
00:55:55 openstackgerrit Merged openstack/nova master: Fix member_of with sharing providers https://review.openstack.org/561400
00:56:01 openstackgerrit Merged openstack/nova master: Expand member_of functional test cases https://review.openstack.org/566011
00:56:57 melwitt naichuans: cool, thanks
00:57:14 naichuans Np
01:08:22 openstackgerrit Nakanishi Tomotaka proposed openstack/nova master: Test Compute API in multiple cells https://review.openstack.org/534116
01:20:49 openstackgerrit Naichuan Sun proposed openstack/nova master: xenapi(N-R-P): support compute node resource provider update https://review.openstack.org/521041
01:23:38 openstackgerrit Naichuan Sun proposed openstack/nova master: xenapi(N-R-P): support compute node resource provider update https://review.openstack.org/521041
01:25:26 openstackgerrit Naichuan Sun proposed openstack/nova master: xenapi(N-R-P): support compute node resource provider update https://review.openstack.org/521041
01:40:50 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Return nested providers in get_by_request https://review.openstack.org/567113
01:40:51 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Add traits check in nested provider candidates https://review.openstack.org/567150
01:40:52 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Support nested alloc cands with sharing providers https://review.openstack.org/567508
01:40:53 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Return all resources in provider_summaries https://review.openstack.org/558045
01:40:54 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Return all nested providers in tree https://review.openstack.org/559480
01:40:55 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Add microversion for nested allocation candidate https://review.openstack.org/565487
01:56:55 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Add tests for _get_trees_matching_all() function https://review.openstack.org/568737
01:56:56 openstackgerrit Tetsuro Nakamura proposed openstack/nova master: Return nested providers in get_by_request https://review.openstack.org/567113

Earlier   Later