JUJU_MODEL not being respected

[Jason Hobbs]

I'm exporting JUJU_MODEL but juju seems to be ignoring it.

http://paste.ubuntu.com/24647512/

If I use -m to pass the model name in, everything works fine. Even juju status -m $JUJU_MODEL is working, so I know my model string is correct.

This is with 2.2-beta4-xenial-amd64

[John A Meinel (jameinel)]

I do see this in cmd/modelcmd/modelcommand.go:
func GetCurrentModel(store jujuclient.ClientStore) (string, error) {
    if model := os.Getenv(osenv.JujuModelEnvKey); model != "" {
        return model, nil
    }

    currentController, err := store.CurrentController()
...

but its the only place I see an actual reference to JujuModelEnvKey, so its possible some other refactoring broke this support.

[Anastasia (anastasia-macmood)]

Adding 'eda' tag - we should have a functional test(s) to detect that we do not break JUJU_MODEL.

[Anastasia (anastasia-macmood)]

PR against develop (2.2): https://github.com/juju/juju/pull/7394