test_list_baremetal_nodes race fails with a node not found 404

[Matt Riedemann]

http://logs.openstack.org/35/158435/1/check/check-grenade-dsvm-ironic-sideways/2beafaf/logs/new/screen-n-api.txt.gz?level=TRACE

Apparently this is unhandled and we get a 500 response:

http://logs.openstack.org/35/158435/1/check/check-grenade-dsvm-ironic-sideways/2beafaf/console.html#_2015-02-23_22_11_18_978

http://logstash.openstack.org/#eyJzZWFyY2giOiJtZXNzYWdlOlwiYmFyZW1ldGFsX25vZGVzLnB5XCIgQU5EIG1lc3NhZ2U6XCJOb2RlXCIgQU5EIG1lc3NhZ2U6XCJjb3VsZCBub3QgYmUgZm91bmRcIiBBTkQgdGFnczpcInNjcmVlbi1uLWFwaS50eHRcIiIsImZpZWxkcyI6W10sIm9mZnNldCI6MCwidGltZWZyYW1lIjoiNjA0ODAwIiwiZ3JhcGhtb2RlIjoiY291bnQiLCJ0aW1lIjp7InVzZXJfaW50ZXJ2YWwiOjB9LCJzdGFtcCI6MTQyNDgwNjgwMzM5MX0=

21 hits in the last 7 days, check and gate, master and stable/juno, all failures.

[Matt Riedemann (mriedem)]

This fixes the 500:

https://review.openstack.org/#/c/158888/

[Dmitry Tantsur (divius)]

Hi! Could you clarify, is there something we can fix from Ironic point of view?

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/158888
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=a36c24d5b1c452f5be4c0074cb4d2a3e39a832b4
Submitter: Jenkins
Branch: master

commit a36c24d5b1c452f5be4c0074cb4d2a3e39a832b4
Author: Matt Riedemann <email address hidden>
Date: Tue Feb 24 13:25:18 2015 -0800

    Handle 404 in os-baremetal-nodes GET

    Handle the 404 that python-ironicclient raises so we don't return a 500
    to the caller.

    Partial-Bug: #1425258

    Change-Id: Id9304844742ee3d34f88e661aadfd737e9515aa1

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/juno
Review: https://review.openstack.org/160476

[Adam Gandelman (gandelman-a)]

This a tempest bug. with the 500 fixed, we're still hitting intermittent 404s. The test itself does no resource setup and relies on nodes existing in the inventory that were created by devstack. If this runs in parallel to the other baremetal tests in tempest.api.baremetal, we end up with temporary test nodes created there polluting the inventory while this one is run. https://review.openstack.org/162773 should help by creating a set of nodes for this test.

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/160476
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=f9bcf740b9fb105768c514e09837e4ef6c0a93f8
Submitter: Jenkins
Branch: stable/juno

commit f9bcf740b9fb105768c514e09837e4ef6c0a93f8
Author: Adam Gandelman <email address hidden>
Date: Mon Mar 2 12:02:05 2015 -0800

    Handle 404 in os-baremetal-nodes GET

    Handle the 404 that python-ironicclient raises so we don't return a 500
    to the caller.

    Partial-Bug: #1425258

    This is a backport of a36c24d5b1c452f5be4c0074cb4d2a3e39a832b4

    Change-Id: Id9304844742ee3d34f88e661aadfd737e9515aa1

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/162773
Committed: https://git.openstack.org/cgit/openstack/tempest/commit/?id=118cd39c61996785f21acfb1afecba5f0d3e7fb9
Submitter: Jenkins
Branch: master

commit 118cd39c61996785f21acfb1afecba5f0d3e7fb9
Author: Adam Gandelman <email address hidden>
Date: Mon Mar 9 14:41:36 2015 -0700

    Create test nodes for test_baremetal_nodes

    This test currently relies on pre-existing resources and races if run
    in parallel to other baremetal tests. This adds creation of 3 test
    nodes directly in Ironic to be tested via the Nova API extension.

    This also tags said test with the 'baremetal' test attribute.

    Closes-bug: #1425258

    Change-Id: I4dbd37bdb2019b6eb0140d46a605d5c8392323f4