| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2017-10-11 | |||
| 17:45:54 | abhishek | Hi all.. can someone please review https://review.openstack.org/#/c/505546/ | |
| 17:52:43 | efried | jaypipes https://review.openstack.org/#/c/377138/ not sure if I'm off base there... | |
| 18:10:54 | openstackgerrit | Jay Pipes proposed openstack/nova master: placement: allow filter providers in tree https://review.openstack.org/377215 | |
| 18:10:54 | openstackgerrit | Jay Pipes proposed openstack/nova master: placement: add nested resource providers https://review.openstack.org/377138 | |
| 18:10:55 | openstackgerrit | Jay Pipes proposed openstack/nova master: placement: update client to set parent provider https://review.openstack.org/385693 | |
| 18:10:55 | openstackgerrit | Jay Pipes proposed openstack/nova master: placement: adds REST API for nested providers https://review.openstack.org/384807 | |
| 18:10:59 | jaypipes | efried: good feedback, thanks. fixed ^^ | |
| 18:11:04 | efried | jaypipes rgr | |
| 18:11:06 | efried | sdague https://review.openstack.org/#/c/490031/ succeeded, so that ought to mean https://review.openstack.org/#/c/490057/ is good to go. (To be extra sure, I'm still waiting for jenkins to vote on https://review.openstack.org/#/c/511006/, which runs the Nova CI with everything in place.) | |
| 18:12:38 | efried | jaypipes Cool man, I think I'm +1 on the whole stack. Let me know if I missed any. | |
| 18:12:59 | jaypipes | efried: thanks for the great reviews. appreciated. | |
| 18:13:05 | efried | yahyoubetcha :) | |
| 18:16:09 | openstackgerrit | Peter Hamilton proposed openstack/nova-specs master: Add support for certificate validation https://review.openstack.org/488541 | |
| 18:28:40 | openstackgerrit | Dan Smith proposed openstack/nova master: Merge build requests into the sortmaster 9000 https://review.openstack.org/508595 | |
| 18:28:41 | openstackgerrit | Dan Smith proposed openstack/nova master: Make get_instance_objects_sorted() be smart about cells https://review.openstack.org/509003 | |
| 18:28:41 | openstackgerrit | Dan Smith proposed openstack/nova master: Add CellMapping.get_by_project_id() query method https://review.openstack.org/509002 | |
| 18:32:16 | mriedem | dansmith: if you're going to fix the indent issue in the bottom change in the series, you could add a todo i pointed out in here https://review.openstack.org/#/c/498949/ | |
| 18:32:21 | mriedem | that would be great mmmkay? | |
| 18:32:58 | melwitt | thaaaaanks | |
| 18:33:38 | dansmith | mriedem: wonkified? honestly matt... | |
| 18:34:47 | dansmith | mriedem: so, on the shared storage thing, | |
| 18:35:00 | dansmith | mriedem: I think that when we do that we'll have the instance hold its shared storage allocation on the new host, | |
| 18:35:10 | dansmith | not duplicate it for the migration one | |
| 18:35:37 | dansmith | the only case would be if we're root-on-shared-ephemeral I guess | |
| 18:36:08 | dansmith | so, I dunno, I'll put a NOTE as I don't really know what the TODO is until later | |
| 18:37:43 | openstackgerrit | Dan Smith proposed openstack/nova master: Revert allocations by migration uuid https://review.openstack.org/498949 | |
| 18:37:43 | openstackgerrit | Dan Smith proposed openstack/nova master: Split get_allocations_for_instance() into useful bits https://review.openstack.org/511306 | |
| 18:37:44 | openstackgerrit | Dan Smith proposed openstack/nova master: Make migration uuid hold allocations for migrating instances https://review.openstack.org/506420 | |
| 18:37:44 | openstackgerrit | Dan Smith proposed openstack/nova master: Refactor resource tracker to account for migration allocations https://review.openstack.org/506419 | |
| 18:37:45 | openstackgerrit | Dan Smith proposed openstack/nova master: Make live migration hold resources with a migration allocation https://review.openstack.org/507638 | |
| 18:39:02 | mriedem | ok | |
| 18:39:20 | mriedem | wonkification of the SchedulerReportClient 2.0 | |
| 18:40:59 | dansmith | heh | |
| 18:55:07 | openstackgerrit | Matt Riedemann proposed openstack/nova-specs master: Spec for flavor description https://review.openstack.org/501017 | |
| 18:55:11 | mriedem | sdague: made those changes on restricting the id and name fields ^ | |
| 18:55:49 | sdague | mriedem: will look | |
| 19:07:00 | openstackgerrit | Merged openstack/nova master: Update cinder in RequestContext service catalog https://review.openstack.org/510947 | |
| 19:12:51 | sdague | mriedem: that looks completely sane to me | |
| 19:13:32 | mriedem | not only sane, but completely sane? | |
| 19:13:33 | mriedem | nice | |
| 19:17:41 | mriedem | huh https://developer.openstack.org/api-ref/placement/#resource-provider-inventories | |
| 19:17:43 | mriedem | is missing POST | |
| 19:37:29 | openstackgerrit | sean mooney proposed openstack/nova master: WIP reorder destination host setup https://review.openstack.org/510658 | |
| 19:41:18 | openstackgerrit | melanie witt proposed openstack/nova stable/ocata: Target context for build notification in conductor https://review.openstack.org/511325 | |
| 19:47:45 | mriedem | easy fix here: https://review.openstack.org/#/c/511209/ | |
| 19:47:55 | mriedem | hot dogs and cracker jacks here! | |
| 19:49:07 | melwitt | +W | |
| 19:50:23 | mriedem | https://bugs.launchpad.net/nova/+bug/1722891 in case someone wants it | |
| 19:50:24 | openstack | Launchpad bug 1722891 in OpenStack Compute (nova) "api-ref: POST /resource_providers/{uuid}/inventories is missing" [Medium,Confirmed] - Assigned to hongbin (hongbin034) | |
| 19:50:57 | openstackgerrit | Hongbin Lu proposed openstack/nova master: Handle not found error on taking snapshot https://review.openstack.org/511074 | |
| 19:52:01 | mriedem | grenade jobs seem totally borked in zuulv3 | |
| 19:53:31 | melwitt | mriedem: looks like hongbin is already gonna work on the bug you linked, so that's cool | |
| 19:55:22 | mriedem | something really weird happens with our request id log handlin | |
| 19:55:27 | mriedem | seems to be not thread safe or something | |
| 19:55:40 | mriedem | i'm tracing a server create request through the n-cpu logs and see the normal stuff i'd expect while creating the instance, | |
| 19:55:52 | mriedem | but then i see the same request id showing up in log messages for periodic tasks that are running | |
| 19:56:52 | melwitt | hm, I can't say I've seen that before | |
| 19:57:25 | melwitt | which makes me think it's a relatively new issue | |
| 20:34:10 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Update "SHUTOFF" description in API guide https://review.openstack.org/510697 | |
| 20:34:10 | openstackgerrit | Matt Riedemann proposed openstack/nova master: api-ref: fix server status values in GET /servers docs https://review.openstack.org/510696 | |
| 20:43:08 | mriedem | uh, filter(~models.Migration.status.in_ means NOT IN right? | |
| 20:43:11 | mriedem | the ~ | |
| 20:47:20 | efried | sdague https://review.openstack.org/#/c/511006/ basically passed - the one failure seems unrelated and I'd rather not choke the already-seemingly-choking gate further. | |
| 20:54:57 | mriedem | dansmith: want to double check my thoughts about migration.status = 'done' in here? https://review.openstack.org/#/c/506419/ otherwise i think it's ok | |
| 21:01:23 | mriedem | this also seems like it could bite us https://github.com/openstack/nova/blob/64635ad4a5f60a79e1ec2d5369a8f84bfeccb7e4/nova/db/sqlalchemy/api.py#L4859-L4862 | |
| 21:01:47 | mriedem | when the RT pulls migration records, it's for all migrations where either the source or dest is our local node | |
| 21:08:13 | dansmith | mriedem: yep will in a sec | |
| 21:20:25 | dansmith | mriedem: replied | |
| 21:22:22 | mriedem | ok didn't think about not having migration allocations for evacs | |
| 21:22:41 | mriedem | but makes sense to not put the source node allocations on the migration record for an evacuation since the source node should be dead | |
| 21:22:53 | dansmith | and you're never going back there.. no rollback | |
| 21:22:58 | mriedem | but...are we doubling today? | |
| 21:23:07 | mriedem | in the scheduler when we evacuate we must be doubling | |
| 21:23:12 | dansmith | if we are it's not for any useful reason I think | |
| 21:23:25 | mriedem | i think it's just because we do it generically | |
| 21:23:28 | dansmith | I think we delete the allocation for the old node first anyway right? | |
| 21:25:25 | dansmith | allocate _for_evacuate_dest_host | |
| 21:26:27 | mriedem | that's when we're forcing the host during evacuate and bypassing the scheduler | |
| 21:26:33 | dansmith | yeah, we end up calling claim_resources and doubling | |
| 21:26:45 | dansmith | no, that method calls the scheduler | |
| 21:26:57 | mriedem | _allocate_for_evacuate_dest_host? | |
| 21:26:59 | mriedem | it doesn't | |
| 21:27:08 | mriedem | it calls scheduler utils | |
| 21:27:11 | dansmith | https://github.com/openstack/nova/blob/master/nova/conductor/manager.py#L782-L782 | |
| 21:27:16 | mriedem | b/c we do the same util thing for live migration with a forced host | |
| 21:27:17 | dansmith | which calls claim_resources | |
| 21:27:29 | dansmith | sorry, maybe it does't call scheduler, but it does the doubling, which is what I meant | |
| 21:27:30 | mriedem | right, that's not select_destinations | |
| 21:27:33 | mriedem | yeah | |
| 21:27:39 | dansmith | sure, I was focused on the allocs | |
| 21:27:44 | mriedem | this is the thing i want to remove with a skip_filters flag to select_destinations | |
| 21:27:56 | mriedem | so getting back to my original question i guess, | |
| 21:27:59 | dansmith | regardless, I don't think this affects the new-world path | |
| 21:28:05 | mriedem | we never remove the allocation from the source node for an evacuation | |
| 21:28:15 | mriedem | except when the source node comes back, if it does | |
| 21:28:18 | openstackgerrit | Hongbin Lu proposed openstack/nova master: placement: add API reference for create inventory https://review.openstack.org/511342 | |
| 21:28:29 | dansmith | you think we never do it now? | |
| 21:28:47 | dansmith | or you think we never will with the new-world way? | |
| 21:28:51 | mriedem | https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L680 | |
| 21:28:55 | dansmith | ah, maybe your point is, | |