Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-07-27
20:13:31 mriedem and i think revert on the source doesn't do anything either, since it's already got resources claimed
20:13:35 mriedem so there is nothing to unclaim
20:14:10 bauzas folks, I will have to bail out, but I'll look at the IRC channel tomorrow morning
20:14:40 mriedem i'm just about to push a change to add some logging and crap in the scheduler.reportclient.delete_allocations_for_instance to sanity check the allocations before we blow them away, to at least see if we're hitting weird stuff in there during migrate tests
20:14:43 mriedem bauzas: o/
20:14:44 bauzas fer sur, if you need my review, lemme know
20:15:07 dansmith jaypipes: ugh
20:15:08 openstackgerrit OpenStack Proposal Bot proposed openstack/python-novaclient master: Updated from global requirements https://review.openstack.org/488125
20:15:47 sdague efried: link me
20:16:06 efried sdague https://review.openstack.org/488137 bam
20:16:10 efried sdague TIA.
20:16:26 sdague mriedem: https://review.openstack.org/#/c/487860/ - nova-manage list_cells enhancement
20:16:30 sdague with working tests
20:16:36 sdague I will keep bugging you about it :)
20:16:36 jaypipes dansmith, mriedem: so this means really the only thing we can do during confirm_resize() (since it's on the source host) is recalculate the allocation (which will be the doubled-up thing) on the source host RT and remove all entries in the allocation set that refer to the source compute host UUID
20:18:03 dansmith jaypipes: yeah
20:18:16 dansmith jaypipes: I was thinking something different, but that's smarter :)
20:19:39 sdague efried: I'm surprised this is 'glance' and not 'image' - https://review.openstack.org/#/c/488137/1/nova/image/glance.py@127
20:20:39 efried sdague It's the conf group name, which needs to correspond to the project name.
20:20:46 openstackgerrit Matt Riedemann proposed openstack/nova master: Sanity check delete_allocation_for_instance https://review.openstack.org/488187
20:20:53 mriedem dansmith: jaypipes: cdent: ^
20:20:58 efried sdague Which we then look up in service-types-authority to get the service_type, which is indeed `image`
20:20:58 mriedem just for testing at this point
20:21:27 sdague efried: ok, I was surprised we couldn't just call it image to start with, but if that's how it is, that's fine
20:21:36 jaypipes mriedem: coo.
20:21:56 sdague efried: do we have a test job running this with api_servers not set in devstack?
20:22:18 mriedem jaypipes: "is recalculate the allocation (which will be the doubled-up thing) on the source host RT and remove all entries in the allocation set that refer to the source compute host UUID" is i thought what dansmith and i were talking about earlier,
20:22:25 mriedem which is similar to what my patch is checkingfor
20:22:43 efried sdague Yeah, now that you're saying it, I admit it feels a tad weird. But the point is that nova.utils.get_endpoint_data needs to be able to use that param to find the appropriate conf to load, as well as to find the service_type if it's not specified in the conf.
20:22:48 dansmith mriedem: well, I was assuming we could do it on the destination host
20:23:03 dansmith mriedem: but it doesn't really matter, so yes it's pretty much what we were saying
20:23:05 mriedem ok, i wasn't - i was thinking this was purely source host
20:23:11 sdague efried: yeh, it would be nice in the future if we could just specify "image" as well
20:23:12 dansmith well, you're just smarter than us
20:23:26 mriedem heh
20:23:52 mriedem i wouldn't go that far
20:23:55 jaypipes I would
20:24:00 efried sdague You can specify it in the conf: [glance] service_type = image
20:24:01 jaypipes in any case, I'm on it.
20:24:02 dansmith hey
20:24:34 efried sdague I think you're saying you want to specify the service type directly to nova.utils.get_endpoint_data
20:25:04 mriedem dansmith: was there more to that hey or just that your feelings were hurt?
20:25:27 efried sdague That would get confusing if the operator did in fact specify [glance] service_type = <whatever>
20:25:27 mriedem dansmith: is this similar to what you were thinking? https://review.openstack.org/#/c/488187/1/nova/scheduler/client/report.py@1085
20:25:29 dansmith mriedem: like, it's okay for me to say you're smarter than me, but not okay for jaypipes to say it
20:25:35 mriedem oh i get it
20:26:00 jaypipes everyone's smarter than me
20:26:02 mriedem feel free to compliment me on my ability to dig up useless pop trivia
20:26:08 mriedem but not my smarts in general
20:26:23 mriedem laura had to explain her work schedule to me for this weekend at least 4 times
20:26:34 dansmith heh
20:27:16 sdague efried: ah
20:27:29 sdague efried: I didn't realize people were allowed to override these
20:27:40 sdague efried: what's the use case there?
20:27:51 dansmith mriedem: yes, that's similar to what I was thinking
20:27:54 openstackgerrit OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/488034
20:28:09 sdague efried: anyway, on this patch, I think it looks overall good, I want to see a devstack run with api_servers not set to see it working
20:28:23 sdague after which I'll +2
20:28:31 sdague and I'll leave the rest of my questions for mordred
20:28:36 sdague and you at later dates
20:28:42 efried sdague Well, the overall use case is to consolidate/centralize/consistentify (look it up) the way we get clients.
20:29:09 efried sdague So for glance it might be a no-brainer that the service type should always be 'image'.
20:29:13 melwitt mriedem: what type of things will be allowed after today feature freeze? quota cleanups (like test coverage, removing unused stuff, changing the name of recheck_quota config option) or just bug fixes?
20:29:30 efried sdague But we want to be able to do it more or less the same way for e.g. cinder, which (egads) is nowhere near as simple.
20:30:15 sdague efried: yeh, the cinder edge case definitely is a thing.
20:30:50 efried sdague If you want a leetle preview of what that *might* look like: https://review.openstack.org/#/c/487621/1/nova/volume/cinder.py
20:31:02 mriedem melwitt: test coverage is obviously ok, and removing dead code
20:31:27 melwitt k
20:32:14 mriedem jaypipes: let me know before you push anything since i'd like to see a run with https://review.openstack.org/#/c/488187/
20:32:43 mriedem sdague: i think the list_cells thing will have to be a next week thing
20:32:48 mriedem brain is fried on placement stuff today
20:33:05 openstackgerrit OpenStack Proposal Bot proposed openstack/python-novaclient master: Updated from global requirements https://review.openstack.org/488125
20:33:28 sdague mriedem: that's fine, as long as we can still move it in
20:39:49 jaypipes mriedem: will do.
20:40:28 mriedem should have results in ~15 minutes for the live migration job
20:50:50 mriedem sdague: looks like https://review.openstack.org/#/c/487246/ broke the powervm in tree CI
20:50:55 mriedem efried: thorst: ^
20:51:03 mriedem i'm seeing that here http://184.172.12.213/87/488187/1/check/nova-in-tree-pvm/9c61778/logs/
20:51:17 mriedem openstack --os-cloud devstack-admin --os-region RegionOne compute service list --host powervm-ci-powervm-devstacked-28324 --service nova-compute -c ID -f value
20:51:19 efried mriedem looking
20:52:38 mriedem http://logs.openstack.org/46/487246/2/check/gate-tempest-dsvm-ironic-ipa-wholedisk-bios-agent_ipmitool-tinyipa-ubuntu-xenial-nv/bec38d6/logs/devstacklog.txt.gz#_2017-07-26_11_55_57_117
20:52:38 mriedem shit and it killed ironic
20:52:58 sdague mriedem: ah... gah
20:53:37 sdague right, all the compute for multiple hosts drivers
20:54:13 sdague mriedem: ok, hitting revert
20:54:48 mriedem ack
20:54:56 mriedem the one time i actually look at in-tree powervm CI results :)
20:54:59 sdague heh
20:55:13 sdague you get credit for finding it first
20:55:16 mriedem yay
20:55:21 mriedem i have notified the ironicers
20:55:33 sdague well the revert is submitted and approved
20:55:49 mriedem yeah i see it
20:56:17 sdague so... actually, why isn't the host set that way on powervm and ironic?
20:56:32 openstackgerrit Ed Leafe proposed openstack/nova master: Handle hash ring rebalancing in ironic flavor migration https://review.openstack.org/487954
20:56:48 sdague efried: you have a powervm setup somewhere that you can query?
20:57:01 efried esberglu needs to be involved here.
20:59:17 edleafe dansmith: ^^ incorporated rloo's suggestions
20:59:47 efried That powervm failure *might* be unrelated. We shouldn't be trying to connect to localhost.
21:00:01 efried sdague Did that change *when* the compute service gets started?

Earlier   Later