| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-01-05 | |||
| 17:53:42 | openstack | Launchpad bug 1653953 in OpenStack Compute (nova) "Unable to remove snapshots after an instance is unshelved when using the rbd imagebackend" [Undecided,New] | |
| 17:53:42 | melwitt | nic: hey, are you around? we've been looking at a rbd bug recently where after shelve/unshelve, the snapshot that was made, can't be deleted from glance. have you run into the same problem? https://bugs.launchpad.net/nova/+bug/1653953 | |
| 17:55:47 | openstackgerrit | Chris Dent proposed openstack/nova: HTTP interface for resource providers by aggregates https://review.openstack.org/407742 | |
| 17:56:33 | cdent | jaypipes: ^ better? | |
| 17:57:30 | cdent | btw, I made your suggested array slice change. I rarely use that, not sure why. I guess I find it semantically ugly. Meaning: I don't like to count. I guess. | |
| 17:58:35 | mriedem | i for one hate slice notation | |
| 17:58:42 | mriedem | because i suck at it | |
| 17:58:56 | jaypipes | +2 from me cdent. mriedem https://review.openstack.org/#/c/407742/ could use your eyes. | |
| 17:59:08 | dansmith | slices are the best thing | |
| 17:59:08 | mriedem | ok i need food first | |
| 17:59:13 | dansmith | since at least, I dunno, sliced bread | |
| 17:59:21 | jaypipes | lol :) | |
| 17:59:24 | mriedem | *rimshot* | |
| 17:59:45 | dansmith | slices are usually the first thing I miss in other languages | |
| 17:59:58 | jaypipes | and since we're going to be rewriting everything in Golang, and slices are a full-fledge type in Golang, we really should use them to their fullest. | |
| 18:00:54 | mriedem | maybe i'll get a slice of pizza | |
| 18:01:01 | dtp | is the [reply done] feature in gerrit used/useful for reviewers? | |
| 18:01:47 | mriedem | dtp: yes | |
| 18:01:52 | mriedem | as an ack that you saw the comment and handled it | |
| 18:01:53 | cfriesen | any chance of getting a second +2 on https://review.openstack.org/#/c/393487 ? Pretty trivial backport to newton, fixes a real issue. (I just tripped over it in Mitaka) | |
| 18:01:55 | dtp | k | |
| 18:02:06 | jaypipes | dtp: yeah, I use it. | |
| 18:02:17 | dtp | coo | |
| 18:04:35 | edleafe | jaypipes: cdent: not crazy about the slice change, since we might need to support "nin" too ("not in") | |
| 18:05:05 | jaypipes | edleafe: meh? :) | |
| 18:05:09 | cdent | edleafe: likely all that hoopa joop is gonna change as a result of the switch to jsonschema validation of query params | |
| 18:05:12 | edleafe | for anti-affinity and such | |
| 18:05:15 | cdent | and nin is yagni talk | |
| 18:05:21 | jaypipes | edleafe: what cdent said :) | |
| 18:05:38 | jaypipes | about the jsonschema thing, not the yagni thing. | |
| 18:05:54 | cdent | won't someone please think of the yagni | |
| 18:06:14 | edleafe | cdent: I am, as in "you ARE gonna need it" :) | |
| 18:07:14 | purplerbot | <jaypipes> edleafe: meh? :) [2017-01-05 18:05:05] [n 25H] | |
| 18:07:14 | cdent | [t 25H] | |
| 18:15:40 | dtp | what is soft delete? | |
| 18:17:08 | dtp | tentative delete but kept on disk for a bit? | |
| 18:17:46 | melwitt | dtp: yeah. "soft delete" as a feature is a deferred delete where there's a config option for low long before it gets really deleted forever. in the meantime, it can be restored | |
| 18:18:15 | dansmith | dtp: soft delete is an abomination | |
| 18:18:19 | dtp | thanks | |
| 18:18:24 | dtp | oh? | |
| 18:20:06 | melwitt | dtp: we often say "soft delete" to mean the way the "deleted" column is used in a lot of the database schemas too, fyi. in the code though, soft_delete means the feature | |
| 18:20:37 | dansmith | yes, there are two soft deletes | |
| 18:20:40 | dansmith | and they're both abominations | |
| 18:20:45 | dansmith | but in different ways | |
| 18:20:55 | dansmith | i.e. two roads to hell | |
| 18:21:16 | dtp | better bring a cooler | |
| 18:21:18 | dtp | when a soft-deleted instance is deleted for realzies, does it go through task_state=DELETING like normal? | |
| 18:21:51 | dansmith | and, that's why it's an abominaton | |
| 18:22:03 | dansmith | because to answer that, I'd have to look at the code, get the answer wrong once, then try again, etc | |
| 18:22:25 | dtp | i see. lets leave that alone for now | |
| 18:22:46 | dtp | i detect no emotional anomalies | |
| 18:23:01 | dansmith | heh | |
| 18:24:51 | mriedem | btw, | |
| 18:24:53 | mriedem | BTW, | |
| 18:25:09 | mriedem | soft delete in the API is configured via a config option that is actually only used on the compute that the instance is running on | |
| 18:25:14 | mriedem | and if those are different, oops! | |
| 18:25:26 | mriedem | s/configured/toggled/ in the api | |
| 18:25:33 | mriedem | so a config-driven API feature | |
| 18:26:34 | mriedem | not that we dont already have crap like that based on which virt driver is on the compute host the instance is running on...but that's more of a known evil | |
| 18:26:36 | melwitt | that's even worse than I thought | |
| 18:27:47 | ildikov | Hi Nova people and Happy New Year! | |
| 18:27:58 | ildikov | I see you found nice topics for the first week already :) | |
| 18:28:24 | ildikov | Is there any one around who could lecture me a bit on how to configure Nova to use the Cinder v3 API? | |
| 18:28:37 | mriedem | sec | |
| 18:28:37 | mriedem | ildikov: scottda has a WIP for that | |
| 18:29:08 | mriedem | https://review.openstack.org/#/c/385682/ i think | |
| 18:29:12 | ildikov | I thought I found the right config option for that and now in my one node Devstack install and the nova API succeeds with the v3 api call and the compute tells me it cannot find the endpoint... | |
| 18:29:48 | mriedem | ildikov: i believe it would be either endpoint_template | |
| 18:29:57 | ildikov | mriedem: right, so I set the catalog_info to v3 | |
| 18:30:06 | ildikov | in the nova.conf | |
| 18:30:08 | mriedem | or service_type or service_name, one of those is cinderv3 i think | |
| 18:30:43 | ildikov | it's volumev3:cinderv3:publicURL | |
| 18:30:50 | ildikov | in my config | |
| 18:30:55 | mriedem | and that fails? | |
| 18:31:05 | ildikov | only in the compute | |
| 18:31:21 | mriedem | is it a new devstack? | |
| 18:31:38 | ildikov | it says: EndpointNotFound: publicURL endpoint for volumev3 service named cinderv3 in RegionOne region not found | |
| 18:31:44 | ildikov | yes, new devstack | |
| 18:31:56 | mriedem | what's in the catalog? did you do openstack catalog list | |
| 18:31:59 | mriedem | or whatever that command is | |
| 18:32:22 | ildikov | this is the log from the API: Cinder api version 3 and url http://127.0.0.1:8776/v3/0f810b0913db4fc0a77135d2e40dfd23 | |
| 18:32:34 | ildikov | it's a one node devstack install | |
| 18:32:53 | ildikov | cinderv3 is listed among the endpoints | |
| 18:33:20 | ildikov | I'm sure it's something dumb obvious misconfiguration, but I cannot figure out what it can be | |
| 18:33:33 | mriedem | but can you list volumes via the nova api? | |
| 18:33:47 | mriedem | like: nova --os-compute-api-version 2.35 volume-list ? | |
| 18:33:55 | mriedem | that will proxy through to cinder | |
| 18:36:13 | ildikov | with volume-attachments it works | |
| 18:36:25 | mriedem | volume-attachments doesn't call cinder | |
| 18:36:30 | mriedem | it uses the block_device_mappings table in the nova db | |
| 18:36:53 | mriedem | i've got a newish devstack, i'll see if i can reproduce | |
| 18:38:58 | mriedem | this may take awhile, the cafeteria was playing don henley and i can't get it out of my head | |
| 18:39:06 | mriedem | in a bad way | |
| 18:39:59 | ildikov | pfff, ok, I will pick another one then as volume-list wasn't in the list | |
| 18:40:07 | mriedem | yeah we deleted that... | |
| 18:40:08 | mriedem | forgot | |
| 18:40:48 | mriedem | i'm going to install an older novaclient in a venv | |
| 18:41:35 | scottda | ildikov: I haven't tested that in a while. Let me set it up and see.... | |
| 18:42:22 | ildikov | scottda: if that works I will build that in to my patch, I'm currently only trying to make catalog_info work with Cinder v3 | |
| 18:42:34 | ildikov | scottda: using the latest attach code is hardcoded atm | |