remove juju list-shares

Bug #1623097 reported by Richard Harding
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Canonical Juju
Fix Released
Critical
Tim Penhey
juju-ci-tools
Fix Released
Critical
Andrew James Beach

Bug Description

You can already see the list of users with access to a model using

    juju show-model

Having a unique command that is based on "shares" which don't exist causes confusion and should be removed before we go to rc1.

Tags: ci regression
Tim Penhey (thumper)
Changed in juju:
assignee: nobody → Tim Penhey (thumper)
status: Triaged → In Progress
Revision history for this message
Ian Booth (wallyworld) wrote :

After discussing with Rick, we agreed to:

$ juju list-users [<model>]

ie add an additional positional argument to list-users.

The output would be the same as for list-users for the controller except that it would not have the "CONTROLLER: foo" header and would be for the specified model.

Tim Penhey (thumper)
Changed in juju:
status: In Progress → Fix Committed
Revision history for this message
Nicholas Skaggs (nskaggs) wrote :

The functional-grant-revoke test needs to stop use of list-shares and adopt show-model instead. I trust the functionality isn't lost.

Changed in juju-ci-tools:
status: New → Triaged
importance: Undecided → Critical
Curtis Hovey (sinzui)
tags: added: ci regression
Curtis Hovey (sinzui)
Changed in juju-ci-tools:
assignee: nobody → Andrew James Beach (andrewjbeach)
status: Triaged → Fix Released
Curtis Hovey (sinzui)
Changed in juju:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

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