Log error entry was marked with debug

Bug #1554546 reported by Egor Kotko
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Committed
Medium
Georgy Kibardin
Mitaka
Fix Released
Medium
Georgy Kibardin

Bug Description

Log error entry was marked with debug/info

Steps to reproduce:
1. Execute test ceph_rados_gw

Actual result:
The test failed on execution of tasks, logs with ERRORs were marked as DEBUG/INFO

http://snag.gy/zPHzZ.jpg

Revision history for this message
Egor Kotko (ykotko) wrote :
Changed in fuel:
importance: Undecided → High
Dmitry Pyzhov (dpyzhov)
tags: added: area-library
removed: logging
Changed in fuel:
assignee: Fuel Python Team (fuel-python) → Fuel Library Team (fuel-library)
importance: High → Medium
Revision history for this message
Matthew Mosesohn (raytrac3r) wrote :

Astute is responsible for indicating the level of the logged message. In this case, reporting a node as error state should be an ERROR level message, not debug.

Changed in fuel:
assignee: Fuel Library Team (fuel-library) → Fuel Python Team (fuel-python)
tags: added: area-python logging module-astute
removed: area-library
Dmitry Pyzhov (dpyzhov)
tags: added: feature-logging
removed: logging
Changed in fuel:
status: New → Confirmed
Revision history for this message
Bug Checker Bot (bug-checker) wrote : Autochecker

(This check performed automatically)
Please, make sure that bug description contains the following sections filled in with the appropriate data related to the bug you are describing:

version

expected result

For more detailed information on the contents of each of the listed sections see https://wiki.openstack.org/wiki/Fuel/How_to_contribute#Here_is_how_you_file_a_bug

tags: added: need-info
Revision history for this message
Dmitry Pyzhov (dpyzhov) wrote :

Logs on UI is going to be deprecated, see bug #1556244. Marking the bug as Won't fix.

Changed in fuel:
status: Confirmed → Won't Fix
Revision history for this message
Nastya Urlapova (aurlapova) wrote :

@Dima, we cannot move issue to "Won't Fix" with reference on unimplemented functionality.

Changed in fuel:
status: Won't Fix → Confirmed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-astute (master)

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-astute (master)

Reviewed: https://review.openstack.org/325808
Committed: https://git.openstack.org/cgit/openstack/fuel-astute/commit/?id=5e07646ddac5f0fcfd55999cda607e671866396b
Submitter: Jenkins
Branch: master

commit 5e07646ddac5f0fcfd55999cda607e671866396b
Author: Georgy Kibardin <email address hidden>
Date: Mon Jun 6 13:00:23 2016 +0300

    Use ERROR level for error messages

    Messages with 'error' status must definitely be logged on ERROR level

    Change-Id: Idb047f9253ce630e3ce4353f3e576c3fff3f63af
    Closes-Bug: #1554546

Changed in fuel:
status: In Progress → Fix Committed
Dmitry Pyzhov (dpyzhov)
no longer affects: fuel/newton
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-astute (stable/mitaka)

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

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

Reviewed: https://review.openstack.org/355341
Committed: https://git.openstack.org/cgit/openstack/fuel-astute/commit/?id=2a994e9f5571b2c442ccebf9fae4b51a1f7ad6f5
Submitter: Jenkins
Branch: stable/mitaka

commit 2a994e9f5571b2c442ccebf9fae4b51a1f7ad6f5
Author: Georgy Kibardin <email address hidden>
Date: Mon Jun 6 13:00:23 2016 +0300

    Use ERROR level for error messages

    Messages with 'error' status must definitely be logged on ERROR level

    Change-Id: Idb047f9253ce630e3ce4353f3e576c3fff3f63af
    Closes-Bug: #1554546
    (cherry picked from commit 5e07646ddac5f0fcfd55999cda607e671866396b)

tags: added: on-verification
Revision history for this message
Alexey. Kalashnikov (akalashnikov) wrote :
tags: removed: on-verification
Revision history for this message
Georgy Kibardin (gkibardin) wrote :

This log record is about sending a message to Nailgun. In this case message content must not be reflected on message error level.

Revision history for this message
ElenaRossokhina (esolomina) wrote :

verified:
CUSTOM_VERSION=snapshot #229
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=78e594c093e52853cc2678fd903bedd1fb9fb96e
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-09-06-174322
MOS_CENTOS_OS_MIRROR_ID=os-2016-06-23-135731
MOS_CENTOS_PROPOSED_MIRROR_ID=proposed-2016-09-06-174322
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

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.