| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-07-28 | |||
| 16:38:12 | figleaf | leakypipes: sorry, but the nuns beat that stuff into me | |
| 16:38:21 | leakypipes | mriedem: right, and the idea was to not have to call put_allocations() once the placement-claims stuff was done. | |
| 16:38:33 | cdent | I’m thinking in terms of these vaunted “heals” we love. Is a heal going to be correct in the face of a shared provider | |
| 16:38:43 | figleaf | cdent: OIC what you're getting at | |
| 16:39:24 | figleaf | cdent: yeah, that would only work if allocations were consumer/rp specific | |
| 16:39:42 | mriedem | leakypipes: yeah, and my original understanding from 4+ months ago was we'd put code into the computes that wouldn't do anything with allocations if they were already created by the scheduler, and we'd not do claims in the scheduler until all computes had that code | |
| 16:39:48 | mriedem | but then that went away | |
| 16:41:06 | superdan | mriedem: that code _is_ in the computes though | |
| 16:41:28 | superdan | well, part of it | |
| 16:41:32 | mriedem | superdan: the diff thing in the report client you mean right? | |
| 16:41:40 | mriedem | my_allocations vs current_allocations | |
| 16:41:41 | mriedem | ? | |
| 16:41:47 | superdan | yes, but the thing is, | |
| 16:41:55 | mriedem | my_allocations doesn't account for shared storage | |
| 16:41:56 | mriedem | right? | |
| 16:41:57 | superdan | we still have to have the healing from the compute node side regardless | |
| 16:44:36 | leakypipes | mriedem: if you create a bug, I'll get to work on a fix. | |
| 16:45:12 | mriedem | which bug are we talking about? the fact we don't sum allocations when resize to same host in scheduler? | |
| 16:45:36 | mriedem | sounds like multiple bugs | |
| 16:45:44 | mriedem | because of the self-heal issue, or is that the one from yesterday? | |
| 16:46:15 | leakypipes | mriedem: no, sorry, I was referring to addressing the TODOs left by cdent around the shared providers (the links you pasted above to the report.py module) | |
| 16:46:35 | cfriesen_ | leakypipes: you could always just stop supporting resize-to-same-host and let all the users scream. /s | |
| 16:46:51 | leakypipes | cfriesen_: cool with me. | |
| 16:47:03 | melwitt | yeah really. especially since they complain that it doesn't mean "force resize to same host" | |
| 16:47:06 | mriedem | leakypipes: ok, but we also have a bug for the resize to same host wrt the scheduler double fudge goodness right? | |
| 16:47:20 | mriedem | let's call it double fudge now | |
| 16:47:28 | leakypipes | mriedem: yeah :) | |
| 16:47:33 | leakypipes | two different bugs | |
| 16:47:45 | mriedem | ok | |
| 16:47:48 | mriedem | will do in a bit | |
| 16:49:24 | cdent | leakypipes: could you please remind me what the _provider_aggregate_map in in the report client was destined for? | |
| 16:49:57 | leakypipes | mriedem: thanks Matt | |
| 16:50:19 | leakypipes | cdent: precisely the case for determining if the compute node was associated to providers of shared resources. | |
| 16:50:43 | cdent | so is it of use in one of these two bugs? | |
| 16:51:50 | openstack | Launchpad bug 1707252 in OpenStack Compute (nova) "Claims in the scheduler does not account for doubling allocations on resize to same host" [Medium,Confirmed] | |
| 16:51:50 | mriedem | leakypipes: bug the first https://bugs.launchpad.net/nova/+bug/1707252 | |
| 16:51:58 | mriedem | superdan: ^ make sure i made sense in there | |
| 16:52:16 | superdan | hang on, I'm reviewing leakypipes' other patch | |
| 16:55:30 | mriedem | even with the fake rpc sleep thing in https://review.openstack.org/#/c/488500/ i'm not seeing the case that the source node is not in the allocations when it goes to delete the allocations for the instance | |
| 16:57:23 | superdan | mriedem: but we know it can happen | |
| 16:57:49 | superdan | so I guess if you want to just leave it, then that's fine, but it'll be super hard to track down if it really happens | |
| 16:58:18 | superdan | mriedem: yes I think your bug text makes sense | |
| 16:58:38 | openstackgerrit | Sean Dague proposed openstack/nova master: Add cinder keystone client opts to config reference https://review.openstack.org/488530 | |
| 16:59:17 | mriedem | superdan: leave what? the part of my patch that returns if the source node isn't in the current allocations? | |
| 16:59:39 | mriedem | the point was really just safe guarding against that and providing logging in case it happens | |
| 16:59:43 | superdan | mriedem: leave it like not apply your patch | |
| 16:59:49 | mriedem | oh | |
| 16:59:51 | mriedem | yeah | |
| 16:59:58 | mriedem | i figured we'd sort that out later | |
| 17:00:04 | superdan | you said this morning something indicating you might want to not commit this patch | |
| 17:00:07 | superdan | I think we should but... | |
| 17:00:27 | mriedem | i think it would be ok to do it - it's not going to impact performance by pulling the allocations again, because this code only runs from the compute periodic | |
| 17:00:52 | mriedem | i intentionally didn't pass the rp uuid in from the scheduler so the scheduler won't double check | |
| 17:01:09 | superdan | okay | |
| 17:03:54 | openstack | Launchpad bug 1707256 in OpenStack Compute (nova) "Scheduler report client is not account for shared resource providers" [High,Confirmed] | |
| 17:03:54 | mriedem | leakypipes: superdan: cdent: here is the other bug https://bugs.launchpad.net/nova/+bug/1707256 | |
| 17:04:04 | mriedem | omg me fail english | |
| 17:04:08 | superdan | is not account? | |
| 17:04:10 | superdan | mah | |
| 17:04:10 | superdan | oh | |
| 17:04:11 | superdan | god | |
| 17:04:17 | superdan | one for the record books kids | |
| 17:04:31 | superdan | let it be known henceforth that mriedem is not perfect | |
| 17:04:35 | mriedem | i was surrounded by a whirlwind of 6 year old in pink | |
| 17:04:41 | mriedem | rattled me | |
| 17:05:24 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove compatibility code for flavors https://review.openstack.org/460377 | |
| 17:08:55 | cdent | you reall do have my disease mriedem : “compute node things it needs” | |
| 17:09:07 | sdague | mriedem: for one glorious moment, 0 bugs in New state - https://bugs.launchpad.net/nova/+bugs?search=Search&field.status=New | |
| 17:09:15 | mriedem | o.O | |
| 17:09:31 | mriedem | sdague: did you just invalidate everything? :) | |
| 17:09:53 | purplerbot | <cdent> so is it of use in one of these two bugs? [2017-07-28 16:50:43.278883] [n 3edk] | |
| 17:09:53 | purplerbot | <leakypipes> cdent: precisely the case for determining if the compute node was associated to providers of shared resources. [2017-07-28 16:50:19.034601] [n 3rxI] | |
| 17:09:53 | cdent | leakypipes: [t 3rxI] [t 3edk] | |
| 17:10:17 | sdague | I read every new bug, moved a bunch of them to Incomplete with specific questions, found all the ones that really were going to need specs and linked them to the specs process and put them in Opinion | |
| 17:10:20 | sdague | duped a few | |
| 17:10:25 | leakypipes | cdent: yes, the latter. | |
| 17:10:28 | sdague | found some that had been fixed | |
| 17:10:31 | mriedem | sdague: thanks | |
| 17:10:42 | sdague | so, it's mostly a legit cleaning of the New state | |
| 17:11:02 | cdent | cool, thanks leakypipes, I figured as much but wanted to be sure | |
| 17:11:54 | sdague | also... evacuate bugs... zomg | |
| 17:11:59 | melwitt | sdague: you are a champion | |
| 17:12:31 | cdent | gerrit hates me | |
| 17:13:15 | cdent | voila | |
| 17:13:18 | superdan | cdent: hates everyone apparently | |
| 17:13:23 | superdan | *it hates | |
| 17:13:29 | cdent | s/:// | |
| 17:13:37 | superdan | well, I suspected, but | |
| 17:13:49 | melwitt | yeah, gerrit hates us all. it's not you | |
| 17:15:21 | cfriesen_ | review.openstack.org is down? | |
| 17:15:52 | cfriesen_ | never mind, just saw the note | |
| 17:19:46 | openstack | bug 1669054 in OpenStack Compute (nova) "RequestSpec.ignore_hosts from resize is reused in subsequent evacuate" [Undecided,Opinion] https://launchpad.net/bugs/1669054 | |
| 17:19:46 | cfriesen_ | sdague: arguably bug 1669054 is not opinion...if CONF.allow_resize_to_same_host is False and we do a resize then a subsequent rebuild/evacuate, it will continue to ignore the original host. | |
| 17:20:09 | cfriesen_ | sdague: the only "might" is around exactly where to do the fix | |
| 17:23:55 | openstackgerrit | Peter Hamilton proposed openstack/nova-specs master: Add support for certificate validation https://review.openstack.org/488541 | |
| 17:26:34 | openstackgerrit | melanie witt proposed openstack/nova master: Detach device from live domain even if not found on persistent https://review.openstack.org/488545 | |
| 17:34:27 | openstackgerrit | Peter Hamilton proposed openstack/nova-specs master: Add support for certificate validation https://review.openstack.org/488541 | |
| 17:40:16 | openstackgerrit | Chris Dent proposed openstack/nova master: Optional separate database for placement API https://review.openstack.org/362766 | |
| 17:54:28 | openstackgerrit | Dan Smith proposed openstack/nova master: Add some more cellsv2 doc goodness https://review.openstack.org/487183 | |
| 17:59:50 | sdague | cfriesen_: Opinion often means bring it back in via the specs process. Did I not do that with this one? | |