| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-09-20 | |||
| 16:50:34 | mriedem | likely a todo to throw on the pile for my patch at this point | |
| 16:50:39 | dansmith | ack | |
| 16:50:51 | mriedem | i have officially started losing sleep over this | |
| 16:51:16 | mriedem | need to contract out some hot alaski action | |
| 16:54:29 | dansmith | heh | |
| 16:56:59 | mnaser | i dont have time right now to get this audit code into nova-manage so i'll leave it for anyone to hack on later (till iget the time) | |
| 16:57:22 | mriedem | mnaser: can you report a bug and throw it in there so we don't lose track of it? | |
| 16:57:33 | mnaser | mriedem: fair enough | |
| 16:59:25 | mnaser | https://bugs.launchpad.net/nova/+bug/1793569 | |
| 16:59:25 | openstack | Launchpad bug 1793569 in OpenStack Compute (nova) "Add placement audit commands" [Undecided,New] | |
| 17:05:52 | mriedem | thanks | |
| 17:13:40 | openstackgerrit | Surya Seetharaman proposed openstack/nova master: API microversion bump for handling-down-cell https://review.openstack.org/591657 | |
| 17:19:36 | lucidguy | if I curl the placment API for an allocation (instance) its | |
| 17:19:36 | lucidguy | responding it exists on the wrong hypervisor ID. Openstack | |
| 17:19:39 | lucidguy | calls show it in the right place. How can I edit this, can't | |
| 17:19:42 | lucidguy | figure out the PUT equivalent syntax. Tempted to delete but | |
| 17:19:45 | lucidguy | that sounds very wrong | |
| 17:19:48 | lucidguy | Ouch, sorry about that | |
| 17:20:56 | lucidguy | if I curl the placment API for an allocation (instance) its responding it exists on the wrong hypervisor ID. Openstack calls show it in the right place. How can I edit this, can't figure out the PUT equivalent syntax. Tempted to delete but that sounds very wrong. | |
| 17:21:03 | lucidguy | That's better | |
| 17:25:04 | dansmith | lucidguy: you should use osc-placement to fix it | |
| 17:25:54 | lucidguy | osc-placement.. hmm, never heard of it. | |
| 17:27:09 | lucidguy | How does it work. | |
| 17:27:43 | dansmith | https://docs.openstack.org/osc-placement/latest/index.html | |
| 17:27:58 | dansmith | lucidguy: also check out https://github.com/larsks/os-placement-tools | |
| 17:28:18 | larsks | dansmith: hah, I just pasted that to him in #openstack :) | |
| 17:28:26 | lucidguy | Sounds like something new and time consuming to learn. | |
| 17:29:48 | lucidguy | I would pay for a quick solution.. :) | |
| 17:30:12 | larsks | lucidguy: I haven't been following the conversation, but that script on github may be a quick solution. | |
| 17:30:30 | dansmith | larsks: good, because that's the right place to discuss :) | |
| 17:30:37 | larsks | Yeah | |
| 17:32:24 | lucidguy | I don't think the orginization would like a random script hitting the production enviroment. | |
| 17:32:38 | lucidguy | I'm sure it's fine, but if things were to go wrong, I would be screwed | |
| 17:32:52 | dansmith | so wat | |
| 17:32:55 | dansmith | *wait | |
| 17:33:04 | dansmith | you wanted a quick solution instead of learning a tool, | |
| 17:33:12 | dansmith | and were just going to curl delete an instance's resource allocation, | |
| 17:33:33 | dansmith | but a "random script" that was written by people that know openstack and have tested it in production is not okay? | |
| 17:35:12 | lucidguy | Again, I'm sure it's safe. But my knowledge of the placment service is little to none. Ideally there was a tool provided that can assist me. | |
| 17:35:14 | cdent | lucidguy: what release are you running? | |
| 17:36:05 | lucidguy | Queens | |
| 17:36:51 | cdent | lucidguy: yeah, in that case osc-placement _is_ the tool provided to assist you | |
| 17:37:24 | dansmith | and check_placement is the automated use of that tool to fix your exact problem without you having to learn everything about the tool | |
| 17:38:00 | cdent | lucidguy: also see https://bugs.launchpad.net/nova/+bug/1793569 | |
| 17:38:00 | openstack | Launchpad bug 1793569 in OpenStack Compute (nova) "Add placement audit commands" [Wishlist,Confirmed] | |
| 17:39:06 | lucidguy | I can see https://paste.ubuntu.com/p/8F3JwHqSjz/ with the appropriate curl GET command, is it as simple as performing a PUT to change that hypervisor ID? | |
| 17:39:29 | bauzas | dansmith: mriedem: others: gentle notice that I'll be on PTO tomorrow | |
| 17:39:51 | bauzas | I'm actively working on libvirt reshape, still work to do on inventories | |
| 17:40:03 | bauzas | then I'll push a WIP for review | |
| 17:40:11 | dansmith | bauzas: ack | |
| 17:40:15 | cdent | lucidguy: which curl GET produced that? | |
| 17:40:23 | dansmith | FWIW, I'm out next week from tuesday->friday | |
| 17:40:30 | bauzas | ok | |
| 17:40:33 | dansmith | bauzas: so just have it all ready on monday, k? :) | |
| 17:40:35 | cdent | me M->W | |
| 17:40:47 | cdent | going to dan's house | |
| 17:40:51 | bauzas | dansmith: I'm really rushing to provide it ASAP :) | |
| 17:41:03 | bauzas | but there are a couple of things I need to consider | |
| 17:41:13 | jiteka | mriedem: thanks for the answer for my question from yesterday about cold-migration support in Mitaka | |
| 17:41:23 | mriedem | jiteka: yw | |
| 17:41:36 | bauzas | since I was a bit off when implementing reshape, I just wonder if the only trigger for reshape is the ReshapeNeeded exception that then sends allocations ? | |
| 17:41:38 | bauzas | dansmith: ^ | |
| 17:41:38 | jiteka | mriedem: Today I did some testing and face the same exception that was I was getting from live-migration | |
| 17:41:40 | lucidguy | cdent: https://paste.ubuntu.com/p/n3GCFR8MSt/ | |
| 17:42:17 | bauzas | dansmith: IIUC, the update_provider_tree() method is just running periodically, but we only pass allocations as a non-null param when we trigger a reshape, right? | |
| 17:42:36 | jiteka | mriedem: http://paste.openstack.org/show/730464/ | |
| 17:42:53 | dansmith | bauzas: I haven't followed the whole thing, but IIRC you only get to raise that on the first call of update so that it's done before compute starts up | |
| 17:43:02 | bauzas | dansmith: yup, that I understood | |
| 17:43:08 | cdent | lucidguy: so the is issue is that you've got the allocations showing up on two different resource providers and it should just be one. If that's the case, edit the json to remove the block identified by the wrong one, and PUT it back to the same URL | |
| 17:43:14 | bauzas | dansmith: and then we call again this method by passing an allocations param | |
| 17:43:31 | bauzas | so we only reshape when allocations is non-None | |
| 17:43:50 | bauzas | the other cases are then periodic | |
| 17:43:51 | dansmith | bauzas: not sure the details | |
| 17:43:59 | lucidguy | cdent: Sadly I don't know how to do that. | |
| 17:44:19 | dansmith | lucidguy: luckily there is a script that fixes this for you so you don't haveto | |
| 17:44:41 | cdent | yeah, lucidguy, if you're not comfortable running curl, use the scripts | |
| 17:44:53 | bauzas | dansmith: no worries, I'll test it on my devbox then | |
| 17:45:04 | bauzas | I should dump the DB to fake a reshape then | |
| 18:29:25 | openstackgerrit | Eric Fried proposed openstack/nova master: Nix update_instance_allocation, _allocate_for_instance https://review.openstack.org/604186 | |
| 18:45:39 | efried | jaypipes-ooo, gibi, mriedem: --^ | |
| 18:46:56 | mriedem | fake news | |
| 18:49:21 | mriedem | lemme -1 quick | |
| 18:50:44 | mriedem | just need to update some docstrings | |
| 18:50:56 | mriedem | otherwise i'm in love with your patch | |
| 18:51:30 | efried | mm, good eye | |
| 18:51:46 | mriedem | i only have one | |
| 18:51:55 | efried | I used ^g, not grep | |
| 18:53:06 | openstackgerrit | Eric Fried proposed openstack/nova master: Nix update_instance_allocation, _allocate_for_instance https://review.openstack.org/604186 | |
| 18:53:14 | efried | mriedem: Not sure if I did that right | |
| 18:53:27 | efried | I didn't go and look at the resource tracker | |
| 18:54:00 | mriedem | i did, with jay's change underneath it's good | |
| 18:54:29 | efried | coo | |
| 18:54:41 | openstackgerrit | Elancheran S proposed openstack/nova master: Add exact match aggregate image properties matcher/filter https://review.openstack.org/593167 | |
| 18:56:18 | mriedem | baby talk will get you everywhere | |
| 19:01:48 | openstackgerrit | Merged openstack/nova master: doc trivial: additional info to admin-password-injection https://review.openstack.org/603414 | |
| 19:14:02 | openstackgerrit | Merged openstack/nova stable/pike: Fix DB archiver AttributeError due to wrong table name attribute used https://review.openstack.org/599883 | |
| 19:23:12 | openstackgerrit | Jim Rollenhagen proposed openstack/nova master: ironic: stop hammering ironic API in power sync loop https://review.openstack.org/602127 | |
| 19:23:13 | jroll | mriedem: that has a bug report and such now per your request, should also be passing tests now ^ | |
| 19:24:01 | mriedem | ok | |
| 19:25:59 | openstackgerrit | Merged openstack/nova master: XenAPI/Stops the migration of volume backed VHDS https://review.openstack.org/533168 | |