<div dir="ltr"><br><div>Dan,</div><div><br></div><div><span style="color:rgb(83,83,83);font-family:'andale mono','lucida console',monospace;font-size:12px;line-height:18px;white-space:pre-wrap">> ></span><i style="color:rgb(83,83,83);font-family:'andale mono','lucida console',monospace;font-size:12px;line-height:18px;white-space:pre-wrap"> If some kind soul can guide me on adding unit tests - per my question</i><br>

<span style="color:rgb(83,83,83);font-family:'andale mono','lucida console',monospace;font-size:12px;line-height:18px;white-space:pre-wrap">> ></span><i style="color:rgb(83,83,83);font-family:'andale mono','lucida console',monospace;font-size:12px;line-height:18px;white-space:pre-wrap"> below - then I'll add them, otherwise I'll just complete the fix for now</i><br>

<span style="color:rgb(83,83,83);font-family:'andale mono','lucida console',monospace;font-size:12px;line-height:18px;white-space:pre-wrap">> ></span><i style="color:rgb(83,83,83);font-family:'andale mono','lucida console',monospace;font-size:12px;line-height:18px;white-space:pre-wrap"> and add the tests in a later change.</i><br>

<font color="#535353" face="andale mono, lucida console, monospace"><span style="font-size:11.818181991577148px;line-height:17.99715805053711px;white-space:pre-wrap">> <br></span></font><span style="color:rgb(83,83,83);font-family:'andale mono','lucida console',monospace;font-size:12px;line-height:18px;white-space:pre-wrap">> The change really needs to come with tests. A fix is only good if we</span><br>

<span style="color:rgb(83,83,83);font-family:'andale mono','lucida console',monospace;font-size:12px;line-height:18px;white-space:pre-wrap">> know it's a fix :)</span><br></div><div><span style="color:rgb(83,83,83);font-family:'andale mono','lucida console',monospace;font-size:12px;line-height:18px;white-space:pre-wrap"><br>

</span></div><div><span style="color:rgb(83,83,83);font-family:'andale mono','lucida console',monospace;font-size:12px;line-height:18px;white-space:pre-wrap">I pulled out the above lines.</span></div><div>

<br></div><div>OK, I'll plead for help on #openstack-dev/#openstack-nova.</div><div><br></div><div>Let me restate here though (where I can explain context) that I don't know where to implement such tests and I can only implement them if I find a related test which I understand sufficiently to modify.</div>

<div><br></div><div>I looked at </div><div><br></div><div>I've looked at the tests in <span style="font-size:12.727272033691406px;color:rgb(80,0,80);font-family:arial,sans-serif">nova/tests/</span><span style="font-size:12.727272033691406px;color:rgb(80,0,80);font-family:arial,sans-serif">compute/test_compute_api.py.</span></div>

<div><span style="font-size:12.727272033691406px;color:rgb(80,0,80);font-family:arial,sans-serif">In particular at </span><span style="font-size:12.727272033691406px;color:rgb(80,0,80);font-family:arial,sans-serif">test_create_quota_exceeded_</span><span style="font-size:12.727272033691406px;color:rgb(80,0,80);font-family:arial,sans-serif">messages which I half understand (literally).</span></div>

<div><span style="font-size:12.727272033691406px;color:rgb(80,0,80);font-family:arial,sans-serif"><br></span></div><div><span style="font-size:12.727272033691406px;color:rgb(80,0,80);font-family:arial,sans-serif">I don't know if this is the appropriate place to add tests.</span></div>

<div><span style="font-size:12.727272033691406px;color:rgb(80,0,80);font-family:arial,sans-serif">If it is I don't see how to modify quota (correctly) to create my test conditions.</span></div><div><span style="font-size:12.727272033691406px;color:rgb(80,0,80);font-family:arial,sans-serif"><br>

</span></div><div><span style="font-size:12.727272033691406px;color:rgb(80,0,80);font-family:arial,sans-serif">... and if it isn't the correct place I of course don't know where is ...</span></div><div><br></div>
<div>
Thanks *anyone*,</div><div>Mike.</div><div><br></div><div><br></div><span style="color:rgb(83,83,83);font-family:'andale mono','lucida console',monospace;font-size:12px;line-height:18px;white-space:pre-wrap">>></span><i style="color:rgb(83,83,83);font-family:'andale mono','lucida console',monospace;font-size:12px;line-height:18px;white-space:pre-wrap"> As I don't see how to "keep it in the review", I'll copy to openstack-dev.</i><br>

<font color="#535353" face="andale mono, lucida console, monospace"><span style="font-size:11.818181991577148px;line-height:17.99715805053711px;white-space:pre-wrap">> <br></span></font><span style="color:rgb(83,83,83);font-family:'andale mono','lucida console',monospace;font-size:12px;line-height:18px;white-space:pre-wrap">> Just keep making your comments in Gerrit. That way all the discussion</span><br>

