← Back to branch summary

~rogpeppe/juju-core/azure

« back to all changes in this revision

Viewing changes to environs/local/environprovider.go

  • Committer: Tarmac
  • Author(s): jtv at canonical
  • Date: 2013-07-15 04:51:31 UTC
  • mfrom: (1440.2.4 var-lib-juju)
  • Revision ID: tarmac-20130715045131-ecmza5n94prfehz1
[r=jtv] Make "/var/lib/juju" path a variable.

That particular path, for cloudinit's benefit, is hard-coded all over the
provider code. It may be an obstacle to testing, because tests can't
substitute a fake (and obviously tests shouldn't mess with /var/lib/juju, or
even rely on its existence).

If we later find that one of these mentions is different from the others, it's
quite possible that we have to specialize some of the uses into separate
variables. For now, the main things are to be aware, and to have an injection
point for tests.

Show diffs side-by-side

added added

removed removed

Lines of Context:
environs/local/environprovider.go
27
27
        environs.RegisterProvider("local", &environProvider{})
28
28
}
29
29
 
30
 
var (
31
 
        defaultRootDir = "/var/lib/juju/"
32
 
)
33
 
 
34
30
// Open implements environs.EnvironProvider.Open.
35
31
func (environProvider) Open(cfg *config.Config) (environs.Environ, error) {
36
32
        logger.Infof("opening environment %q", cfg.Name())
69
65
        }
70
66
        dir := utils.NormalizePath(localConfig.rootDir())
71
67
        if dir == "." {
72
 
                dir = filepath.Join(defaultRootDir, localConfig.namespace())
 
68
                dir = filepath.Join(environs.DataDir, localConfig.namespace())
73
69
                localConfig.attrs["root-dir"] = dir
74
70
        }
75
71