Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixes lp#1751849: Registration token used up, users needs guidance to resolve. #8435

Merged
merged 1 commit into from Feb 28, 2018

Conversation

anastasiamac
Copy link
Contributor

Description of change

'juju register' accepts a one-time registration token. Once used, re-running the same command will give users an obscure error message with no solution nor a way forward.

This PR changes the message we give the users when we suspect that the token was already used/expired.

QA steps

  1. bootstrap
  2. add a user
  3. run register command as provided by output from step 2
  4. re-run 3

EXPECTED OUTPUT:

ERROR Provided registration token may have been expired.
A controller administrator must reset your user to issue a new token.
See "juju help change-user-password" for more information.

Bug reference

https://bugs.launchpad.net/juju/+bug/1751849

@anastasiamac
Copy link
Contributor Author

$$merge$$

@jujubot jujubot merged commit b37fecf into juju:2.3 Feb 28, 2018
@anastasiamac anastasiamac deleted the token-used-msg-lp1751849 branch February 28, 2018 01:08
@jameinel jameinel mentioned this pull request Feb 28, 2018
jujubot added a commit that referenced this pull request Feb 28, 2018
#8437

## Description of change

Bring develop up to the tip of 2.3.

 prdesc Merge pull request #8435 from anastasiamac/token-used-msg-lp1751849
 prdesc Merge pull request #8433 from manadart/inc-2.3.5
 prdesc Merge pull request #8420 from jameinel/2.3-run-hook-intermittent-1749320
 prdesc Merge pull request #8431 from anastasiamac/block-duplicate-msg-fix-lp1751903
 prdesc Merge pull request #8430 from howbazaar/2.3-fix-pubsub-race

## QA steps

See individual PR.

## Documentation changes

None.

## Bug reference

 prdesc https://bugs.launchpad.net/juju/+bug/1749320
 prdesc https://bugs.launchpad.net/juju/+bug/1751849
 prdesc https://bugs.launchpad.net/juju/+bug/1751903
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants