Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-10-11
17:38:55 efried But no, they haven't cut over yet.
17:39:17 efried http://eavesdrop.openstack.org/irclogs/%23openstack-infra-incident/%23openstack-infra-incident.2017-10-11.log in case you're interested.
17:39:54 melwitt indeed they know about it in #infra and they're working on it
17:40:12 dansmith so yeah, like I said, it's getting old data because it's not doing the actual merges
17:40:46 melwitt okay. I didn't know how it worked
17:41:27 jaypipes efried, dansmith: k, thx for the info.
17:41:58 openstackgerrit Dan Smith proposed openstack/nova master: Revert allocations by migration uuid https://review.openstack.org/498949
17:41:59 openstackgerrit Dan Smith proposed openstack/nova master: Refactor resource tracker to account for migration allocations https://review.openstack.org/506419
17:41:59 openstackgerrit Dan Smith proposed openstack/nova master: Make migration uuid hold allocations for migrating instances https://review.openstack.org/506420
17:42:00 openstackgerrit Dan Smith proposed openstack/nova master: Make live migration hold resources with a migration allocation https://review.openstack.org/507638
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: add nested resource providers https://review.openstack.org/377138
18:10:54 openstackgerrit Jay Pipes proposed openstack/nova master: placement: allow filter providers in tree https://review.openstack.org/377215
18:10:55 openstackgerrit Jay Pipes proposed openstack/nova master: placement: adds REST API for nested providers https://review.openstack.org/384807
18:10:55 openstackgerrit Jay Pipes proposed openstack/nova master: placement: update client to set parent provider https://review.openstack.org/385693
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: Add CellMapping.get_by_project_id() query method https://review.openstack.org/509002
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: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: Split get_allocations_for_instance() into useful bits https://review.openstack.org/511306
18:37:43 openstackgerrit Dan Smith proposed openstack/nova master: Revert allocations by migration uuid https://review.openstack.org/498949
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:44 openstackgerrit Dan Smith proposed openstack/nova master: Make migration uuid hold allocations for migrating instances https://review.openstack.org/506420
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: api-ref: fix server status values in GET /servers docs https://review.openstack.org/510696
20:34:10 openstackgerrit Matt Riedemann proposed openstack/nova master: Update "SHUTOFF" description in API guide https://review.openstack.org/510697
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

Earlier   Later