<div dir="ltr"><div><div><div><div>Hi,<br><br></div>I had posted a patch to <a href="https://review.openstack.org/#/c/56820/" target="_blank">https://review.openstack.org/#/c/56820/</a> for implementing <a href="https://bugs.launchpad.net/nova/+bug/1240197" target="_blank">https://bugs.launchpad.net/nova/+bug/1240197</a> 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.<br>

<br></div>I couldn't use something like "ec2_workers=auto" because the worker configurations are defined as-<br><br>cfg.IntOpt('ec2_workers',<br>               help='Number of workers for EC2 API service')<br>

<br></div>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?<br><br></div>Thanks!<br></div><div class="gmail_extra">
<br><br><div class="gmail_quote">On Sun, Oct 27, 2013 at 5:53 PM, Shantanu Tushar Jha <span dir="ltr"><<a href="mailto:shaan7in@gmail.com" target="_blank">shaan7in@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr"><div>Hi everyone,<br><br>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 <a href="https://bugs.launchpad.net/nova/+bug/1240197" target="_blank">#1240197</a> which sounds pretty interesting to implement.<br>

<br></div><div>I made a comment around the approach but it didn't really get any reply, can someone take a look and suggest?<br><br>P.S. If I should be directly emailing the bug reporter instead of posting to the ML, let me know :)<br>

<br></div><div>Thanks!<span class="HOEnZb"><font color="#888888"><br></font></span></div><span class="HOEnZb"><font color="#888888"><div><div><br>-- <br>Shantanu Tushar    (UTC +0530)<br><a href="http://www.shantanutushar.com" target="_blank">http://www.shantanutushar.com</a>
</div></div></font></span></div>
</blockquote></div><br><br clear="all"><br>-- <br>Shantanu Tushar    (UTC +0530)<br><a href="http://www.shantanutushar.com" target="_blank">http://www.shantanutushar.com</a>
</div>