[system-tests] We should to be sure that close all transport connectons in the tests

Bug #1346362 reported by Tatyanka
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
Medium
Maksym Strukov

Bug Description

http://paste.openstack.org/show/87426/

To avoid this we should carefully close connection in the finally block

{"build_id": "2014-07-21_14-04-23", "ostf_sha": "9863db951a6e159f4fa6e6861c8331e1af069cf8", "build_number": "441", "auth_required": false, "api": "1.0", "nailgun_sha": "438cbb8687bc0c205475e0cfd7a6d637b3e46356", "production": "docker", "fuelmain_sha": "539a5bf7493a5d14690a34bb18c3ad1c75b4f37f", "astute_sha": "fd9b8e3b6f59b2727b1b037054f10e0dd7bd37f1", "feature_groups": ["mirantis"], "release": "5.1", "fuellib_sha": "1ec799bc6c8b08b8c9c6243c426507cb7a46459b"}

Changed in fuel:
milestone: 5.1 → 6.0
Changed in fuel:
status: New → Triaged
Changed in fuel:
milestone: 6.0 → 6.1
summary: - [systest] We should to be sure that close all transport connectons in
- the tests
+ [system-tests] We should to be sure that close all transport connectons
+ in the tests
Revision history for this message
Tatyanka (tatyana-leontovich) wrote :

move to 7.0 according it happens from time to time and there is no clear steps how to reproduce the issue

Changed in fuel:
milestone: 6.1 → 7.0
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to fuel-qa (master)

Related fix proposed to branch: master
Review: https://review.openstack.org/207636

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Related fix proposed to branch: master
Review: https://review.openstack.org/209590

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to fuel-qa (master)

Reviewed: https://review.openstack.org/210259
Committed: https://git.openstack.org/cgit/stackforge/fuel-qa/commit/?id=9c4e3ecb0fbba1f2613a784f8b9cc13e93a64826
Submitter: Jenkins
Branch: master

commit 9c4e3ecb0fbba1f2613a784f8b9cc13e93a64826
Author: Dennis Dmitriev <email address hidden>
Date: Fri Aug 7 11:57:14 2015 +0300

    Close remote connections in ntp.py

    Refactor GroupNtpSync() as a context manager to get remote
    connection closed at the end of time synchronization.

    Change-Id: I9cfa0a2a00ede6f416edf9b659b4bbbd469f0c0d
    Related-Bug:#1346362

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on fuel-qa (master)

Change abandoned by Maksym Strukov (<email address hidden>) on branch: master
Review: https://review.openstack.org/209590

Changed in fuel:
milestone: 7.0 → 8.0
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to fuel-qa (master)

Reviewed: https://review.openstack.org/207636
Committed: https://git.openstack.org/cgit/stackforge/fuel-qa/commit/?id=1b432a77a79073791b17a376960bc487d0551bc9
Submitter: Jenkins
Branch: master

commit 1b432a77a79073791b17a376960bc487d0551bc9
Author: Maksym Strukov <email address hidden>
Date: Fri Jul 31 00:34:38 2015 +0300

    Use context manager for ssh sessions

    Change-Id: I4dfd0918b1a9664e5c7a775688a059043b591969
    Related-Bug: #1346362

Revision history for this message
Matthew Mosesohn (raytrac3r) wrote :
Revision history for this message
Vasily Gorin (vgorin) wrote :

Please Change status if it is already closed .

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/231026

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

Reviewed: https://review.openstack.org/231026
Committed: https://git.openstack.org/cgit/stackforge/fuel-qa/commit/?id=96bccaccdc5ecc4c6274f22169ea589304a4174d
Submitter: Jenkins
Branch: master

commit 96bccaccdc5ecc4c6274f22169ea589304a4174d
Author: Maksym Strukov <email address hidden>
Date: Mon Oct 5 17:44:41 2015 +0300

    Close remote connections

    Refactor system tests as a context manager to get remote
    connection closed at the end of test

    Change-Id: I066e71914300c9fb745891c0a19a717fc84fd8f6
    Closes-Bug: #1346362

Changed in fuel:
status: In Progress → Fix Committed
Dmitry Pyzhov (dpyzhov)
tags: added: area-qa
Changed in fuel:
status: Fix Committed → Fix Released
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.