Fuel for OpenStack

Need to add more info level logs

Bug #1483298 reported by Vasily Gorin
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
Low
Vasily Gorin
Fuel for OpenStack 8.0
7.0.x
Fix Released
High
Vadim Rovachev
Fuel for OpenStack 7.0-updates

Bug Description

I think it is necessary to add more info level logs in following methods in checkers:
upload_tarball
check_file_exists
untar
run_script
wait_upgrade_is_done
check_upgraded_containers

For example, if test upgrades Fuel we can see in stdout next log:

2015-08-10 13:20:45,621 - INFO fuel_web_client.py:870 -- Get ID of a last created cluster
2015-08-10 13:45:08,558 - INFO fuel_web_client.py:267 -- Assert fuel version is 7.0

Difference between first line and second line is 25 minutes. Sometimes I though that everything was failed.
We can make something like this:

2015-08-10 13:20:45,621 - INFO fuel_web_client.py:870 -- Get ID of a last created cluster

> File 'name' was uploaded
> Untarring file 'name'
> ...
> ...etc

2015-08-10 13:45:08,558 - INFO fuel_web_client.py:267 -- Assert fuel version is 7.0

Roman Prykhodchenko (romcheg)
Changed in fuel:
status: New → Confirmed
Vasily Gorin (vgorin)
tags: added: system-tests
Nastya Urlapova (aurlapova)
Changed in fuel:
milestone: 7.0 → 8.0
Vasily Gorin (vgorin)
Changed in fuel:
assignee: Fuel QA Team (fuel-qa) → Vasily Gorin (vgorin)
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/229338

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

Reviewed: https://review.openstack.org/229338
Committed: https://git.openstack.org/cgit/stackforge/fuel-qa/commit/?id=61aa44745d657811d0b95883ed238c7261423cb8
Submitter: Jenkins
Branch: master

commit 61aa44745d657811d0b95883ed238c7261423cb8
Author: vgorin <email address hidden>
Date: Wed Sep 30 13:01:26 2015 +0300

    Add more info level logs

    in following methods of checkers py:

    upload_tarball
    check_file_exists
    untar
    run_script
    wait_upgrade_is_done
    check_upgraded_containers

    Delete duplicated code of upgrading master in test_os_upgrade.py using upgrade_master function.

    Rename run_script to run_upgrade_script, because current name can mislead.

    Change-Id: Id3810d0b66f738c6b174a6c22d99e9634da46d14
    Closes-Bug: #1483298

Changed in fuel:
status: In Progress → Fix Committed
Dmitry Pyzhov (dpyzhov)
tags: added: area-qa
Tatyanka (tatyana-leontovich)
Changed in fuel:
status: Fix Committed → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-qa (stable/7.0)

Fix proposed to branch: stable/7.0
Review: https://review.openstack.org/304984

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

Reviewed: https://review.openstack.org/304984
Committed: https://git.openstack.org/cgit/openstack/fuel-qa/commit/?id=c93ac8597d6cce27d6f3f5e51ec8dfcc6195ab7c
Submitter: Jenkins
Branch: stable/7.0

commit c93ac8597d6cce27d6f3f5e51ec8dfcc6195ab7c
Author: vgorin <email address hidden>
Date: Wed Sep 30 13:01:26 2015 +0300

    Add more info level logs

    in following methods of checkers py:

    upload_tarball
    check_file_exists
    untar
    run_script
    wait_upgrade_is_done
    check_upgraded_containers

    Delete duplicated code of upgrading master in test_os_upgrade.py using upgrade_master function.

    Rename run_script to run_upgrade_script, because current name can mislead.

    Change-Id: Id3810d0b66f738c6b174a6c22d99e9634da46d14
    Closes-Bug: #1483298

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

Other bug subscribers

Remote bug watches

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