Canonical Juju

help text for juju add-user needs improving

Bug #1557148 reported by Peter Matulis
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Canonical Juju
Fix Released
Medium
Reed O'Brien
Canonical Juju 2.0-beta3

Bug Description

e.g.:

Usage: juju add-user [options] <user name> [<display name>]

Summary:
Adds a Juju user to a controller.

Options:

--acl (= "read")
    Access controls
-c, --controller (= "")
    Controller to operate in
--models (= "")
    Models the new user is granted access to (space separated)

Details:
This allows the user to register with the controller and use the optionally shared model.
A `juju register` command will be printed, which must be executed by the user to complete the registration process.
The user's details are stored within the shared model, and will be removed when the model is destroyed.
Some machine providers will require the user to be in possession of certain credentials in order to create a model.
Note that the default permission for model access is read-only.

Examples:

    juju add-user bob
    juju add-user --models mymodel1 mymodel2 jim
    juju add-user --controller mycontroller sam
    juju add-user --models mymodel --acl write tom

See also: register
          grant
          show-user
          list-users
          switch-user
          disable-user
          enable-user
          change-user-password

See original description
Tags: helpdocs
Cheryl Jennings (cherylj)
Changed in juju-core:
status: New → Triaged
importance: Undecided → Low
milestone: none → 2.0-beta3
Nick Veitch (evilnick)
description: updated
Peter Matulis (petermatulis)
description: updated
description: updated
Cheryl Jennings (cherylj)
Changed in juju-core:
importance: Low → Medium
Peter Matulis (petermatulis)
description: updated
Matthew Helmke (matthelmke)
description: updated
Revision history for this message
Peter Matulis (petermatulis) wrote :

Note that I removed [--share <model name>] from the current/old Usage section.

description: updated
description: updated
Matthew Helmke (matthelmke)
description: updated
Revision history for this message
Reed O'Brien (reedobrien) wrote :

Are the line lengths correct on the details? They look significantly different than the other updates so far. I will proceed breaking at 75th column as that seems to be what all the others do. Please confirm.

Revision history for this message
Reed O'Brien (reedobrien) wrote :

Also, the optional arguments don't match those in this issue.

Options:
--acl (= "read")
    access controls
-c, --controller (= "")
    Controller to operate in
--models (= "")
    models the new user is granted access to

Revision history for this message
Peter Matulis (petermatulis) wrote :

For some reason, this command was changed significantly after this bug was filed so, yes, there are now discrepancies.

Revision history for this message
Peter Matulis (petermatulis) wrote :

Is there any reason why --acl and --models are in the current (software) Usage section? They are also listed options. Isn't that good enough?

Revision history for this message
Peter Matulis (petermatulis) wrote :

@Reed
Updated.

description: updated
description: updated
Reed O'Brien (reedobrien)
Changed in juju-core:
status: Triaged → Fix Committed
assignee: nobody → Reed O'Brien (reedobrien)
Revision history for this message
Peter Matulis (petermatulis) wrote :

I discovered that switch-user has been removed in beta4. It should therefore not appear in 'See also:'.

Revision history for this message
Reed O'Brien (reedobrien) wrote : Re: [Bug 1557148] Re: help text for juju add-user needs improving

On Tue, Apr 12, 2016 at 7:47 PM, Peter Matulis <email address hidden>
wrote:

> I discovered that switch-user has been removed in beta4. It should
> therefore not appear in 'See also:'.
>

This is already in, so it should prolly be a new issue. Cheryl, any
particular tags?

--
Reed O'Brien
✉ <email address hidden>
✆ 415-562-6797

Revision history for this message
Reed O'Brien (reedobrien) wrote :

Looks like this failed to merge last night, too. Pushing the remove switch-user change to the existing PR.

Revision history for this message
Reed O'Brien (reedobrien) wrote :

nope I was wrong, it did merge, I just have a stale branch. I am slipping this update in on the PR for bug 1563932

Curtis Hovey (sinzui)
Changed in juju-core:
status: Fix Committed → Fix Released
Canonical Juju QA Bot (juju-qa-bot)
affects: juju-core → juju
Changed in juju:
milestone: 2.0-beta3 → none
milestone: none → 2.0-beta3
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

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