| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-03-14 | |||
| 16:13:09 | mriedem | and then once that's fixed (with a functional recreate test) we can do a 13.0.2 shortly after | |
| 16:13:27 | mriedem | which reminds me to beg for stable reviews on https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:stable/ocata | |
| 16:13:32 | hamzy | mriedem, http://paste.openstack.org/show/602712/ and thanks... "openstack server create" was able to boot an instance | |
| 16:13:33 | melwitt | okay | |
| 16:13:34 | mriedem | once those are in we can do a 13.0.1 release | |
| 16:13:48 | mriedem | hamzy: nice, cool. | |
| 16:13:58 | mriedem | hamzy: i'll make a note of the workaround instructions in the release note for ocata 13.0.1 | |
| 16:14:14 | mriedem | *er 15.0.1 | |
| 16:14:20 | hamzy | yeah that would be great | |
| 16:16:23 | mriedem | https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:stable/ocata | |
| 16:16:26 | mriedem | oops | |
| 16:16:39 | mriedem | sdague: bauzas: claudiub: ^ can you help out here? | |
| 16:19:58 | openstackgerrit | Gábor Antal proposed openstack/nova-specs master: Show persisted scheduler hints in server details https://review.openstack.org/440580 | |
| 16:24:58 | openstack | Launchpad bug 1672624 in OpenStack Compute (nova) "Ceph volumes attached to local deleted instance could not be correctly handled" [Undecided,New] | |
| 16:24:58 | melwitt | Kevin_Zheng: I was just starting to look at this bug you reported https://bugs.launchpad.net/nova/+bug/1672624 do you know if it's a regression? | |
| 16:26:43 | sdague | mriedem: after I get off a call | |
| 16:45:10 | jaypipes | rpodolyaka: we'd welcome you help on the osc plugin for placement. :) | |
| 16:45:33 | openstackgerrit | Matthew Booth proposed openstack/nova master: Add InstanceMappingList.get_by_instance_uuids https://review.openstack.org/445139 | |
| 16:45:34 | openstackgerrit | Matthew Booth proposed openstack/nova master: Add RequestContext.for_cell() https://review.openstack.org/445140 | |
| 16:45:34 | openstackgerrit | Matthew Booth proposed openstack/nova master: Make server_external_events use an optimal number of db queries https://review.openstack.org/445142 | |
| 16:45:35 | openstackgerrit | Matthew Booth proposed openstack/nova master: Add InstanceList.get_by_instance_mapping_list() https://review.openstack.org/445141 | |
| 16:49:21 | rpodolyaka | jaypipes: sure, np! :) | |
| 16:50:36 | mriedem | rpodolyaka: thanks, you're the owner on the bp now | |
| 16:52:27 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Refactor InstancePayload creation https://review.openstack.org/418488 | |
| 16:52:28 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Remove **kwargs passing in payload __init__ https://review.openstack.org/418489 | |
| 16:52:33 | rpodolyaka | mriedem: thanks! | |
| 16:54:03 | mriedem | lyarwood: i just posted a release note to ocata for 15.0.1 also https://review.openstack.org/#/c/445575/ | |
| 16:57:41 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Transform aggregate.add_host notification https://review.openstack.org/396210 | |
| 16:57:42 | openstackgerrit | Balazs Gibizer proposed openstack/nova master: Transform aggregate.remove_host notification https://review.openstack.org/396211 | |
| 16:58:05 | openstackgerrit | John Garbutt proposed openstack/nova-specs master: Add locked-by-service spec https://review.openstack.org/438134 | |
| 17:03:01 | johnthetubaguy | siva_krishnan: thinking about this spec: https://review.openstack.org/#/c/347161 are we not now back to basically what the original spec said now? | |
| 17:04:08 | mdbooth | johnthetubaguy: What's a service locked instance? | |
| 17:04:39 | johnthetubaguy | mdbooth: its writing up the old story we had around trove "locking" and instance from a user deleting it | |
| 17:05:14 | siva_krishnan | johnthetubaguy: yeah only with one change. instead of generating unrescue.xml from database we are copying it to destination from source | |
| 17:05:39 | johnthetubaguy | siva_krishnan: ah, OK, could we add a not in the git commit saying thats the only change? | |
| 17:05:50 | johnthetubaguy | s/not/note/ | |
| 17:06:04 | openstackgerrit | Sujitha proposed openstack/nova master: Add description to policies in aggregates.py https://review.openstack.org/445230 | |
| 17:06:09 | johnthetubaguy | mdbooth: yeah, I hadn't noticed that overloaded term, damm | |
| 17:06:20 | mdbooth | What's a service, then? | |
| 17:06:51 | johnthetubaguy | mdbooth: in that context, is the service token in the context | |
| 17:07:19 | johnthetubaguy | mdbooth: so trove has a service token, it should be in the spec, although something tells me its probably not clear in there | |
| 17:07:44 | siva_krishnan | johnthetubaguy: sure. will update the commit message | |
| 17:08:01 | mdbooth | I've read the description and use cases sections a couple of times and hadn't managed to infer it from context. | |
| 17:08:08 | mdbooth | That could just be me, though. | |
| 17:09:30 | openstackgerrit | John Garbutt proposed openstack/nova-specs master: Add locked-by-service spec https://review.openstack.org/438134 | |
| 17:10:21 | johnthetubaguy | mdbooth: not sure, the folks reading that so far have a lot of shared context around the problem | |
| 17:12:29 | openstackgerrit | Gábor Antal proposed openstack/nova master: Add json style checking for sample notifications https://review.openstack.org/443677 | |
| 17:18:57 | johnthetubaguy | siva_krishnan: thanks | |
| 17:19:20 | mdbooth | johnthetubaguy: Incidentally, you mentioned (last week, I think) you'd have another look at https://review.openstack.org/#/c/383859/ | |
| 17:20:02 | johnthetubaguy | mdbooth: damm, yes, still not got out of the specs really | |
| 17:20:54 | mdbooth | I'd really like to get that landed, or at least know what needs doing to it. | |
| 17:21:08 | mdbooth | It's a bit in limbo right now. | |
| 17:21:53 | openstackgerrit | Sivasathurappan Radhakrishnan proposed openstack/nova-specs master: Live Migration of Rescued Instances https://review.openstack.org/347161 | |
| 17:22:11 | jaypipes | mriedem, dansmith: your reviews on https://review.openstack.org/#/c/441544/ would be great. thanks in advance! | |
| 17:22:15 | johnthetubaguy | siva_krishnan: acutally that spec is really different from the previously approved one: http://specs.openstack.org/openstack/nova-specs/specs/newton/approved/live-migrate-rescued-instances.html | |
| 17:22:31 | mriedem | jaypipes: ack | |
| 17:22:46 | johnthetubaguy | mdbooth: understood, I hope to get there really soon | |
| 17:22:47 | sdague | mriedem: you are going to have to explain the flavor param on https://review.openstack.org/#/c/445236/1to me | |
| 17:22:55 | jaypipes | dansmith: what's your most pressing patch for me to review? | |
| 17:23:11 | sdague | because I'm not seeing any logic that makes that a different value in the new code than the old code | |
| 17:23:34 | dansmith | sdague: it's because of lazy loading | |
| 17:23:40 | dansmith | sdague: needing to go to the right cell when that happens | |
| 17:24:00 | dansmith | jaypipes: I'm currently moving code around so.. nothing right now :( | |
| 17:24:16 | mriedem | sdague: must did | |
| 17:24:18 | mriedem | *just did | |
| 17:24:20 | mdbooth | johnthetubaguy: Thanks! | |
| 17:24:25 | dansmith | jaypipes: you can look at this if you want: https://review.openstack.org/#/c/427903 | |
| 17:24:28 | mriedem | sdague: it's all about cell context | |
| 17:24:35 | jaypipes | dansmith: cool, will do. | |
| 17:24:40 | dansmith | but it's not super pressing | |
| 17:25:13 | dansmith | jaypipes: not sure I have enough context in those commit messages for you to know why that's important actually.. unless you were paying attention at the PTG :) | |
| 17:25:31 | siva_krishnan | johnthetubaguy: we have almost changed all the textual content of previously approved spec but functionality wise this was the only change we agreed on. | |
| 17:25:32 | mriedem | dansmith: nor the bp description :) | |
| 17:26:00 | dansmith | I know | |
| 17:26:03 | jaypipes | dansmith: I'll ask if I don't understand :) | |
| 17:26:16 | mriedem | i'll add notes to the bp | |
| 17:29:51 | mriedem | dansmith: there https://blueprints.launchpad.net/nova/+spec/discover-hosts-faster | |
| 17:30:25 | dansmith | mriedem: <3 | |
| 17:30:27 | dansmith | jaypipes: ^ | |
| 17:30:32 | sdague | mriedem / dansmith ok, so you end up having to change context to look up the flavor, because it's only in the the cell db ? | |
| 17:30:44 | mriedem | sdague: the instance is in the cell0 db | |
| 17:30:45 | sdague | but then you are making all the quota changes at the api db? | |
| 17:30:49 | mriedem | the context, by default, looks in the nova db | |
| 17:30:58 | jaypipes | bueno. | |
| 17:31:01 | mriedem | no, the quota changes are in the nova db | |
| 17:31:13 | mriedem | quotas aren't in the api db yet, which is why we have this bug | |
| 17:31:19 | dansmith | well, | |
| 17:31:20 | dansmith | not really | |
| 17:31:26 | dansmith | but if he buys that, then run with it | |
| 17:31:29 | dansmith | oops, was that public? | |
| 17:31:34 | mriedem | ha | |
| 17:31:42 | mriedem | you typed too fast | |
| 17:32:05 | dansmith | sdague: api hedged and put the quota reservation in the non-cell0 cell the instance was destined for, | |
| 17:32:14 | dansmith | but then we put it in cell0 when we couldn't schedule it | |
| 17:32:20 | sdague | so, I guess I'm not sure I understand why the internals interface doesn't make instance.flavor just go lazy load from the right place | |
| 17:32:25 | dansmith | sdague: once our quotas are "fixed for cells" then this won't be a problem | |
| 17:32:53 | sdague | or is that just because this was a late breaking bug, and this was the smallest backportable thing? | |
| 17:33:09 | dansmith | sdague: because the instance is now in cell0, | |
| 17:33:17 | dansmith | sdague: but the reservations were created in cellN in a previous request | |
| 17:34:36 | mriedem | "why the internals interface doesn't make instance.flavor just go lazy load from the right place" the instance object would have to lookup it's own instance mapping to find the cell it's in, and then do a lookup using that context i believe, | |