<div dir="ltr"><div dir="ltr"><br><br><div class="gmail_quote"><div>On Wed, Oct 31, 2018 at 7:38 AM Dmitry Tantsur <<a href="mailto:dtantsur@redhat.com">dtantsur@redhat.com</a>> wrote:</div><div> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">[trim]<br></blockquote><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div>
>     Ditto, grenade jobs do not cover our tests at all. Also this is the very job we<br>
>     run on other projects (nova, neutron, maybe more), so it will be a bit painful<br>
>     to remove it.<br>
> <br>
> <br>
> We run the basic baremetal ops test, which tests deploy. If we're already <br>
> covering the same code paths in other tests (which I feel we are), then the test <br>
> feels redundant to me. I'm not worried about the effort to change the job in <br>
> other gates. We really need to pull agent_ipmitool out of the name if we keep it <br>
> anyway... which still means going through zuul configs.<br>
<br>
Do not smoke tests cover rescue with bare metal? Because our jobs do.<br>
<br></div></blockquote><div><br></div><div>Smoke tests do not as far as I can tell, but I believe we run rescue by default when our test scenarios execute on our other tempest executing jobs as well as it is a superset of the main scenario.</div><div>Random example testr results: <a href="http://logs.openstack.org/72/614472/1/check/ironic-tempest-dsvm-ipa-partition-redfish-tinyipa/7537b02/testr_results.html.gz">http://logs.openstack.org/72/614472/1/check/ironic-tempest-dsvm-ipa-partition-redfish-tinyipa/7537b02/testr_results.html.gz</a><br></div><div> <br></div></div></div></div>