test_list_baremetal_nodes race fails with a node not found 404

Bug #1425258 reported by Matt Riedemann on 2015-02-24
28
This bug affects 4 people
Affects Status Importance Assigned to Milestone
Ironic
Medium
Unassigned
OpenStack Compute (nova)
Medium
Unassigned
Juno
Medium
Adam Gandelman
tempest
Undecided
Unassigned
Dmitry Tantsur (divius) wrote :

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

Changed in ironic:
status: New → Incomplete

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

Changed in nova:
status: New → Confirmed
importance: Undecided → Medium
Changed in ironic:
status: Incomplete → Confirmed
tags: added: nova-driver
Changed in ironic:
importance: Undecided → Medium
Changed in ironic:
status: Confirmed → Invalid
Changed in nova:
status: Confirmed → Invalid
Changed in tempest:
status: New → Confirmed
Changed in tempest:
status: Confirmed → In Progress
Adam Gandelman (gandelman-a) wrote :

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.

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

tags: added: in-stable-juno

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

Changed in tempest:
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Duplicates of this bug

Other bug subscribers