Code review comment for lp:~mskalka/juju-ci-tools/assess-destroy-model

Revision history for this message
Aaron Bentley (abentley) wrote :

get_current_model and get_current_controller appear to reimplement Juju2Backend.get_active_model. If they do not, I recommend explaining the difference in the docstring.

I recommend using --format json or --format yaml in general. It is much more precise than the default textual output. For example, with list-models, parsed['current-model'] gives you the current model.

« Back to merge proposal