Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-10-10
16:20:52 sean-k-mooney im not sure if its intended or not as i only noticed it happening when the instance goes to error state and i delete it
16:22:21 mriedem error state during scheduling?
16:22:24 sean-k-mooney mriedem: well tempest is ment to work across releases so if this changes across release it an api change
16:22:24 mriedem or during building on the host?
16:22:48 mriedem sean-k-mooney: well, in this case, tempest is requesting to detach a volume that is already detached
16:22:52 mriedem it's part of a cleanup routine
16:22:55 sean-k-mooney mriedem: durign building basically when the instance fails to boot due to port binding failed
16:23:02 mriedem arguably the client shouldn't be making that request
16:23:39 sean-k-mooney mriedem: ah ok ya that sound like a client bug as the precondition is not ment. e.g. it must be attached to detach it
16:23:41 mriedem sean-k-mooney: are you sure you're setting delete_on_termination=True for the bdm?
16:24:41 openstackgerrit Jackie Truong proposed openstack/nova master: Implement certificate_utils https://review.openstack.org/479949
16:24:48 efried mriedem Should removal of cinder v2 support also have removed volumev2 (and maybe also volume) from https://github.com/openstack/nova/blob/master/nova/context.py#L121 ?
16:24:50 sean-k-mooney i was booting from horizon and setting the Delete volumne on onstance delete option
16:25:01 sean-k-mooney so i assume that set it under the hood
16:25:58 mriedem efried: good point, probably yeah
16:26:05 mriedem did we remove volumev2 support already?
16:26:08 mriedem i think i pushed the change for it...
16:26:11 efried yeah
16:26:16 efried swhy I tagged ya
16:26:41 mriedem ah yeah https://github.com/openstack/nova/commit/eadbacbda628ecc969a980378faf55bc02f514bf#diff-d24869e36361bb430acd67658a83b0b5
16:26:47 mriedem yeah we should remove volume and volumev2 from context
16:27:11 efried mriedem But include block-storage
16:27:23 efried since that's the official service type name
16:27:38 efried though I don't think devstack is setting that up yet.
16:29:07 mriedem lyarwood: interesting fallout bug https://bugs.launchpad.net/tempest/+bug/1722577/comments/3
16:29:08 openstack Launchpad bug 1722577 in OpenStack Compute (nova) "test_list_get_volume_attachments failing with 400 error on teardown when detaching an already detached volume" [Undecided,Triaged]
16:29:09 mriedem mtreinish: ^
16:29:24 mriedem mtreinish: i'm inclined to fix ^ in tempest
16:30:42 mriedem lyarwood: result of this change https://review.openstack.org/#/c/440693/
16:45:32 lyarwood mriedem: huh, nice find, I guess I missed that detach_volume is cast at the compute layer so anything following the detach api call could race
16:45:43 lyarwood is a cast*
16:46:32 mriedem yeah, seemed innocent enough. i'm working a tempest patch.
16:46:50 mriedem cinder team is also considering just not returning a 400 if you ask to detach an already detached volume
16:47:54 sean-k-mooney mriedem: would they retrun a 200 instead in that case?
16:48:18 mriedem don't know
16:49:18 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add Virtual Ethernet device pair https://review.openstack.org/484726
16:49:59 openstackgerrit Balazs Gibizer proposed openstack/nova master: fix cleaning up evacuated instances https://review.openstack.org/510938
16:50:17 cfriesen sean-k-mooney: I think their normal success code is 202
16:51:09 sean-k-mooney cfriesen: 202 accepted would also be fine it they match the retrun code that would be retruned if it did detach then that would be fine
16:52:18 openstackgerrit Rodolfo Alonso Hernandez proposed openstack/os-vif master: Add Open vSwitch patch port https://review.openstack.org/485228
16:52:39 mriedem the cinder detach api returns a 202 even though it's a synchronous api
16:52:47 mriedem it's an rpc call to the volume backend
16:52:48 openstackgerrit Balazs Gibizer proposed openstack/nova master: Reproduce bug 1721652 in the functional test env https://review.openstack.org/510176
16:52:49 openstackgerrit Balazs Gibizer proposed openstack/nova master: fix cleaning up evacuated instances https://review.openstack.org/510938
16:52:49 openstack bug 1721652 in OpenStack Compute (nova) "Evacuate cleanup fails at _delete_allocation_for_moved_instance" [High,In progress] https://launchpad.net/bugs/1721652 - Assigned to Balazs Gibizer (balazs-gibizer)
16:56:38 openstackgerrit Brianna Poulos proposed openstack/nova master: Add trusted_certs object https://review.openstack.org/489408
16:56:39 openstackgerrit Brianna Poulos proposed openstack/nova master: Add trusted_image_certificates to REST API https://review.openstack.org/486204
17:03:40 openstackgerrit Brianna Poulos proposed openstack/nova master: Implement certificate_utils https://review.openstack.org/479949
17:04:44 openstackgerrit Eric Fried proposed openstack/nova master: Update cinder in RequestContext service catalog https://review.openstack.org/510947
17:04:56 efried mriedem mordred ^
17:09:03 openstackgerrit Dan Smith proposed openstack/nova master: Refactor resource tracker to account for migration allocations https://review.openstack.org/506419
17:09:03 openstackgerrit Dan Smith proposed openstack/nova master: Revert allocations by migration uuid https://review.openstack.org/498949
17:09:04 openstackgerrit Dan Smith proposed openstack/nova master: Make live migration hold resources with a migration allocation https://review.openstack.org/507638
17:09:04 openstackgerrit Dan Smith proposed openstack/nova master: Make migration uuid hold allocations for migrating instances https://review.openstack.org/506420
17:09:17 openstackgerrit Jay Pipes proposed openstack/nova master: rp: streamline InventoryList.get_all_by_rp_uuid() https://review.openstack.org/509030
17:09:18 openstackgerrit Jay Pipes proposed openstack/nova master: rp: remove ability to delete 1 allocation record https://review.openstack.org/509032
17:09:18 openstackgerrit Jay Pipes proposed openstack/nova master: rp: remove dead code in Allocation._create_in_db() https://review.openstack.org/509031
17:09:19 openstackgerrit Jay Pipes proposed openstack/nova master: rp: rework AllocList.get_all_by_consumer_id() https://review.openstack.org/509035
17:09:19 openstackgerrit Jay Pipes proposed openstack/nova master: rp: fix up AllocList.get_by_resource_provider_uuid https://review.openstack.org/509033
17:09:20 openstackgerrit Jay Pipes proposed openstack/nova master: placement: integrate ProviderTree to report client https://review.openstack.org/415921
17:09:20 openstackgerrit Jay Pipes proposed openstack/nova master: placement: set/check if inventory change in tree https://review.openstack.org/470575
17:09:21 openstackgerrit Jay Pipes proposed openstack/nova master: placement: allow filter providers in tree https://review.openstack.org/377215
17:09:21 openstackgerrit Jay Pipes proposed openstack/nova master: placement: add nested resource providers https://review.openstack.org/377138
17:09:22 openstackgerrit Jay Pipes proposed openstack/nova master: placement: update client to set parent provider https://review.openstack.org/385693
17:09:22 openstackgerrit Jay Pipes proposed openstack/nova master: placement: adds REST API for nested providers https://review.openstack.org/384807
17:09:58 jaypipes dansmith, mriedem, efried: k, n-r-p series updated... ^
17:10:11 efried woot
17:13:51 gibi mriedem: I just noticed that indeed there is one notification transformation patch that already has a +2 https://review.openstack.org/#/c/453077/
17:14:15 gibi mriedem: you were even +2 before on that
17:14:29 dansmith mriedem: I'm getting pretty fatigued on this one, so please apply extra critical thinking when reviewing: https://review.openstack.org/#/c/507638/
17:14:36 openstackgerrit Chris Dent proposed openstack/nova master: [placement] manage cache headers for inventories https://review.openstack.org/510954
17:14:36 openstackgerrit Chris Dent proposed openstack/nova master: [placement] manage cache headers for /resource_providers https://review.openstack.org/495380
17:14:49 dansmith I pushed it up to see if it really passes live migration tests and then I might try to think on some more cleanup of it
17:15:13 mriedem easy regression fix needed for pike https://review.openstack.org/#/c/510603/
17:15:47 mriedem gibi: ok
17:15:48 mriedem dansmith: ok
17:15:51 mriedem everyone: ok
17:21:11 openstackgerrit Matt Riedemann proposed openstack/nova stable/pike: Fix sending legacy instance.update notification https://review.openstack.org/510957
17:26:47 mriedem weeee we stacktrace during snapshot a whole bunch https://bugs.launchpad.net/nova/+bug/1722571
17:26:48 openstack Launchpad bug 1722571 in OpenStack Compute (nova) "NotImplementedError(_('direct_snapshot() is not implemented')) stacktraces in n-cpu logs" [Medium,Confirmed] - Assigned to hongbin (hongbin034)
17:28:06 hongbin mriedem: i am looking into it (try to fix some bugs to pick up)
18:02:48 mriedem hongbin: thanks, should be pretty easy
18:06:17 openstackgerrit Andrey Volkov proposed openstack/osc-placement master: CLI for inventories https://review.openstack.org/457533
18:06:18 openstackgerrit Andrey Volkov proposed openstack/osc-placement master: CLI for usages https://review.openstack.org/457535
18:06:18 openstackgerrit Andrey Volkov proposed openstack/osc-placement master: CLI for allocations https://review.openstack.org/457534
18:06:19 openstackgerrit Andrey Volkov proposed openstack/osc-placement master: CLI for aggregates https://review.openstack.org/505643
18:18:33 mriedem sdague: i just realized that when showing server details, we provide a link to view image details, and the link for the image details is to the compute api images proxy api, which is deprecated starting in 2.36 :)
18:19:30 mriedem https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/views/servers.py#L254
18:20:04 mriedem seems like if the microversion is >= 3.26 we should generate the alternate link to the images api
18:20:20 mriedem https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/views/images.py#L133
18:22:05 mriedem ooo, and with efried's new service type stuff, if you don't set CONF.glance.api_servers, we'd actually generate maybe a valid images api url for you
18:31:20 sdague mriedem: honestly, I'm not sure we should be generating an image link
18:31:28 sdague I thought we had gotten rid of that
18:31:48 mriedem are you thinking of the location header in the response when creating a snapshot or backup image?
18:32:10 mriedem https://docs.openstack.org/nova/pike/reference/api-microversion-history.html#id40
18:32:46 mriedem yeah we probably should have used that microversion to also stop returning image api url links
18:33:07 sdague I actually misremembered that we did
18:33:16 sdague maybe we stopped accepting image url somewhere
18:33:29 mriedem i remember gmann fixing something but can't remember what
18:36:15 mriedem unrelated, for some weird reason, i'm tracing a reboot request through the API, and i see the request ID in the n-api logs but it's not showing up in the n-cpu logs

Earlier   Later