Fix tech-debt around creating new api clients from facades
Bug #1300637 reported by
Matthew Williams
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| juju-core |
Won't Fix
|
Low
|
Matthew Williams | ||
Bug Description
In juju/api.go
Rather than create a new function in juju for every single API client, just export NewAPIClient and clients use facade New
methods as appropriate.
Delete NewKeyManagerClient and NewUserManagerC
api.Client, KeyManager and UserManager.
Revision history for this message
| John A Meinel (jameinel) wrote | #1 |
| Changed in juju-core: | |
| status: | New → Triaged |
| Changed in juju-core: | |
| status: | Triaged → Won't Fix |
To post a comment you must log in.
Thanks for filing this. I think there is disagreement within the team about the best way forward. So I wouldn't work too much on this until that discussion has been had.