fuel-qa must have configurable timeout for diagnostic snapshot

Bug #1580799 reported by Sergey Yudin
12
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Committed
Medium
Unassigned
Mitaka
Fix Released
Medium
Unassigned

Bug Description

This timeout(60 * 10) must be configurable and must rely on parameter so any testgroup may have parameterized decorator and have own timeout for diagnostic snapshot

https://github.com/openstack/fuel-qa/blob/master/fuelweb_test/helpers/decorators.py#L332

Tags: area-qa
Changed in fuel:
importance: Undecided → Medium
status: New → Confirmed
assignee: nobody → Fuel QA Team (fuel-qa)
milestone: none → 10.0
tags: added: area-qa
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-qa (master)

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

Changed in fuel:
assignee: Fuel QA Team (fuel-qa) → Maksym Strukov (unbelll)
status: Confirmed → In Progress
Changed in fuel:
assignee: Maksym Strukov (unbelll) → Dennis Dmitriev (ddmitriev)
Changed in fuel:
assignee: Dennis Dmitriev (ddmitriev) → Maksym Strukov (unbelll)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-qa (master)

Reviewed: https://review.openstack.org/336301
Committed: https://git.openstack.org/cgit/openstack/fuel-qa/commit/?id=c4ab00da693236bd75c11d603ed27ce3ed87ea52
Submitter: Jenkins
Branch: master

commit c4ab00da693236bd75c11d603ed27ce3ed87ea52
Author: Maksym Strukov <email address hidden>
Date: Fri Jul 1 01:10:32 2016 +0300

    Configure log snapshot timeout via LOG_SNAPSHOT_TIMEOUT

    Change-Id: I2f14e1e879220f9123af9211a0c02c07f8f652aa
    Closes-Bug: #1580799

Changed in fuel:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-qa (stable/mitaka)

Fix proposed to branch: stable/mitaka
Review: https://review.openstack.org/341531

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

Reviewed: https://review.openstack.org/341531
Committed: https://git.openstack.org/cgit/openstack/fuel-qa/commit/?id=6739f5f8cb9a8014a23964fa0aba2c8f3ede97c8
Submitter: Jenkins
Branch: stable/mitaka

commit 6739f5f8cb9a8014a23964fa0aba2c8f3ede97c8
Author: Maksym Strukov <email address hidden>
Date: Fri Jul 1 01:10:32 2016 +0300

    Configure log snapshot timeout via LOG_SNAPSHOT_TIMEOUT

    Change-Id: I2f14e1e879220f9123af9211a0c02c07f8f652aa
    Closes-Bug: #1580799

Revision history for this message
Sergey Yudin (tsipa740) wrote :

Err, guys, i've lost track of what you're doing here.

Do you even understand what and why you doing? I have testgroup with 50 nodes and with 5 nodes. Obviously same timeout will not fit for them.
I want to wrap 50 nodes TG with apropriate decorator and set timeout to something suitable.
What you've done - you've saying that i have to wrap whole test in jenkins in bash with something
if [ TESTGROUP = ]; then

Obviously i don't want to modify jenkins job every time i'm creating testgroup.

tags: added: on-verification
Revision history for this message
Sergey Yudin (tsipa740) wrote :

ok, let's leave solution as-is

Revision history for this message
ElenaRossokhina (esolomina) wrote :

Verified on 9.0 + snapshot#103
CUSTOM_VERSION=snapshot #103
MAGNET_LINK=magnet:?xt=urn:btih:bfec808dd71ff42c5613a3527733d9012bb1fabc&dn=MirantisOpenStack-9.0.iso&tr=http%3A%2F%2Ftracker01-bud.infra.mirantis.net%3A8080%2Fannounce&tr=http%3A%2F%2Ftracker01-scc.infra.mirantis.net%3A8080%2Fannounce&tr=http%3A%2F%2Ftracker01-msk.infra.mirantis.net%3A8080%2Fannounce&ws=http%3A%2F%2Fvault.infra.mirantis.net%2FMirantisOpenStack-9.0.iso
FUEL_QA_COMMIT=1026e687d0478b026e3184c82045a0c99f854a29
UBUNTU_MIRROR_ID=ubuntu-2016-08-03-174238
CENTOS_MIRROR_ID=centos-7.2.1511-2016-05-31-083834
MOS_UBUNTU_MIRROR_ID=9.0-2016-08-05-174321
MOS_CENTOS_OS_MIRROR_ID=os-2016-06-23-135731
MOS_CENTOS_PROPOSED_MIRROR_ID=proposed-2016-08-05-174321
MOS_CENTOS_UPDATES_MIRROR_ID=updates-2016-06-23-135916
MOS_CENTOS_HOLDBACK_MIRROR_ID=holdback-2016-06-23-140047
MOS_CENTOS_HOTFIX_MIRROR_ID=hotfix-2016-07-18-162958
MOS_CENTOS_SECURITY_MIRROR_ID=security-2016-06-23-140002

tags: removed: on-verification
Maksym Strukov (unbelll)
Changed in fuel:
assignee: Maksym Strukov (unbelll) → nobody
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.