<div dir="ltr">Thanks for raising this point!<br><br><div class="gmail_quote"><div dir="ltr">On Tue, Apr 11, 2017 at 5:34 AM Ghanshyam Mann <<a href="mailto:ghanshyammann@gmail.com">ghanshyammann@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi All,<br class="gmail_msg">
<br class="gmail_msg">
There is tempest tests for compute baremetal GET nodes tests[1]. This<br class="gmail_msg">
tests involve ironic and nova. Ironic to create baremetal nodes and<br class="gmail_msg">
then GET nodes using nova APIs.<br class="gmail_msg">
Nova only provides GET APIs for baremetal nodes which are already<br class="gmail_msg">
deprecated [2].<br class="gmail_msg">
<br class="gmail_msg">
As nova baremetal APIs are deprecated and test needs Ironic to be<br class="gmail_msg">
present and so ironic baremetal service client, we propose to remove<br class="gmail_msg">
this test from tempest[3]. We have coverage of that feature/API in<br class="gmail_msg">
ironic tempest plugin for node GET/POST and nova API in nova<br class="gmail_msg">
functional tests[4].<br class="gmail_msg"></blockquote><div>+1<br></div><div><br></div><div>That test requires the ironic plugin to be installed, so it does not run in any</div><div>of the Tempest gates today.</div><div><br></div><div>I think Tempest cannot / should not host tests that depend on plugins (which </div><div>in turn depend on Tempest).</div><div><br></div><div>andreaf</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br class="gmail_msg">
I have been objecting this in past but now I feel its not worth to<br class="gmail_msg">
test this in Tempest due to its complexity of Ironic requirement.<br class="gmail_msg">
This is part of tempest tests removal standard, feel free to let us<br class="gmail_msg">
know in case of any objection.<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
..1 <a href="https://git.openstack.org/cgit/openstack/tempest/tree/tempest/api/compute/admin/test_baremetal_nodes.py" rel="noreferrer" class="gmail_msg" target="_blank">https://git.openstack.org/cgit/openstack/tempest/tree/tempest/api/compute/admin/test_baremetal_nodes.py</a><br class="gmail_msg">
..2 <a href="https://developer.openstack.org/api-ref/compute/#bare-metal-nodes-os-baremetal-nodes-deprecated" rel="noreferrer" class="gmail_msg" target="_blank">https://developer.openstack.org/api-ref/compute/#bare-metal-nodes-os-baremetal-nodes-deprecated</a><br class="gmail_msg">
..3 <a href="https://review.openstack.org/#/c/449158/" rel="noreferrer" class="gmail_msg" target="_blank">https://review.openstack.org/#/c/449158/</a><br class="gmail_msg">
..4 <a href="http://git.openstack.org/cgit/openstack/nova/tree/nova/tests/functional/api_sample_tests/test_baremetal_nodes.py" rel="noreferrer" class="gmail_msg" target="_blank">http://git.openstack.org/cgit/openstack/nova/tree/nova/tests/functional/api_sample_tests/test_baremetal_nodes.py</a><br class="gmail_msg">
<br class="gmail_msg">
-gmann<br class="gmail_msg">
<br class="gmail_msg">
__________________________________________________________________________<br class="gmail_msg">
OpenStack Development Mailing List (not for usage questions)<br class="gmail_msg">
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" class="gmail_msg" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br class="gmail_msg">
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" class="gmail_msg" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br class="gmail_msg">
</blockquote></div></div>