| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-03-03 | |||
| 21:17:53 | superdan | https://review.openstack.org/#/c/416521/ | |
| 21:17:54 | melwitt | it was passing tempest | |
| 21:18:14 | melwitt | yeah. I think I have to rebase it | |
| 21:18:33 | superdan | okay, well, that's encouraging either way | |
| 21:19:17 | melwitt | yeah, definitely will be easy to get a run and see how many failures go away | |
| 21:23:14 | mriedem | siva_krishnan: your patch kills libvirt http://logs.openstack.org/70/434870/3/check/gate-tempest-dsvm-py35-ubuntu-xenial/a2c2fdd/logs/screen-n-cpu.txt.gz?level=TRACE#_2017-03-01_01_51_37_675 | |
| 21:23:17 | mriedem | which takes down the compute service | |
| 21:23:24 | mriedem | and in a single node job, there is nothing to schedule to | |
| 21:23:28 | mriedem | so you get NoValidHost | |
| 21:25:37 | mriedem | i left a comment | |
| 21:25:43 | mriedem | post copy is not available until libvirt 1.3.3 | |
| 21:25:45 | mriedem | we test against 1.3. | |
| 21:25:49 | mriedem | *1.3.1 | |
| 21:27:06 | siva_krishnan | mriedem: ah ok . didn't check this job. thanks a lot. will makes changes accordingly | |
| 21:27:59 | openstackgerrit | Dan Smith proposed openstack/nova master: WIP: Get instance availability_zone without hitting the api db https://review.openstack.org/439754 | |
| 21:28:00 | openstackgerrit | Dan Smith proposed openstack/nova master: Target cell in super conductor operations https://review.openstack.org/438022 | |
| 21:28:00 | openstackgerrit | Dan Smith proposed openstack/nova master: Change MQ targeting to honor only what is in the context https://review.openstack.org/438023 | |
| 21:28:01 | openstackgerrit | Dan Smith proposed openstack/nova master: Make scheduler target cells to get compute node instance info https://review.openstack.org/439891 | |
| 21:28:02 | openstackgerrit | Dan Smith proposed openstack/nova master: Make consoleauth target the proper cell https://review.openstack.org/441461 | |
| 21:28:03 | openstackgerrit | Dan Smith proposed openstack/nova master: Add cell field to Destination object https://review.openstack.org/438024 | |
| 21:28:07 | openstackgerrit | Dan Smith proposed openstack/nova master: WIP: Make conductor ask scheduler to limit migrates to same cell https://review.openstack.org/438025 | |
| 21:31:17 | openstackgerrit | Maciej Szankin proposed openstack/nova master: conf: remove console_driver opt https://review.openstack.org/441485 | |
| 21:33:50 | openstackgerrit | Dan Smith proposed openstack/nova master: Count resources to check quota for cells https://review.openstack.org/416521 | |
| 21:33:52 | openstackgerrit | Dan Smith proposed openstack/nova master: Make Quotas object favor the API database https://review.openstack.org/410945 | |
| 21:33:52 | superdan | melwitt: rebasificated ^ | |
| 21:33:52 | openstackgerrit | Dan Smith proposed openstack/nova master: Add online migration to move quotas to API database https://review.openstack.org/410946 | |
| 21:34:45 | melwitt | superdan: I was just in the middle of doing that. thanks | |
| 21:34:51 | superdan | melwitt: oh sorry | |
| 21:36:07 | melwitt | superdan: np. I didn't mention I was doing it so :P | |
| 21:37:53 | melwitt | I was being paranoid running all the unit and functional tests on it | |
| 21:38:19 | superdan | meh, jenkins'll do that :P | |
| 21:39:09 | superdan | stacking with your stuff under mine now | |
| 21:39:11 | melwitt | it's gonna fail pep8 because a new hacking change went in since last I updated it, I think | |
| 21:39:23 | melwitt | everything else should be okay. *should* be okay | |
| 21:39:44 | superdan | meh, pep8 is for losers | |
| 21:39:53 | melwitt | that's true | |
| 21:41:09 | cburgess | lol | |
| 22:00:50 | efried_ | If an op like pause or suspend isn't implemented in my driver, and I do that op from the UI, shouldn't it fail? | |
| 22:01:19 | efried_ | n-cpu shows the NotImplementedError, but the UI gives me the green box claiming it succeeded. Though the state doesn't change (so I can't e.g. resume after that) | |
| 22:03:55 | sdague | efried_: the commands are async | |
| 22:04:14 | sdague | it succeeds with having submitted the command | |
| 22:04:43 | efried_ | Whoah. Okay. That's news to me. sdague Not all of the commands are async, are they? | |
| 22:04:54 | sdague | efried_: most of them | |
| 22:05:26 | efried_ | Guess I'm not used to using anything other than spawn, destroy, and migrate. Thanks for the info sdague | |
| 22:05:40 | sdague | all of those are async | |
| 22:06:29 | efried_ | sdague Is that new, then? Cause I swear when something goes wrong in the middle of e.g. spawn, the UI shows me a red failure box. | |
| 22:06:39 | efried_ | I haven't done that from the UI in a while... | |
| 22:07:12 | superdan | melwitt: well it stacked | |
| 22:07:15 | sdague | efried_: it might be polling, but the API call does some basic sanity checking, then returns success well before a vm is ever booting | |
| 22:07:31 | sdague | which is generally the model all the commands work in | |
| 22:07:51 | efried_ | Okay. | |
| 22:08:18 | melwitt | superdan: what do you mean, locally? no change in failures? | |
| 22:08:29 | superdan | melwitt: yeah, locally | |
| 22:08:33 | superdan | running some tests now | |
| 22:08:53 | superdan | holy | |
| 22:08:54 | superdan | shite | |
| 22:08:55 | melwitt | hm | |
| 22:09:05 | superdan | I shan't believe my eyes | |
| 22:09:08 | melwitt | what | |
| 22:09:11 | superdan | oh wait | |
| 22:09:16 | superdan | this passed by itself | |
| 22:09:33 | superdan | so at least it runs.. I gotta let it run a bunch of crap to see if it actually fixed my problem | |
| 22:10:55 | melwitt | okay | |
| 22:12:56 | superdan | man these scenario tests ain't in a hurry | |
| 22:13:48 | mriedem | superdan: ask sdague about his love of the current state of the scenario tests | |
| 22:13:58 | superdan | mriedem: I know, he loooooves them | |
| 22:14:41 | mriedem | sdague loves the scenario tests more than a warm cup of coffee on a crisp october morn | |
| 22:15:33 | sdague | hey, we're going to trim a bunch by default RSN | |
| 22:16:28 | mriedem | leakypipes: just the man i want to harass | |
| 22:16:40 | mriedem | leakypipes: so we need a launchpad site for os-traits https://review.openstack.org/#/c/441437/2 | |
| 22:16:44 | mriedem | we can't release w/o one | |
| 22:19:17 | mriedem | i'll poke around about that | |
| 22:23:34 | leakypipes | mriedem: np. gimme a few to get it done. | |
| 22:23:47 | mriedem | https://launchpad.net/os-traits | |
| 22:23:56 | mriedem | i think you need to make me maintainer to keep making changes to it | |
| 22:24:01 | mriedem | i'm going through https://docs.openstack.org/infra/manual/creators.html#set-up-launchpad | |
| 22:24:44 | leakypipes | mriedem: done | |
| 22:25:08 | mriedem | word | |
| 22:27:09 | leakypipes | mriedem: btw, I am now officially within striking^Wthrowing distance of superdan, so if you want me to throw anything at him, just say the word. | |
| 22:27:18 | superdan | mriedem: likewise | |
| 22:27:26 | leakypipes | heh | |
| 22:27:52 | mriedem | hmmm | |
| 22:28:08 | mriedem | bag of dicts | |
| 22:28:20 | mriedem | engage, number one | |
| 22:29:18 | superdan | melwitt: we merged the spec for this quota junk right? | |
| 22:29:39 | melwitt | junk?? quota gems | |
| 22:29:45 | superdan | um, yeah. | |
| 22:29:53 | melwitt | one spec merged, there's still one that needs +W | |
| 22:30:06 | melwitt | https://review.openstack.org/#/c/431748/ | |
| 22:30:16 | superdan | it's seeming pretty gem-like right now as I watch "...ok" slowly scroll across my screen at the speed of an i386-16 | |
| 22:30:32 | melwitt | lol | |
| 22:32:23 | superdan | both the tests that failed this way passed | |
| 22:32:30 | superdan | still waiting for the rest to go, but.. looking good | |
| 22:32:47 | superdan | I'm fairly excited by that fact | |
| 22:33:56 | melwitt | medium excitement | |
| 22:34:14 | superdan | yeah | |
| 22:34:26 | superdan | hot damn, all the scenarios passed except for one that is os-hypervisors relate | |
| 22:34:35 | superdan | so I'd say those patches are as good as gold, no need to review | |
| 22:35:04 | superdan | rightly so | |
| 22:36:57 | melwitt | I'll fix up the top patch first thing next week, since we know now that they're a dependency | |