<span style="color:rgb(83,83,83);font-family:'andale mono','lucida console',monospace;font-size:12px;line-height:18px;white-space:pre-wrap">> related to a specific patch is preserved with proper linkage in case we</span><br>

<span style="color:rgb(83,83,83);font-family:'andale mono','lucida console',monospace;font-size:12px;line-height:18px;white-space:pre-wrap">> ever need to go back to it.</span><div><font color="#535353" face="andale mono, lucida console, monospace"><span style="font-size:11.818181991577148px;line-height:17.99715805053711px;white-space:pre-wrap"><br>

</span></font></div><div><font color="#535353" face="andale mono, lucida console, monospace"><span style="font-size:11.818181991577148px;line-height:17.99715805053711px;white-space:pre-wrap">OK, I suppose you mean the comment made when I perform a 'git review'.</span></font></div>

<div><font color="#535353" face="andale mono, lucida console, monospace"><span style="font-size:11.818181991577148px;line-height:17.99715805053711px;white-space:pre-wrap">Nothing is obvious for me I'm afraid ;-)</span></font></div>

<div><font color="#535353" face="andale mono, lucida console, monospace"><span style="font-size:11.818181991577148px;line-height:17.99715805053711px;white-space:pre-wrap"><br></span></font><span style="color:rgb(83,83,83);font-family:'andale mono','lucida console',monospace;font-size:12px;line-height:18px;white-space:pre-wrap">> ></span><i style="color:rgb(83,83,83);font-family:'andale mono','lucida console',monospace;font-size:12px;line-height:18px;white-space:pre-wrap"> OK, I think I see what I need to do to do to not abandon the current</i><br>

<span style="color:rgb(83,83,83);font-family:'andale mono','lucida console',monospace;font-size:12px;line-height:18px;white-space:pre-wrap">> ></span><i style="color:rgb(83,83,83);font-family:'andale mono','lucida console',monospace;font-size:12px;line-height:18px;white-space:pre-wrap"> change <<a href="https://review.openstack.org/#/c/51263/" style="margin:0px;padding:0px;border:0px;font-style:inherit;font-size:12px;font-family:inherit;vertical-align:baseline;color:rgb(188,21,24);text-decoration:none">https://review.openstack.org/#/c/51263/</a>> (i.e. to merge with</i><br>

<span style="color:rgb(83,83,83);font-family:'andale mono','lucida console',monospace;font-size:12px;line-height:18px;white-space:pre-wrap">> ></span><i style="color:rgb(83,83,83);font-family:'andale mono','lucida console',monospace;font-size:12px;line-height:18px;white-space:pre-wrap"> master whilst not breaking my DevStack) - I'll be back for help if I</i><br>

<span style="color:rgb(83,83,83);font-family:'andale mono','lucida console',monospace;font-size:12px;line-height:18px;white-space:pre-wrap">> ></span><i style="color:rgb(83,83,83);font-family:'andale mono','lucida console',monospace;font-size:12px;line-height:18px;white-space:pre-wrap"> muck up!</i><br>

<font color="#535353" face="andale mono, lucida console, monospace"><span style="font-size:11.818181991577148px;line-height:17.99715805053711px;white-space:pre-wrap">> <br></span></font><span style="color:rgb(83,83,83);font-family:'andale mono','lucida console',monospace;font-size:12px;line-height:18px;white-space:pre-wrap">> Probably just:</span><br>

<font color="#535353" face="andale mono, lucida console, monospace"><span style="font-size:11.818181991577148px;line-height:17.99715805053711px;white-space:pre-wrap">> <br></span></font><span style="color:rgb(83,83,83);font-family:'andale mono','lucida console',monospace;font-size:12px;line-height:18px;white-space:pre-wrap">>  $ git fetch origin</span><br>

<span style="color:rgb(83,83,83);font-family:'andale mono','lucida console',monospace;font-size:12px;line-height:18px;white-space:pre-wrap">>  $ git rebase origin/master</span><br><font color="#535353" face="andale mono, lucida console, monospace"><span style="font-size:11.818181991577148px;line-height:17.99715805053711px;white-space:pre-wrap">> <br>

</span></font><span style="color:rgb(83,83,83);font-family:'andale mono','lucida console',monospace;font-size:12px;line-height:18px;white-space:pre-wrap">> is the safest thing to do.</span></div><div><font color="#535353" face="andale mono, lucida console, monospace"><span style="font-size:11.818181991577148px;line-height:17.99715805053711px;white-space:pre-wrap"><br>

</span></font></div><div><font color="#535353" face="andale mono, lucida console, monospace"><span style="font-size:11.818181991577148px;line-height:17.99715805053711px;white-space:pre-wrap">Thanks I'll note that (though my concern was more to do with DevStack's  stopping/starting/pulling of code - but I got it done OK).</span></font></div>

