refactor resource commands
Bug #1706809 reported by
Anastasia
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Canonical Juju |
Fix Released
|
Medium
|
Anastasia |
Bug Description
Resources commands - 'juju charm resources", "juju resources", "juju attach" - need to be moved from resources/cmd under a common infrastructure in cmd package.
They are also under-tested. We need to add unit tests to exercise them.
Changed in juju: | |
status: | Triaged → In Progress |
Changed in juju: | |
status: | In Progress → Fix Committed |
milestone: | none → 2.3-alpha1 |
Changed in juju: | |
status: | Fix Committed → Fix Released |
To post a comment you must log in.
This is not a small piece of work.
I have broken it into several parts to facilitate and expedite reviews:
1. straightforward move of resource/cmd into cmd/juju/resource;
2. breakout from component registration;
3. resource listener should use accessor function internal similar to store;
4. sanity feature testing for full-stack connectivity.
Part 1 against develop: https:/ /github. com/juju/ juju/pull/ 7680