collect-logs script hard-codes the "controller" model
Bug #1618091 reported by
Eric Snow
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Landscape Charm |
Fix Released
|
Undecided
|
Eric Snow | ||
Bug Description
See: http://
Lines: 273, 317, 439
The current code works fine for Juju 1 environments and for Juju 2 controllers using the "controller" model. However, the script will fail under Juju 2 with other models.
Instead, the script should use the current model by default for the outer model. An option will determine the inner model name (defaulting to "controller").
Related branches
lp://qastaging/~ericsnowcurrently/landscape-charm/fix-1618091-different-models
- Chris Glass (community): Approve
- Francis Ginther (community): Approve
-
Diff: 572 lines (+190/-85)3 files modifiedcollect-logs (+162/-61)
db-migrator.py (+6/-5)
test_collect-logs.py (+22/-19)
| Changed in landscape-charm: | |
| assignee: | nobody → Eric Snow (ericsnowcurrently) |
| status: | New → In Progress |
| tags: | removed: kanban |
| Changed in landscape-charm: | |
| status: | In Progress → Fix Committed |
| Changed in landscape-charm: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
