[fuel-devops] Devops classes should implement custom __repr__ method
Bug #1484480 reported by
Dmitry Tyzhnenko
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Fuel for OpenStack |
Confirmed
|
Medium
|
Fuel QA Team | ||
| Mitaka |
Won't Fix
|
Medium
|
Fuel QA Team | ||
| Newton |
Confirmed
|
Medium
|
Fuel QA Team | ||
Bug Description
When instance of a devops classe is printed to the debug log the __repr__ method is used.
Example of current debug log:
2015-08-13 08:17:09,782 - DEBUG __init__.py:45 -- Calling: get_ntp with args: (<devops.
We can implement own __repl__ method for improve understanding our debug logs
Some proposal of new log view:
2015-08-13 08:17:09,782 - DEBUG __init__.py:45 -- Calling: get_ntp with args: (SSHClient <0x7f4b59c62f10, host: '10.109.0.2', user: 'root'>, 'admin') {}
Revision history for this message
| Roman Prykhodchenko (romcheg) wrote | #1 |
| description: | updated |
| summary: |
- [fuel-devops] When instance of devops classes printed in debug log it - use default __repr__ method + [fuel-devops] Devops classes should implement custom __repr__ method |
| Changed in fuel: | |
| status: | New → Confirmed |
| Changed in fuel: | |
| assignee: | nobody → Fuel QA Team (fuel-qa) |
| tags: | added: area-qa |
Revision history for this message
| Tatyanka (tatyana-leontovich) wrote | #2 |
| Changed in fuel: | |
| milestone: | 8.0 → 9.0 |
To post a comment you must log in.
A big +1 for this but isn't exposing credentials bad for us?