<div><font color="#535353" face="andale mono, lucida console, monospace"><span style="font-size:11.818181991577148px;line-height:17.99715805053711px;white-space:pre-wrap"> <br></span></font><span style="color:rgb(83,83,83);font-family:'andale mono','lucida console',monospace;font-size:12px;line-height:18px;white-space:pre-wrap">> ></span><i style="color:rgb(83,83,83);font-family:'andale mono','lucida console',monospace;font-size:12px;line-height:18px;white-space:pre-wrap"> There was no test failure, the existing tests have nothing to detect</i><br>

<span style="color:rgb(83,83,83);font-family:'andale mono','lucida console',monospace;font-size:12px;line-height:18px;white-space:pre-wrap">> ></span><i style="color:rgb(83,83,83);font-family:'andale mono','lucida console',monospace;font-size:12px;line-height:18px;white-space:pre-wrap"> whether the fix for </i><br>

<span style="color:rgb(83,83,83);font-family:'andale mono','lucida console',monospace;font-size:12px;line-height:18px;white-space:pre-wrap">> ></span><i style="color:rgb(83,83,83);font-family:'andale mono','lucida console',monospace;font-size:12px;line-height:18px;white-space:pre-wrap">     Bug #1224453 “min_count ignored for instance create” : Bugs :</i><br>

<span style="color:rgb(83,83,83);font-family:'andale mono','lucida console',monospace;font-size:12px;line-height:18px;white-space:pre-wrap">> ></span><i style="color:rgb(83,83,83);font-family:'andale mono','lucida console',monospace;font-size:12px;line-height:18px;white-space:pre-wrap"> OpenStack Compute (nova) <<a href="https://bugs.launchpad.net/nova/+bug/1224453" style="margin:0px;padding:0px;border:0px;font-style:inherit;font-size:12px;font-family:inherit;vertical-align:baseline;color:rgb(188,21,24);text-decoration:none">https://bugs.launchpad.net/nova/+bug/1224453</a>></i><br>

<span style="color:rgb(83,83,83);font-family:'andale mono','lucida console',monospace;font-size:12px;line-height:18px;white-space:pre-wrap">> ></span><i style="color:rgb(83,83,83);font-family:'andale mono','lucida console',monospace;font-size:12px;line-height:18px;white-space:pre-wrap"> is present or not.</i><br>

<font color="#535353" face="andale mono, lucida console, monospace"><span style="font-size:11.818181991577148px;line-height:17.99715805053711px;white-space:pre-wrap">> <br></span></font><span style="color:rgb(83,83,83);font-family:'andale mono','lucida console',monospace;font-size:12px;line-height:18px;white-space:pre-wrap">> You were referencing a bug in your original mail that alaski pointed you</span><br>

<span style="color:rgb(83,83,83);font-family:'andale mono','lucida console',monospace;font-size:12px;line-height:18px;white-space:pre-wrap">> at, which I thought you said you were hitting locally:</span><br>

<font color="#535353" face="andale mono, lucida console, monospace"><span style="font-size:11.818181991577148px;line-height:17.99715805053711px;white-space:pre-wrap">> <br></span></font>> <a href="https://bugs.launchpad.net/nova/+bug/1239898" style="font-family:inherit;font-size:12px;line-height:18px;white-space:pre-wrap;font-style:inherit;margin:0px;padding:0px;border:0px;vertical-align:baseline;color:rgb(188,21,24);text-decoration:none">https://bugs.launchpad.net/nova/+bug/1239898</a><br>

<font color="#535353" face="andale mono, lucida console, monospace"><span style="font-size:11.818181991577148px;line-height:17.99715805053711px;white-space:pre-wrap">> <br></span></font><span style="color:rgb(83,83,83);font-family:'andale mono','lucida console',monospace;font-size:12px;line-height:18px;white-space:pre-wrap">> If not, then ignore.</span><br>

<font color="#535353" face="andale mono, lucida console, monospace"><span style="font-size:11.818181991577148px;line-height:17.99715805053711px;white-space:pre-wrap"><br>Yeah, it's OK now.</span></font></div><div><font color="#535353" face="andale mono, lucida console, monospace"><span style="font-size:11.818181991577148px;line-height:17.99715805053711px;white-space:pre-wrap"><br>

</span></font><font color="#535353" face="andale mono, lucida console, monospace"><span style="font-size:11.818181991577148px;line-height:17.99715805053711px;white-space:pre-wrap">> <br></span></font><span style="color:rgb(83,83,83);font-family:'andale mono','lucida console',monospace;font-size:12px;line-height:18px;white-space:pre-wrap">> Thanks!</span><br>

<font color="#535353" face="andale mono, lucida console, monospace"><span style="font-size:11.818181991577148px;line-height:17.99715805053711px;white-space:pre-wrap">></span></font><div><font color="#535353" face="andale mono, lucida console, monospace"><span style="font-size:11.818181991577148px;line-height:17.99715805053711px;white-space:pre-wrap">> -</span></font><span style="color:rgb(83,83,83);font-family:'andale mono','lucida console',monospace;font-size:12px;line-height:18px;white-space:pre-wrap">-Dan</span><div>

<br></div></div></div></div>