extend plainbox loggers to also support checkbox_ng
Bug #1250358 reported by
Sylvain Pineau
This bug affects 2 people
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Next Generation Checkbox (CLI) |
Fix Released
|
Medium
|
Zygmunt Krynicki | ||
| PlainBox (Toolkit) |
Fix Released
|
High
|
Zygmunt Krynicki | ||
Bug Description
It's actually not possible to log the service module now living in checkbox_ng, only plainbox root loggers are defined in the logging config.
To debug checkbox.
The plainbox loging fetaure should support extra root loggers to allow upper layers to use it super logging options.
Related branches
lp://qastaging/~zyga/checkbox/fix-1250358
- Sylvain Pineau (community): Approve
-
Diff: 18 lines (+4/-0)1 file modifiedcheckbox-ng/checkbox_ng/main.py (+4/-0)
Revision history for this message
| Zygmunt Krynicki (zyga) wrote | #1 |
| Changed in checkbox: | |
| status: | New → Triaged |
| importance: | Undecided → Medium |
| tags: | added: mentoring |
| Changed in checkbox: | |
| milestone: | none → plainbox-0.5 |
| Changed in checkbox: | |
| importance: | Medium → High |
| assignee: | nobody → Zygmunt Krynicki (zkrynicki) |
| milestone: | plainbox-0.5 → plainbox-0.6 |
Revision history for this message
| Zygmunt Krynicki (zyga) wrote | #2 |
| Changed in plainbox: | |
| milestone: | none → 0.6 |
| Changed in checkbox: | |
| milestone: | plainbox-0.6 → none |
| Changed in plainbox: | |
| assignee: | nobody → Zygmunt Krynicki (zkrynicki) |
| Changed in checkbox-ng: | |
| milestone: | none → future |
| Changed in plainbox: | |
| importance: | Undecided → High |
| status: | New → Triaged |
| no longer affects: | checkbox |
| tags: | removed: plainbox |
Revision history for this message
| Zygmunt Krynicki (zyga) wrote | #3 |
| Changed in checkbox-ng: | |
| status: | New → In Progress |
| Changed in plainbox: | |
| status: | Triaged → In Progress |
| Changed in checkbox-ng: | |
| assignee: | nobody → Zygmunt Krynicki (zkrynicki) |
| milestone: | future → 0.4 |
| importance: | Undecided → Medium |
Revision history for this message
| Zygmunt Krynicki (zyga) wrote | #4 |
| Changed in plainbox: | |
| status: | In Progress → Fix Committed |
| Changed in checkbox-ng: | |
| status: | In Progress → Fix Committed |
| Changed in checkbox-ng: | |
| status: | Fix Committed → Fix Released |
| Changed in plainbox: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
This is quite true. I think it would be great to patch the logging helper classes to include a parameter that defines the "root" location or perhaps make them work on the root logger (so they would apply anywhere). What do you think?