| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-03-14 | |||
| 15:57:31 | hamzy | mriedem, http://paste.openstack.org/show/602711/ | |
| 15:58:22 | johnthetubaguy | raj_singh: got a few nits on the follow on spec too | |
| 15:58:32 | mriedem | | instances | {u'limit': 10, u'reserved': 0, u'in_use': 6} | | |
| 15:58:40 | mriedem | yeah usage is still wrong | |
| 15:58:51 | raj_singh | johnthetubaguy: thanks for the review | |
| 16:00:09 | mriedem | hamzy: is the [database]/connection option in the nova.conf that the aio1-nova-conductor-container-41603c3d container is using pointing at the 'nova' database? | |
| 16:01:34 | openstackgerrit | sahid proposed openstack/nova master: numa: fit instance NUMA node with cpus reserved onto host NUMA node https://review.openstack.org/400875 | |
| 16:01:34 | openstackgerrit | sahid proposed openstack/nova master: libvirt: return a CPU overhead if isolate emulator threads requested https://review.openstack.org/385364 | |
| 16:01:36 | openstackgerrit | sahid proposed openstack/nova master: virt: update overhead to take into account vCPUs https://review.openstack.org/385024 | |
| 16:01:39 | hamzy | mriedem, there is a line "connection = mysql+pymysql://nova:random_password@172.29.236.100/nova?charset=utf8" | |
| 16:01:42 | openstackgerrit | sahid proposed openstack/nova master: libvirt: make emulator threads to run on the reserved pCPU https://review.openstack.org/400878 | |
| 16:01:44 | openstackgerrit | sahid proposed openstack/nova master: numa: take into account cpus reserved https://review.openstack.org/400876 | |
| 16:01:46 | openstackgerrit | sahid proposed openstack/nova master: numa: update numa usage incording the cpu reserved https://review.openstack.org/400877 | |
| 16:02:00 | mriedem | hamzy: yeah that's the nova db | |
| 16:02:06 | mriedem | /nova is the nova db | |
| 16:04:19 | mriedem | so i'm thinking the refresh CLI doesn't fix things because the refresh operation thinks that usage is correct, because of the bug, we never decremented usage | |
| 16:04:37 | hamzy | so no --force option? :) | |
| 16:05:21 | mriedem | nova quota-update will just update the limits, not the usages, | |
| 16:06:00 | mriedem | i'm not sure if "nova quota-delete" will do what you want, i suspect that will reset quota limits to the default, but not touch usages, | |
| 16:06:02 | mriedem | i can look quick | |
| 16:06:13 | hamzy | thanks | |
| 16:07:15 | mriedem | actually it looks like it will, | |
| 16:07:28 | mriedem | it should delete the quota limit, usage and any pending reservations | |
| 16:08:01 | mriedem | it sucks, but if you can reset the quota for that project, then that's what you want "nova quota-delete --tenant <tenant id>" | |
| 16:08:33 | hamzy | mriedem, and then how do you recreate the quota? I don't see a quota-create | |
| 16:08:53 | mriedem | there are global defaults for quota in nova.conf | |
| 16:09:12 | mriedem | now might be a good time for me to cleanup https://review.openstack.org/#/c/417563/ | |
| 16:09:21 | mriedem | but ^ is an attempt and documenting some of how quotas work in nova | |
| 16:11:07 | openstackgerrit | Tadas Ustinavičius proposed openstack/nova master: Fixing flake8 errors https://review.openstack.org/445558 | |
| 16:12:10 | melwitt | mriedem: no, didn't do anything with a functional test yet. so feel free :) | |
| 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 | 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. | |