Open Stack

On Tue, Jul 21, 2015 at 2:53 PM, Steve Gordon <sgordon at redhat.com> wrote:

> ----- Original Message -----
> > From: "Matt Kassawara" <mkassawara at gmail.com>
> > To: "Atsushi SAKAI" <sakaia at jp.fujitsu.com>
> >
> > We should call these "weight" or "weights" to agree with every other
> > technical thing that uses this concept.
>
> Aren't these configuration values based on the literal strings actually
> used in the code? E.g. (note that weighter appears to be a typo introduced
> in the docs, scheduler_default_weighters doesn't exist anywhere as a
> configuration key in the source code:
>
> sgordon at localhost:~/Source/cinder(master)> grep -r GoodnessWeighter *
> sgordon at localhost:~/Source/cinder(master)> grep -r GoodnessWeigher *
> cinder/tests/unit/scheduler/test_goodness_weigher.py:class
> GoodnessWeigherTestCase(test.TestCase):
> cinder/tests/unit/scheduler/test_goodness_weigher.py:
> super(GoodnessWeigherTestCase, self).setUp()
> cinder/tests/unit/scheduler/test_goodness_weigher.py:        weigher =
> goodness.GoodnessWeigher()
> cinder/tests/unit/scheduler/test_goodness_weigher.py:        weigher =
> goodness.GoodnessWeigher()
> cinder/tests/unit/scheduler/test_goodness_weigher.py:        weigher =
> goodness.GoodnessWeigher()
> cinder/tests/unit/scheduler/test_goodness_weigher.py:        weigher =
> goodness.GoodnessWeigher()
> cinder/tests/unit/scheduler/test_goodness_weigher.py:        weigher =
> goodness.GoodnessWeigher()
> cinder/tests/unit/scheduler/test_goodness_weigher.py:        weigher =
> goodness.GoodnessWeigher()
> cinder/tests/unit/scheduler/test_goodness_weigher.py:        weigher =
> goodness.GoodnessWeigher()
> cinder/tests/unit/scheduler/test_goodness_weigher.py:        weigher =
> goodness.GoodnessWeigher()
> cinder/tests/unit/scheduler/test_goodness_weigher.py:        weigher =
> goodness.GoodnessWeigher()
> cinder/tests/unit/scheduler/test_goodness_weigher.py:        weigher =
> goodness.GoodnessWeigher()
> cinder/scheduler/weights/goodness.py:class
> GoodnessWeigher(weights.BaseHostWeigher):
> setup.cfg:    GoodnessWeigher =
> cinder.scheduler.weights.goodness:GoodnessWeigher
>
> sgordon at localhost:~/Source/cinder(master)> grep -r
> "scheduler_default_weighters" * -r
> sgordon at localhost:~/Source/cinder(master)> grep -r
> "scheduler_default_weighers" * -r
> cinder/tests/unit/scheduler/test_scheduler.py:        #
> scheduler_default_filters and scheduler_default_weighers when
> cinder/tests/unit/scheduler/test_scheduler.py:
>  scheduler_default_weighers=['CapacityWeigher'])
> cinder/tests/unit/scheduler/test_scheduler.py:
> self.assertEqual(CONF.scheduler_default_weighers,
> cinder/tests/unit/scheduler/test_scheduler.py:
>  scheduler_default_weighers=['CapacityWeigher'])
> cinder/tests/unit/scheduler/test_scheduler.py:
> self.assertEqual(CONF.scheduler_default_weighers,
> cinder/scheduler/simple.py:  scheduler_default_weighers = 'ChanceWeigher'
> cinder/scheduler/simple.py:  scheduler_default_weighers =
> 'AllocatedCapacityWeigher'
> cinder/scheduler/host_manager.py:
> cfg.ListOpt('scheduler_default_weighers',
> cinder/scheduler/host_manager.py:
> CONF.set_override('scheduler_default_weighers',
> cinder/scheduler/host_manager.py:
> CONF.set_override('scheduler_default_weighers',
> cinder/scheduler/host_manager.py:        it uses
> CONF.scheduler_default_weighers instead.
> cinder/scheduler/host_manager.py:            weight_cls_names =
> CONF.scheduler_default_weighers
>

Oh, good. Thanks for looking it up Steve.


>
> > On Tue, Jul 21, 2015 at 12:34 AM, Atsushi SAKAI <sakaia at jp.fujitsu.com>
> > wrote:
> >
> > > Hello
> > >
> > > I am looking around following document and find same two keywords are
> used
> > > on the document.
> > > Which word (weigher/weighter) is correct for this case?
> > >
> > > for example
> > >   GoodnessWeigher
> > >   VolumeNumberWeigher scheduler_default_weighters
> > >
> > > OpenStack Cloud Administrator Guide
> > > http://docs.openstack.org/draft/admin-guide-cloud-rst/
> > >
> > > Thanks
> > > Atsushi SAKAI
> > >
> > >
> > > _______________________________________________
> > > OpenStack-docs mailing list
> > > OpenStack-docs at lists.openstack.org
> > > http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-docs
> > >
> >
> > _______________________________________________
> > OpenStack-docs mailing list
> > OpenStack-docs at lists.openstack.org
> > http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-docs
> >
>
> --
> Steve Gordon, RHCE
> Sr. Technical Product Manager,
> Red Hat Enterprise Linux OpenStack Platform
>
> _______________________________________________
> OpenStack-docs mailing list
> OpenStack-docs at lists.openstack.org
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-docs
>



-- 
Anne Gentle
Rackspace
Principal Engineer
www.justwriteclick.com
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstack.org/pipermail/openstack-docs/attachments/20150721/f51d6c21/attachment.html>