| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-02-03 | |||
| 00:19:30 | mriedem | jroll: i think it will reschedule https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L2015 | |
| 00:20:18 | jroll | mriedem: maaaybe, that's a lot of except blocks | |
| 00:20:21 | mriedem | yeah we call spawn here https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L1927 | |
| 00:20:27 | mriedem | ironicclient returns node not found | |
| 00:20:36 | mriedem | which gets handled here https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L2008 | |
| 00:21:06 | jroll | huh, I swear it said something about not rescheduling in the bug | |
| 00:21:08 | jroll | I could be drunk | |
| 00:21:24 | mriedem | he said it didn't cleanup in a periodic run | |
| 00:21:29 | mriedem | but i could also be drunk | |
| 00:21:36 | jroll | yeah I guess not | |
| 00:22:06 | jroll | I think it's fine | |
| 00:22:15 | openstackgerrit | melanie witt proposed openstack/nova master: WIP Add functional test for boot-from-volume instances https://review.openstack.org/428505 | |
| 00:23:09 | melwitt | dansmith: ^ I put the RT change into there for testing. the instance create works but then fails the resize bc it's requesting disk | |
| 00:23:57 | dansmith | yeah was just looking at it | |
| 00:24:09 | dansmith | I've got a little bit more time and then I'm really out | |
| 00:24:15 | dansmith | mriedem: when do you cut the tag? | |
| 00:26:14 | openstackgerrit | Ed Leafe proposed openstack/nova master: Delete a compute node's resource provider when node is deleted https://review.openstack.org/428375 | |
| 00:26:29 | edleafe | mriedem: ^^ un-rebased | |
| 00:28:13 | melwitt | dansmith: it doesn't look like mriedem put it on the RC1 things etherpad. maybe we could hope for RC2? since other stuff is slated for it also anyway | |
| 00:28:32 | dansmith | melwitt: I wasn't asking for this, I assume this isn't going to make it | |
| 00:28:38 | edleafe | jroll: and while this was noticed by ironic, it aaplies to any compute node | |
| 00:28:45 | melwitt | dansmith: oh, okay. yeah | |
| 00:28:51 | jroll | edleafe: damnit I have draft comments | |
| 00:28:56 | dansmith | melwitt: until your claim thing, I was going to say it looked pretty backportable, but.. | |
| 00:29:32 | edleafe | jroll: gerrit needs a "move to latest PS" button | |
| 00:29:51 | jroll | or a "don't let someone update this patch for a sec" button | |
| 00:29:53 | jroll | :P | |
| 00:30:32 | melwitt | dansmith: yeah. it would still be technically, no db messings up. assuming that legacy request spec doesn't ever get saved anywhere but if it did, that would be a problem anyway | |
| 00:31:04 | edleafe | Ah, I don't mind commenting on a stale patch. I really hate it when I write these deep comments, and then find that someone beat me to it, and mine sound like "me too" | |
| 00:31:06 | dansmith | melwitt: but the scheduler would start sending instances to computes without knowing if they would agree to not count root_gb, if you updated scheduler first | |
| 00:31:28 | dansmith | melwitt: so no architectural blocks, but still probably not free of bumbs | |
| 00:31:32 | dansmith | er, bumps heh :) | |
| 00:31:57 | melwitt | oh, hm | |
| 00:32:13 | jroll | edleafe: yeah, I'm kidding :) | |
| 00:32:16 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Add release note for filter/sort whitelist https://review.openstack.org/428285 | |
| 00:32:16 | dansmith | I'm projecting a little of our OSP backport rules here, but still | |
| 00:32:34 | jroll | edleafe: I'm investigating that big comment to be sure, but have a few there for you | |
| 00:32:35 | edleafe | jroll: heh, but I'm not :) | |
| 00:32:39 | mriedem | laura kicked my ass into eating dinner with her so i'll be back in a bit | |
| 00:32:45 | mriedem | and all questions will be answered | |
| 00:33:40 | melwitt | dansmith: yeah. it takes me awhile to understand the backport implications to upgrades | |
| 00:33:53 | dims_ | mriedem : better listen to her :) | |
| 00:34:15 | dansmith | melwitt: wish you had brought this up earlier, but the actual patch for this request to the scheduler took like four weeks longer than expected, so :? | |
| 00:34:18 | dansmith | er -> :/ | |
| 00:34:58 | edleafe | jroll: doh! Yeah, the way that the merge conflicts were marked in git didn't help | |
| 00:35:08 | edleafe | cleaning that up now | |
| 00:35:15 | jroll | edleafe: yeah no worries | |
| 00:35:26 | jroll | the main problem is the get_by_host thing | |
| 00:35:38 | jroll | I think it should likely be get_by_host_and_node | |
| 00:36:03 | melwitt | dansmith: yeah, I should have been paying more attention to that. I didn't know whether the bfv thing was planned to be handled this cycle. and cells v2 madness occurred, yadayada | |
| 00:39:36 | dansmith | melwitt: yeah | |
| 00:40:19 | edleafe | jroll: the get_by_host_and_node makes sense. I was using what jaypipes suggested, but I think he typed that off the top of his head. | |
| 00:41:05 | jroll | edleafe: the host/node thing is weird when you aren't an ironic person :) | |
| 00:41:45 | edleafe | jroll: and I thought it was always weird | |
| 00:41:54 | dansmith | edleafe: and you would be right | |
| 00:42:28 | jroll | edleafe: true that | |
| 00:42:36 | jroll | I gotta bounce for the night, I'll look again tomorrow | |
| 00:43:19 | edleafe | jroll: thanks | |
| 00:57:29 | mriedem | dansmith: melwitt: let's put the scheduler BFV patch or whatever at the bottom of the etherpad in the nice to haves list | |
| 00:58:39 | dansmith | I still have 21 failures without getting into unshelve | |
| 00:58:50 | dansmith | and I'm about to drop off for the evening | |
| 00:58:51 | dansmith | but, sure. | |
| 00:59:30 | mriedem | ultimately it's going to depend on if we can backport it | |
| 01:02:51 | mriedem | gmann: ok i think https://review.openstack.org/#/c/428285/ is good to go | |
| 01:02:53 | mriedem | oomichi: ^ | |
| 01:03:46 | mriedem | http://docs-draft.openstack.org/85/428285/3/check/gate-nova-releasenotes/aa1d391//releasenotes/build/html/unreleased.html#upgrade-notes | |
| 01:04:32 | oomichi | mriedem: terminated_at is valid, not terminate_at :) | |
| 01:04:53 | mriedem | ? | |
| 01:05:05 | mriedem | terminate_at isn't in there | |
| 01:05:10 | mriedem | i already fixed that bug yesterday | |
| 01:05:32 | oomichi | mriedem: yeah, by you. LGTM for the reno | |
| 01:05:55 | mriedem | just sign with your +W | |
| 01:07:04 | oomichi | yep, done | |
| 01:07:19 | mriedem | great, thanks | |
| 01:13:17 | gmann | mriedem: +1, all grep and extra check good now | |
| 01:13:33 | edleafe | jroll: thanks | |
| 01:13:37 | edleafe | doh | |
| 01:13:51 | mriedem | i'm going to tag the highest for ocata microversions in the docs too, for compute and placement | |
| 01:15:07 | gmann | +1 | |
| 01:15:49 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Mark compute/placement REST API max microversions for Ocata https://review.openstack.org/428528 | |
| 01:16:07 | mriedem | oomichi: ^ | |
| 01:17:31 | oomichi | mriedem: so will you -2 patches which increase microversion in this cycle? | |
| 01:17:42 | oomichi | if so, +2 on the patch | |
| 01:18:14 | gmann | after that branch going to be made right | |
| 01:18:18 | melwitt | dansmith, mriedem: what are the 21 failures? I don't see that on the jenkins run | |
| 01:18:23 | mriedem | we won't have any other microversions in ocata | |
| 01:18:29 | gmann | yea | |
| 01:18:48 | oomichi | ok, +2 | |
| 01:19:00 | dansmith | melwitt: 21 failures after I've started making the other functions do the same | |
| 01:19:38 | openstackgerrit | Dan Smith proposed openstack/nova master: WIP Add functional test for boot-from-volume instances https://review.openstack.org/428505 | |
| 01:19:38 | openstackgerrit | Dan Smith proposed openstack/nova master: WIP Request zero root disk for boot-from-volume instances https://review.openstack.org/428481 | |
| 01:19:42 | dansmith | melwitt: that ^ | |
| 01:19:46 | dansmith | and .. I'm out | |
| 01:20:25 | melwitt | oh, gotcya | |
| 01:20:39 | melwitt | that means ... gotcha and cya at the same time | |
| 01:22:10 | mriedem | we need more depends-ons https://review.openstack.org/#/c/428531/ | |
| 01:25:02 | openstackgerrit | Maciej Szankin proposed openstack/nova master: conf: Resolve TODOs in 'database' https://review.openstack.org/393695 | |
| 01:29:33 | mriedem | melwitt: want to send this in https://review.openstack.org/#/c/428528/ ? | |
| 01:30:12 | openstackgerrit | Ed Leafe proposed openstack/nova master: Delete a compute node's resource provider when node is deleted https://review.openstack.org/428375 | |
| 01:30:51 | melwitt | mriedem: k | |