Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-08-07
15:54:30 dansmith I would expect to be able to brute-force scale the scheduler as high as you want with just that change alone
15:54:40 openstackgerrit Balazs Gibizer proposed openstack/nova master: replace chance with filter scheduler in func tests https://review.openstack.org/491529
15:54:45 bauzas scale out you mean ?
15:54:54 bauzas or scale up the load ?
15:55:02 dansmith bauzas: scale it to as high of a throughput as you need by scaling out yes
15:55:20 bauzas k
15:55:26 bauzas I agree
15:55:28 gibi we can easily remove chance from the func test. here is the patch: https://review.openstack.org/491529
15:56:13 bauzas yeah
15:56:21 bauzas not a big deal
15:56:35 bauzas anyway I need to leave
15:56:38 cdent gibi: nice
15:56:38 bauzas ++
15:56:46 openstackgerrit Stephen Finucane proposed openstack/nova master: hardware: Flatten functions https://review.openstack.org/367470
15:56:47 openstackgerrit Stephen Finucane proposed openstack/nova master: Rename '_numa_get_constraints_XXX' functions https://review.openstack.org/385072
15:56:47 openstackgerrit Stephen Finucane proposed openstack/nova master: Standardize '_get_XXX_constraints' functions https://review.openstack.org/385071
15:56:48 openstackgerrit Stephen Finucane proposed openstack/nova master: De-duplicate _numa_get_flavor_XXX_map_list https://review.openstack.org/385074
15:56:55 bauzas FWIW we don't even need RetryFilter
15:57:01 bauzas if we are singlehost
15:57:53 sdague so... is someone going deprecate chance completely?
15:58:01 dansmith sdague: and caching
15:58:10 dansmith we need to deprecate both in pike
15:58:10 sdague sure, that too
15:58:16 dansmith we can remove one or both when appropriate
15:58:29 sdague ok, is that up for review? we're in rc week
15:58:55 jaypipes gibi: why the change from 1 vcpu to 2 vcpus? that's weird..
15:59:20 dansmith sdague: are you asking if you can do it?
15:59:24 gibi jaypipes: because there is resize to same host where the allocations are doubled up
15:59:36 gibi jaypipes: so at least 2 vcpu is needed
15:59:41 dansmith gibi: yeah
15:59:47 jaypipes gibi: guh, gotcha.
15:59:49 dansmith makes sense, I had to do that in some of the tests as well
16:00:04 dansmith jaypipes: this is the max_unit thing I was talking about with moves
16:00:05 dansmith jaypipes: that is going to piss people off
16:00:12 dansmith because allocation_ratio won't apply to single-host moves
16:00:46 dansmith migration uuid will fix that
16:00:50 jaypipes dansmith: yeah, understood. not sure it's the end of the world, though.
16:00:59 cdent migration uuid will fix everything!
16:01:02 dansmith eff yeah it's the end of the world
16:01:06 jaypipes :)
16:01:11 dansmith I have a single-node cloud at home!
16:01:26 gibi dansmith: only one? ;)
16:01:52 dansmith gibi: many nodes, but only one in the openstack deployment :)
16:03:17 mnaser i'm going over the scheduler code and i've realized that we're pretty much sending instances to our biggest hypervisors and the smaller ones remain empty. the ram weighter by default weighs based on free memory, which means that a hypervisor with 128gb memory and all of it free will have less priority than one that has 384gb with 192gb free
16:03:39 mnaser arguably.. shouldnt it be based on a percentage?
16:04:03 mnaser https://github.com/openstack/nova/blob/master/nova/scheduler/weights/ram.py
16:04:19 jaypipes mnaser: perhaps use the num instances weigher instead?
16:04:43 dansmith or weigh the num_instances higher than the ram one
16:04:49 dansmith mnaser: some people want to pack first
16:04:50 jaypipes mnaser: nm, there is no such thing :(
16:04:52 dansmith because windows licenses
16:05:10 mnaser dansmith: packing would still work with -1.0
16:05:13 dansmith jaypipes: there's something like that pretty sure
16:05:29 mnaser https://github.com/openstack/nova/tree/master/nova/scheduler/weights
16:05:35 mnaser i dont see a num_instances on
16:05:38 mriedem stephenfin: where does this come from? https://review.openstack.org/#/c/490952/3/doc/source/admin/security-groups.rst
16:05:54 mnaser there's a fitler but not a weighter
16:05:58 stephenfin mriedem: cli-nova-manage-projects-security.rst
16:06:01 stephenfin (I think
16:06:03 mriedem oh nvm https://github.com/openstack/openstack-manuals/blob/stable/ocata/doc/admin-guide/source/cli-nova-manage-projects-security.rst
16:06:03 jaypipes mnaser: yeah :(
16:06:04 mriedem yeah
16:06:18 dansmith mnaser: yeah, that's what I'm thinking of, heh
16:06:47 mnaser cause if it was percentage based, the ability to pack vs distribute would still work, but would be based on usage percentage rather than just absolute free memory
16:07:24 mnaser i'm just slightly annoyed because i remember very clearly reading somewhere online (ML?) that someone had this exact issue and they dealt with it somehow but i figured "we won't deal with that" and here i am
16:08:17 dansmith man I really thought we had more weighers
16:09:40 mriedem stephenfin: so on https://review.openstack.org/#/c/490952/ I think we should have those various volume attach troubleshooting docs
16:10:18 jaypipes mnaser: can't believe I'm suggesting this, but you could adapt the existing RAM weigher and make it work on percentages instead of raw capacity numbers.
16:10:40 jaypipes mnaser: else submit a wishlist bug for one of us to get to.
16:10:57 jaypipes mnaser: like dansmith I'm kinda surprised such a thing doesn't already exist :(
16:11:57 mriedem stephenfin: and maybe leave todo comments for combining the quotas pages
16:13:15 mriedem stephenfin: if you want we can just deal with those comments in a follow up patch
16:14:02 stephenfin mriedem: Nah, I might as well address them there now
16:15:45 stephenfin mriedem: Also, what are your thoughts on https://review.openstack.org/#/c/453808 ? Can it still go in before RC1 if a FFE isn't required/suitable?
16:19:13 mriedem stephenfin: replied again
16:19:22 mriedem i don't see why --version is different from --local_cell
16:19:25 mriedem since they are both options
16:19:36 mriedem plus i feel like this is a distraction right now
16:21:56 mriedem sdague: how is https://review.openstack.org/#/c/490644/ different from https://review.openstack.org/#/c/478485/ ?
16:22:05 mriedem note i can't get to docs-draft.openstack.org/44/490644/3/check/gate-nova-docs-ubuntu-xenial/bd5e01d//doc/build/html/ right now
16:22:45 openstackgerrit Eric Fried proposed openstack/nova master: Get auth from context for glance endpoint https://review.openstack.org/490057
16:23:43 sdague mriedem: the TOC approach
16:24:10 sdague mriedem: see commit message
16:39:05 openstackgerrit Merged openstack/nova master: Add resource utilities to scheduler utils https://review.openstack.org/490514
16:44:01 mriedem sdague: do we need to worry about redirects for anything in https://review.openstack.org/#/c/490644/ ?
16:44:41 mriedem sdague: or is that handled by https://review.openstack.org/#/c/490644/3/doc/source/index.rst@186 ?
16:45:34 mnaser jaypipes do you think proposing a fix to replace the existing ram weighter by percentage would be an acceptable change
16:45:43 mnaser or introduce a new weighter instead?
16:45:59 mnaser because i genuinely think a ram weighter based on free memory only is not correct
16:47:08 mriedem sdague: nvm, looks like nothing actually moves
16:47:13 mriedem so no redirects needed
16:49:03 jaypipes mnaser: a new weigher would be better.
16:50:21 openstackgerrit Ed Leafe proposed openstack/nova master: Fix migrate single instance when it was created concurrently https://review.openstack.org/491439
16:51:26 edleafe bauzas: ^^ just fixed the pep8 issue
16:55:22 sdague mriedem: right, I did not move any source files
16:55:43 sdague I was just trying to build landing pages, and tame the TOC on the left sidebar which is a confusing mess right now
16:56:08 mnaser ehh
16:58:37 dansmith jaypipes: mnaser agreed
16:58:46 dansmith behavioral change may surprise people
16:59:10 mnaser yep, that's what i thought, i'm working on something now

Earlier   Later