useless iso8601 debug info displayed if use "--debug"

Bug #1324470 reported by Liusheng
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
python-ceilometerclient
Fix Released
Low
Liusheng
python-heatclient
Fix Released
Low
Steve Baker
python-keystoneclient
Fix Released
Low
Liusheng
python-swiftclient
Fix Committed
Undecided
Qiu Yu

Bug Description

if I use "keystone --debug user-list", CLI will display useless iso8601 debug info like:

DEBUG:iso8601.iso8601:Got u'2014' for 'year' with default None
DEBUG:iso8601.iso8601:Got u'05' for 'monthdash' with default 1
DEBUG:iso8601.iso8601:Got 5 for 'month' with default 5
DEBUG:iso8601.iso8601:Got u'29' for 'daydash' with default 1
DEBUG:iso8601.iso8601:Got 29 for 'day' with default 29
DEBUG:iso8601.iso8601:Got u'10' for 'hour' with default None
DEBUG:iso8601.iso8601:Got u'52' for 'minute' with default None
DEBUG:iso8601.iso8601:Got u'54' for 'second' with default None

additionally, other projects like ceilometer, use keystoneclient use "--debug" parameter in their commands, CLI will also display the iso8601 debug info (e.g. ceilometer --debug alarm-list), this is because other clients must call keystoneclient to get token.

Liusheng (liusheng)
Changed in python-keystoneclient:
assignee: nobody → Liusheng (liusheng)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to python-keystoneclient (master)

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

Changed in python-keystoneclient:
status: New → In Progress
Dolph Mathews (dolph)
Changed in python-keystoneclient:
importance: Undecided → Low
tags: added: user-experience
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to python-keystoneclient (master)

Reviewed: https://review.openstack.org/96413
Committed: https://git.openstack.org/cgit/openstack/python-keystoneclient/commit/?id=4435425c8d847869c19467af98a2c7a24e9a328c
Submitter: Jenkins
Branch: master

commit 4435425c8d847869c19467af98a2c7a24e9a328c
Author: liu-sheng <email address hidden>
Date: Thu May 29 17:58:19 2014 +0800

    Set the iso8601 log level to WARN

    When use "--debug" parameter in other projects's CLI, it may display
    useless iso8601 debug info, this is because other client's CLI will
    call keystoneclient to get token, and keystoneclient will display the
    iso8601 debug info.

    set the iso8601 log level to "WARN" to avoid this issue.

    Change-Id: Id90c86f0e7c8e9ae47ca53667f469d111d6ae373
    Closes-bug: #1324470

Changed in python-keystoneclient:
status: In Progress → Fix Committed
Liusheng (liusheng)
Changed in python-heatclient:
assignee: nobody → Liusheng (liusheng)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to python-heatclient (master)

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

Changed in python-heatclient:
status: New → In Progress
Changed in python-heatclient:
assignee: Liusheng (liusheng) → Steve Baker (steve-stevebaker)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on python-heatclient (master)

Change abandoned by liusheng (<email address hidden>) on branch: master
Review: https://review.openstack.org/101810
Reason: duplicate

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

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

Liusheng (liusheng)
Changed in python-ceilometerclient:
assignee: nobody → Liusheng (liusheng)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to python-ceilometerclient (master)

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

gordon chung (chungg)
Changed in python-ceilometerclient:
importance: Undecided → Low
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to python-heatclient (master)

Reviewed: https://review.openstack.org/101359
Committed: https://git.openstack.org/cgit/openstack/python-heatclient/commit/?id=dc5911e1a34b3323820cd573e094e1ac59fd0987
Submitter: Jenkins
Branch: master

commit dc5911e1a34b3323820cd573e094e1ac59fd0987
Author: Steve Baker <email address hidden>
Date: Wed Jun 25 17:27:00 2014 +1200

    Improve --debug logging output

    This change makes the following improvements to the output of a heat
    command with the --debug option set:
    * Silences spurious logging from iso8601 and urllib3.connectionpool
    * Removes the line number from the logging format, since the intent
      of --debug is to show the user what network requests are occuring.

    Change-Id: Iccbae01a1a5b85db0770a4f56c3b48788dc3008a
    Closes-bug: #1324470

Changed in python-heatclient:
status: In Progress → Fix Committed
Dolph Mathews (dolph)
Changed in python-keystoneclient:
milestone: none → 0.10.0
Dolph Mathews (dolph)
Changed in python-keystoneclient:
status: Fix Committed → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to python-ceilometerclient (master)

Reviewed: https://review.openstack.org/102076
Committed: https://git.openstack.org/cgit/openstack/python-ceilometerclient/commit/?id=617f7f87cb19210382902ece612ba1d4e89ecde7
Submitter: Jenkins
Branch: master

commit 617f7f87cb19210382902ece612ba1d4e89ecde7
Author: liu-sheng <email address hidden>
Date: Tue Jun 24 09:20:49 2014 +0800

    Improve --debug logging output

    This change makes the following improvements to the output of a
    ceilometer command with the --debug option set:

    * Silences spurious logging from iso8601 and urllib3.connectionpool
    * Removes the line number from the logging format, since the intent
      of --debug is to show the user what network requests are occuring.

    Change-Id: I1b286adf63b1ca05ec4fcfb27fbc939aa662cc7e
    Closes-bug: #1324470

Changed in python-ceilometerclient:
status: In Progress → Fix Committed
Changed in python-heatclient:
importance: Undecided → Low
milestone: none → v0.2.11
Changed in python-heatclient:
status: Fix Committed → Fix Released
Changed in python-ceilometerclient:
milestone: none → 1.3.0
status: Fix Committed → Fix Released
Qiu Yu (unicell)
Changed in python-swiftclient:
assignee: nobody → Qiu Yu (unicell)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to python-swiftclient (master)

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

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

Reviewed: https://review.openstack.org/222743
Committed: https://git.openstack.org/cgit/openstack/python-swiftclient/commit/?id=f65641c44da8f16194aae1a95b58150bb09167ae
Submitter: Jenkins
Branch: master

commit f65641c44da8f16194aae1a95b58150bb09167ae
Author: Qiu Yu <email address hidden>
Date: Sat Sep 12 03:07:36 2015 +0800

    Suppress iso8601 logging from --debug output

    This change silences logging from iso8601 when --debug option set

    Change-Id: Ib8b8423012d43ef78d7138609fa98f40d46e7d4b
    Closes-bug: #1324470

Changed in python-swiftclient:
status: In Progress → Fix Committed
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.