juju 2 beta10, resources facade lower-case inconsistency with all other facades

Bug #1597519 reported by Adam Stokes
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Canonical Juju
Fix Released
Critical
Tim Penhey

Bug Description

All facades start with an capital:

https://github.com/juju/juju/blob/master/api/facadeversions.go

There is now a 'resources' facade that gets automagically injected that is lower-case and it is also not listed in the facadeversions.go that we reference when autogenerating our macumba juju api client library.

Tags: conjure
tags: added: conjure
Changed in juju-core:
importance: Undecided → Critical
Revision history for this message
Chad Smith (chad.smith) wrote :

Just looking over the Login response for facades there are other lowercase facades as well that might be looked at if we are changing "resources":

{u'name': u'payloads', u'versions': [1]}, {u'name': u'payloads-hook-context', u'versions': [0]}, {u'name': u'resources', u'versions': [1]}, {u'name': u'resources-hook-context', u'versions': [1]}

tags: added: kanban-cross-team
tags: removed: kanban-cross-team
Ian Booth (wallyworld)
Changed in juju-core:
status: New → Triaged
assignee: nobody → Katherine Cox-Buday (cox-katherine-e)
milestone: none → 2.0-beta11
Tim Penhey (thumper)
Changed in juju-core:
assignee: Katherine Cox-Buday (cox-katherine-e) → Tim Penhey (thumper)
status: Triaged → In Progress
Tim Penhey (thumper)
Changed in juju-core:
status: In Progress → Fix Committed
Curtis Hovey (sinzui)
Changed in juju-core:
status: Fix Committed → Fix Released
affects: juju-core → juju
Changed in juju:
milestone: 2.0-beta11 → none
milestone: none → 2.0-beta11
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.