| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-03-14 | |||
| 16:12:59 | mriedem | melwitt: i think i'm going to just get ocata 13.0.1 out the door for this quotas issue, and add a release note with a known issue for the retry bug, | |
| 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 | 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: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: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: Make server_external_events use an optimal number of db queries https://review.openstack.org/445142 | |
| 16:45:34 | openstackgerrit | Matthew Booth proposed openstack/nova master: Add RequestContext.for_cell() https://review.openstack.org/445140 | |
| 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 | |