Provide return blocked operation message to a user. #6356

Merged
merged 1 commit into from Sep 29, 2016

Conversation

Projects
None yet
3 participants
Member

anastasiamac commented Sep 29, 2016

Previously, when juju model was locked for any type of change - destruction, removal, changes to Juju components such as models, machines, units, etc - when a user ran any of the commands that are blocked, the command will return silently without providing any feedback. There was no way for user to know that the command was blocked except to examine the log.

With this PR, we now inform the user that the operation was blocked and that the environment is locked.

Adjust unit tests accordingly.

  • QA steps *
  • Bootstrap
  • Run "disable-command" to block operations
  • Run any of the disabled commands
    [observe command feedback that operation is blocked]
Member

anastasiamac commented Sep 29, 2016

@dimitern \o/ tyvm!
$$merge$$

Contributor

jujubot commented Sep 29, 2016

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju

@jujubot jujubot merged commit 082c3bc into juju:master Sep 29, 2016

@anastasiamac anastasiamac deleted the anastasiamac:expose-blocked-op-msg branch Sep 29, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment