Open Stack

Hi,

I had posted a patch to https://review.openstack.org/#/c/56820/ for
implementing https://bugs.launchpad.net/nova/+bug/1240197 and received a
review. I agree with the review that its a bad idea to change default
number of worker processes from 0 to CPU_COUNT, which my patch really does.

I couldn't use something like "ec2_workers=auto" because the worker
configurations are defined as-

cfg.IntOpt('ec2_workers',
               help='Number of workers for EC2 API service')

So, does it make sense changing these to StrOpt to detect the string "auto"
and whenever it is actually configured with a number, convert it to int and
use it?

Thanks!


On Sun, Oct 27, 2013 at 5:53 PM, Shantanu Tushar Jha <shaan7in at gmail.com>wrote:

> Hi everyone,
>
> I'm trying to get started with contributing to OpenStack in my free time.
> Going through the list of low-hanging-fruit bugs I found #1240197<https://bugs.launchpad.net/nova/+bug/1240197>which sounds pretty interesting to implement.
>
> I made a comment around the approach but it didn't really get any reply,
> can someone take a look and suggest?
>
> P.S. If I should be directly emailing the bug reporter instead of posting
> to the ML, let me know :)
>
> Thanks!
>
> --
> Shantanu Tushar    (UTC +0530)
> http://www.shantanutushar.com
>



-- 
Shantanu Tushar    (UTC +0530)
http://www.shantanutushar.com
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstack.org/pipermail/openstack-dev/attachments/20140106/40c44463/attachment.html>