[nailgun][tests] sporadic fail in EnvironmentManager.reset_environment

Bug #1558173 reported by Aleksey Kasatkin
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
High
Aleksey Kasatkin
8.0.x
Fix Committed
Critical
Alexey Stupnikov

Bug Description

Tests that use EnvironmentManager.reset_environment fail sporadically with the following:

2016-03-16 16:33:50.509 | nailgun/test/base.py:999: in reset_environment
2016-03-16 16:33:50.509 | self.tester.assertEqual(resp.status_code, expect_http)
2016-03-16 16:33:50.509 | E AssertionError: 200 != 202

Fox example:

http://logs.openstack.org/03/292503/5/gate/gate-fuel-web-python27/a9d33e1/console.html

Changed in fuel:
assignee: Fuel Python Team (fuel-python) → Aleksey Kasatkin (alekseyk-ru)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-web (master)

Fix proposed to branch: master
Review: https://review.openstack.org/293619

Changed in fuel:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-web (master)

Reviewed: https://review.openstack.org/293619
Committed: https://git.openstack.org/cgit/openstack/fuel-web/commit/?id=3e4da833e5e52064615059decce1b58c4e333ea9
Submitter: Jenkins
Branch: master

commit 3e4da833e5e52064615059decce1b58c4e333ea9
Author: Aleksey Kasatkin <email address hidden>
Date: Wed Mar 16 19:38:40 2016 +0200

    Accept both 200 and 202 if no particular result is awaited

    When deferred task is started successfully API can return either 200
    or 202 depending on task status. So, we should not rely on particular value
    if caller did not set it (comparand) explcitly.

    Change-Id: I4c596cd0b591ef5985e2727fe204518d74a61332
    Closes-Bug: #1558173

Changed in fuel:
status: In Progress → Fix Committed
tags: added: dev-to-verify
Artem Roma (aroma-x)
tags: added: on-verification
Artem Roma (aroma-x)
Changed in fuel:
status: Fix Committed → Fix Released
tags: removed: on-verification
Artem Roma (aroma-x)
tags: removed: dev-to-verify
Revision history for this message
Alexey Stupnikov (astupnikov) wrote :
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-web (stable/8.0)

Fix proposed to branch: stable/8.0
Review: https://review.openstack.org/370721

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-web (stable/8.0)

Reviewed: https://review.openstack.org/370721
Committed: https://git.openstack.org/cgit/openstack/fuel-web/commit/?id=1585bbe171fc8011b493a52abd7ee3e55e9d15c0
Submitter: Jenkins
Branch: stable/8.0

commit 1585bbe171fc8011b493a52abd7ee3e55e9d15c0
Author: Aleksey Kasatkin <email address hidden>
Date: Wed Mar 16 19:38:40 2016 +0200

    Accept both 200 and 202 if no particular result is awaited

    When deferred task is started successfully API can return either 200
    or 202 depending on task status. So, we should not rely on particular value
    if caller did not set it (comparand) explcitly.

    Change-Id: I4c596cd0b591ef5985e2727fe204518d74a61332
    Closes-Bug: #1558173
    (cherry picked from commit 3e4da833e5e52064615059decce1b58c4e333ea9)

tags: added: non-release
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.