| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-03-03 | |||
| 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 | |
| 22:37:05 | superdan | melwitt: so I'll fix your pep8 error and resubmit all these together if it's cool | |
| 22:37:06 | superdan | heh | |
| 22:37:19 | superdan | I have them all rebased on top, I might as well just do it if that's cool | |
| 22:37:25 | melwitt | yeah, sure | |
| 22:37:33 | superdan | and I'll name the branch moar-cells or something | |
| 22:38:09 | melwitt | okay. yeah, I was referring to the fixups that are part of johnthetubaguy's feedback from when he reviewed it | |
| 22:38:14 | superdan | dtp: so I'm just about out of other stuff, so if you haven't started on the hypervisors stuff, I'll steal your lock on monday | |
| 22:38:15 | superdan | oh | |
| 22:38:21 | superdan | melwitt: okay well I'll hold off them | |
| 22:38:38 | openstackgerrit | Maciej Szankin proposed openstack/nova-specs master: Add use-local-scheduler spec https://review.openstack.org/438936 | |
| 22:39:09 | melwitt | k | |
| 22:39:21 | dtp | ok superdan | |
| 22:42:16 | mriedem | leakypipes: ok i think this is good now https://launchpad.net/os-traits | |
| 22:42:22 | mriedem | os-traits-drivers is you and me atm | |
| 22:42:28 | mriedem | and openstack-admins, plus should be nova-drivers at some point | |
| 22:42:33 | mriedem | os-traits-bugs is the bugs team | |
| 22:42:36 | mriedem | which is also the nova-bugs team | |
| 22:49:45 | leakypipes | mriedem: cool with me. | |
| 22:51:04 | mriedem | well i love paperwork as much as the next person but this has worn me out and it's time to go home | |
| 23:35:38 | openstackgerrit | Maciej Szankin proposed openstack/nova master: conf: remove console_driver opt https://review.openstack.org/441485 | |
| #openstack-nova - 2017-03-04 | |||
| 02:21:48 | openstackgerrit | Peter Hamilton proposed openstack/nova-specs master: Add support for certificate validation https://review.openstack.org/441538 | |
| 03:10:55 | openstackgerrit | Jay Pipes proposed openstack/nova master: virt: add get_inventory() virt driver API method https://review.openstack.org/441543 | |