MultipleCreateTestJSON.test_multiple_create intermittently fails for cells v1 due to server name check change

Bug #1776684 reported by Matt Riedemann
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Matt Riedemann

Bug Description

2018-06-13 13:26:17.110798 | primary | Captured traceback:
2018-06-13 13:26:17.110851 | primary | ~~~~~~~~~~~~~~~~~~~
2018-06-13 13:26:17.110932 | primary | Traceback (most recent call last):
2018-06-13 13:26:17.111103 | primary | File "tempest/api/compute/servers/", line 43, in test_multiple_create
2018-06-13 13:26:17.111218 | primary | self.assertEqual(set(['VM-1', 'VM-2']), server_names)
2018-06-13 13:26:17.111435 | primary | File "/opt/stack/new/tempest/.tox/tempest/local/lib/python2.7/site-packages/testtools/", line 411, in assertEqual
2018-06-13 13:26:17.111542 | primary | self.assertThat(observed, matcher, message)
2018-06-13 13:26:17.111758 | primary | File "/opt/stack/new/tempest/.tox/tempest/local/lib/python2.7/site-packages/testtools/", line 498, in assertThat
2018-06-13 13:26:17.111830 | primary | raise mismatch_error
2018-06-13 13:26:17.111992 | primary | testtools.matchers._impl.MismatchError: set(['VM-2', 'VM-1']) != set([u'VM'])

This started failing after this change was made:'VM-1'%2C%20'VM-2'%5D)%2C%20server_names)%5C%22%20AND%20tags%3A%5C%22console%5C%22&from=7d

8 hits in 24 hours

Revision history for this message
Matt Riedemann (mriedem) wrote :

This is likely the reason for the failure, cells v1 doesn't honor that multi-create instance name contract:

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to tempest (master)

Fix proposed to branch: master

Changed in tempest:
assignee: nobody → Matt Riedemann (mriedem)
status: New → In Progress
Matt Riedemann (mriedem)
no longer affects: tempest
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on tempest (master)

Change abandoned by Matt Riedemann (<email address hidden>) on branch: master
Reason: Nevermind, will just blacklist the test for cells v1.

Revision history for this message
Matt Riedemann (mriedem) wrote :

Restored the tempest revert:

Changed in tempest:
status: New → In Progress
assignee: nobody → Matt Riedemann (mriedem)
Changed in nova:
status: Confirmed → Invalid
Changed in tempest:
importance: Undecided → High
Changed in tempest:
assignee: Matt Riedemann (mriedem) → Ghanshyam Mann (ghanshyammann)
Matt Riedemann (mriedem)
Changed in tempest:
assignee: Ghanshyam Mann (ghanshyammann) → Matt Riedemann (mriedem)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to tempest (master)

Submitter: Zuul
Branch: master

commit 1604a3854462df3386a5f9861ac5913e1ce97b37
Author: Matt Riedemann <email address hidden>
Date: Wed Jun 13 14:39:11 2018 +0000

    Revert "Add server name check for multiple creations"

    This reverts commit b59f458f2823bfcfd8a530003d34e912110c2f3a.

    This breaks the nova-cells-v1 job since the cells v1 API
    does not honor the same multi-create instance name format.

    Furthermore, prior to Rocky [1], the
    multi_instance_display_name_template config option could be used to
    customize the display name for multiple servers in a single request.
    Since Tempest is branchless, this means deployments older than Rocky
    could now fail this test if they are customizing that option.

    Really, the change to assert the server names shouldn't be possible
    until the nova pike series enters extended maintenance mode since that
    would be the last branch that supports the
    multi_instance_display_name_template config option.


    Change-Id: Ia9273be6bf9fd55822dfe46f56503f70953c3ff3
    Closes-Bug: #1776684

Changed in tempest:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/tempest 19.0.0

This issue was fixed in the openstack/tempest 19.0.0 release.

To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers