<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, Jun 22, 2016 at 11:13 PM, Sean Dague <span dir="ltr"><<a href="mailto:sean@dague.net" target="_blank">sean@dague.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 06/22/2016 09:03 AM, Matt Riedemann wrote:<br>
> On 6/21/2016 12:53 AM, Michael Still wrote:<br>
>> So, <a href="https://review.openstack.org/#/c/317739" rel="noreferrer" target="_blank">https://review.openstack.org/#/c/317739</a> is basically done I think.<br>
>> I'm after people's thoughts on:<br>
>><br>
>>  - I need to do some more things, as described in the commit message.<br>
>> Are we ok with them being in later patches to get reviews moving on this?<br>
><br>
> I'd be OK with caching/performance improvements in subsequent changes.<br>
> Docs on this are going to be important to land, so they could be<br>
> separate but if this is going to get into Newton I'd want the docs to be<br>
> in Newton also.<br>
><br>
>><br>
>>  - I'm unsure what level of tempest testing makes sense here. How much<br>
>> would you like to see? Do we need to add a vendordata REST service to<br>
>> devstack? That might be complicated in the amount of time available...<br>
><br>
> I don't think Tempest tests anything from the metadata API service. We<br>
> only have that running in a handful of jobs (the postgres job is the<br>
> main one). We could probably write a test though that ssh's into a guest<br>
> and then pulls the data from the metadata service. I'm not sure what<br>
> you'd populate into the dynamic vendor data endpoint though, maybe just<br>
> test data in devstack?<br>
><br>
> I think we should have functional tests in the Nova tree for this at<br>
> least - how feasible would that be? In other words, would we have to<br>
> stub out stuff to the point that it would be a useless test in nova's<br>
> functional test tree?<br>
<br>
</span>I'm pretty sure this could be tested reasonably well in Nova's<br>
functional test tree. You could have a real md server, and data behind<br>
it. The only stubbing would be the access path because you'll be hitting<br>
it from localhost instead of a server ip. But that shouldn't invalidate<br>
it too substantially.<br></blockquote><div><br></div><div>I'm happy to take a look at functional tests.</div><div><br></div><div>What I am getting from this is that it sounds like its worth reviewing the current patch while I work on functional tests. I can presumably beg nicely to let a docs review in after the merge deadline if needed, but I'd like to see the other two patches (including the one I haven't written yet) land before the deadline.</div><div><br></div><div>Michael</div><div> </div></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature">Rackspace Australia</div>
</div></div>