| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-08-02 | |||
| 19:18:17 | mriedem | heh | |
| 19:18:17 | mriedem | oh no, i know why it's passing | |
| 19:18:45 | mriedem | yup | |
| 19:19:15 | mriedem | i'll just squash those changes together | |
| 19:19:27 | dansmith | mriedem: can you hold off? | |
| 19:19:29 | mriedem | cdent: ^ is why i'm looping the allocations | |
| 19:19:35 | dansmith | I have a bunch of cuts against all three of these patches | |
| 19:19:35 | mriedem | rather than assuming there is 1 | |
| 19:19:42 | mriedem | dansmith: like, deep cuts? | |
| 19:19:47 | dansmith | gashes | |
| 19:19:56 | dansmith | with rusty blades | |
| 19:20:10 | mriedem | i meant like https://www.youtube.com/watch?v=KCdKBHdPz30 | |
| 19:20:13 | mriedem | deep cuts | |
| 19:20:18 | dansmith | heh | |
| 19:20:20 | dansmith | nice one | |
| 19:21:12 | mriedem | btw, fagen is forced to tour again https://qz.com/1041397/steely-dans-donald-fagen-is-back-on-tour-the-result-of-nobody-buying-music-albums-anymore/ | |
| 19:21:19 | mriedem | streaming music has broken him | |
| 19:21:50 | mriedem | he should probably talk to a financial advisor about diversifying his portfolio | |
| 19:22:04 | melwitt | lol | |
| 19:22:38 | dfisher | is there a known issue with calling nova.context.get_admin_context() from within a virt driver? http://paste.openstack.org/show/617308/ | |
| 19:23:11 | melwitt | did not expect that article to mention Ronnie James Dio | |
| 19:23:18 | mriedem | hologram dio | |
| 19:23:20 | mriedem | terrible | |
| 19:23:26 | melwitt | I know, like, seriously? | |
| 19:23:49 | mriedem | i was at a sabbath reunion show once and dio scolded the audience and threatened to cut the show and leave if they didn't settle down | |
| 19:24:30 | dansmith | cripes | |
| 19:24:43 | dansmith | jaypipes: how do we get the old_flavor if we're here: https://github.com/openstack/nova/blob/master/nova/compute/resource_tracker.py#L460-L460 ? | |
| 19:25:16 | mriedem | dansmith: isn't old_flavor stored on the instance? | |
| 19:25:41 | dansmith | mriedem: well, I would assume not at this point since we go to great lengths to look it up on L443 | |
| 19:26:05 | mriedem | maybe that predates the old_flavor being stored on the instance? | |
| 19:26:08 | dansmith | no, | |
| 19:26:13 | dansmith | we set it to None before we call this here: | |
| 19:26:31 | dansmith | https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L3501-L3521 | |
| 19:26:34 | dansmith | son of a goat | |
| 19:26:41 | mriedem | is it stashed on the migration record? | |
| 19:26:46 | dansmith | no, only the id | |
| 19:26:55 | mriedem | # NOTE(danms): delete stashed migration information | |
| 19:26:55 | mriedem | ha | |
| 19:27:02 | mriedem | BUT WHY?! | |
| 19:27:02 | dansmith | wait | |
| 19:27:20 | dansmith | we pass old_instance_type for instance_type, | |
| 19:27:27 | dansmith | but clearly that can be None sometimes | |
| 19:27:30 | dansmith | but I don't know when | |
| 19:27:36 | dansmith | maybe if we're doing a migrate? | |
| 19:27:43 | mriedem | probably yeah | |
| 19:27:47 | mriedem | no old/new if it's not a resize | |
| 19:27:48 | dansmith | so maybe I can use old_instance_type or instance.flavor | |
| 19:27:54 | mriedem | f yes you can | |
| 19:30:53 | mriedem | dfisher: if you're using a recent devstack, it's running in superconductor mode which means you can't do retries from the compute | |
| 19:31:00 | mriedem | it can't upcall to the api db to get instance group info | |
| 19:31:16 | mriedem | if you're doing some crazy crap in the oracle virt driver, then you're on your own | |
| 19:32:49 | dfisher | i'm really not doing anything crazy … I don't think. | |
| 19:32:59 | dansmith | dfisher: well, except for that one crazy thing | |
| 19:33:18 | dfisher | like, I'm using nova.api.metadata.password.convert_password | |
| 19:33:23 | dfisher | which takes a context | |
| 19:35:07 | dfisher | dansmith: ssssh. don't tell them </loud whisper> | |
| 19:37:38 | mriedem | dfisher: if you're looking at trunk code, build_instances in the conductor is only hit if (1) youre' using cellsv1 which you shouldn't be or (2) if the compute is doing a retry of a build | |
| 19:37:48 | dansmith | okay I think I'm doing the right thing, and now I'm failing because of that 500 from placement | |
| 19:37:56 | dansmith | so I'll push this up and let someone else look at that in parallel | |
| 19:37:59 | mriedem | dfisher: so assuming you're hitting (2), if you're using devstack in the default superconductor mode, the retry isn't going to work | |
| 19:38:16 | dfisher | hmm. ok. | |
| 19:38:20 | mriedem | dfisher: b/c you're hitting the cell conductor which doesn't have access to the scheduler | |
| 19:38:23 | mriedem | or api db for that matter | |
| 19:38:36 | mriedem | if you're not using devstack, or you're using a devstack that's older than like 1 week, then *shrug* | |
| 19:38:50 | dfisher | no, devstack is from yesterday or maybe Monday | |
| 19:38:51 | melwitt | yeah. the error is saying the node handling the database read doesn't have the API database connection configured in nova.conf | |
| 19:39:02 | openstackgerrit | Dan Smith proposed openstack/nova master: remove provider allocs in confirm/revert resize https://review.openstack.org/488510 | |
| 19:39:03 | openstackgerrit | Dan Smith proposed openstack/nova master: Test resize to same host with placement api https://review.openstack.org/489973 | |
| 19:39:03 | openstackgerrit | Dan Smith proposed openstack/nova master: WIP: Sum allocations in the scheduler when resizing to the same host https://review.openstack.org/490085 | |
| 19:39:19 | mriedem | dfisher: if you need retries, then set CELLSV2_SETUP=singleconductor in your stackrc | |
| 19:39:20 | dansmith | cdent: jaypipes^ check out 'tox -efunctional ServerMovingTest' in the top patch there ^ | |
| 19:39:23 | mriedem | and restackaroo | |
| 19:39:31 | dfisher | very cool. thank you | |
| 19:39:40 | dfisher | i really appreciate it | |
| 19:39:44 | mriedem | dfisher: also https://docs.openstack.org/nova/latest/user/cellsv2_layout.html | |
| 19:39:49 | mriedem | digest that | |
| 19:42:00 | jaypipes | dansmith: gotcha. looking at it now. | |
| 19:42:16 | dansmith | also note the FIXME about cheating | |
| 19:42:22 | jaypipes | k | |
| 19:42:26 | dansmith | that's clearly just a hack to do some math, but needs to be integrated in the right place | |
| 19:43:16 | dansmith | IndexError: list index out of range | |
| 19:43:16 | dansmith | specifically this here: consumer_id = allocs[0].consumer_id | |
| 19:43:48 | dansmith | maybe that somehow means I'm passing an empty list of things to set and placement isn't having ti? | |
| 19:43:50 | dansmith | *it | |
| 19:44:44 | jaypipes | not sure, still looking :) | |
| 19:44:57 | dansmith | oh, well, heh | |
| 19:45:01 | dansmith | I'm definitely not adding it back in | |
| 19:45:02 | dansmith | duh | |
| 19:46:14 | dansmith | heyyyyy | |
| 19:46:15 | dansmith | woo | |
| 19:46:25 | dansmith | fix comin' | |
| 19:46:27 | openstackgerrit | Dan Smith proposed openstack/nova master: remove provider allocs in confirm/revert resize https://review.openstack.org/488510 | |
| 19:46:33 | dansmith | that passes & | |
| 19:46:58 | jaypipes | k, lemme pull that | |
| 19:47:33 | dansmith | but, that means that placement will 500 if you pass it an empty set of allocations (/cc cdent) | |
| 19:47:44 | cdent | noted | |
| 19:47:52 | jaypipes | cdent: bug please if you would | |
| 19:48:02 | cdent | yeah, just finishing something else first | |
| 19:48:02 | jaypipes | that should be an easy fix. | |
| 19:52:19 | jaypipes | dansmith: am I kosher to work on fixing that ocata to pike func test failure or are you still working on that patch? | |