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 <[email protected]>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
_______________________________________________
OpenStack-dev mailing list
[email protected]
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
