<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, May 19, 2014 at 1:14 AM, Mike Perez <span dir="ltr"><<a href="mailto:thingee@gmail.com" target="_blank">thingee@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 class="">On 02:04 Tue 29 Apr     , Bohai (ricky) wrote:<br>
> Hi stackers,<br>
><br>
> I found there are two "policy.json" files in cinder project.<br>
> One is for source code(cinder/etc/policy.json), another is for the unit test(cinder/cinder/tests/policy.json).<br>
><br>
> Maybe it's better to united them and make the unit test to use the "policy.json" file in the source code:<br>
> 1. "policy.json" in the source code is really what we want to test but not the one in unit test.<br>
> 2. It's more convenient for the developers, because of only need to modify one policy.json file.<br>
>   Current it's easy to miss one of them.<br>
><br>
> Any advices?<br>
<br>
</div>Seems like the right direction. Don't know why they were separate to begin<br>
with.<br></blockquote><div><br><br></div><div>Nova has the same issue so its probably just historical. I'm not familiar with the cinder policy files, but for<br>Nova the default policy settings are different for the real policy file versus the one used for the unittests<br>
</div><div>and the unittests rely on this. So there's likely there will need to be some cleanup required to use just one policy file<br></div><div>and may complicate the unittests a bit more. But overall sounds like a good idea just to have one policy file.<br>
</div><div> <br></div><div>Chris<br></div></div></div></div>