a few comments, but not really enough to block it. thoughts though?
https://codereview.appspot.com/71490045/diff/100001/cmd/juju/bootstrap.go File cmd/juju/bootstrap.go (right):
https://codereview.appspot.com/71490045/diff/100001/cmd/juju/bootstrap.go#newcode119 cmd/juju/bootstrap.go:119: logger.Warningf("using existing environment file: %q", loc) perhaps "ignoring environments.yaml: using bootstrap config in %q"
https://codereview.appspot.com/71490045/diff/100001/environs/configstore/disk.go File environs/configstore/disk.go (right):
https://codereview.appspot.com/71490045/diff/100001/environs/configstore/disk.go#newcode163 environs/configstore/disk.go:163: func (info *environInfo) Location() string { Path()?
https://codereview.appspot.com/71490045/diff/100001/environs/configstore/disk_test.go File environs/configstore/disk_test.go (right):
https://codereview.appspot.com/71490045/diff/100001/environs/configstore/disk_test.go#newcode113 environs/configstore/disk_test.go:113: c.Assert(info.Location(), gc.Matches, ".*/someenv.jenv") if we know the full path, which we should, I'd really like to assert it specifically
https://codereview.appspot.com/71490045/diff/100001/environs/configstore/mem.go File environs/configstore/mem.go (right):
https://codereview.appspot.com/71490045/diff/100001/environs/configstore/mem.go#newcode76 environs/configstore/mem.go:76: return "memory" ah. hmm. I see. I think I'd really rather have Path() (string, error); return ("", ErrNotADiskStore); and handle that error in the command.
overkill?
https://codereview.appspot.com/71490045/
« Back to merge proposal
a few comments, but not really enough to block it. thoughts though?
https:/ /codereview. appspot. com/71490045/ diff/100001/ cmd/juju/ bootstrap. go bootstrap. go (right):
File cmd/juju/
https:/ /codereview. appspot. com/71490045/ diff/100001/ cmd/juju/ bootstrap. go#newcode119 bootstrap. go:119: logger. Warningf( "using existing environment
cmd/juju/
file: %q", loc)
perhaps "ignoring environments.yaml: using bootstrap config in %q"
https:/ /codereview. appspot. com/71490045/ diff/100001/ environs/ configstore/ disk.go configstore/ disk.go (right):
File environs/
https:/ /codereview. appspot. com/71490045/ diff/100001/ environs/ configstore/ disk.go# newcode163 configstore/ disk.go: 163: func (info *environInfo) Location()
environs/
string {
Path()?
https:/ /codereview. appspot. com/71490045/ diff/100001/ environs/ configstore/ disk_test. go configstore/ disk_test. go (right):
File environs/
https:/ /codereview. appspot. com/71490045/ diff/100001/ environs/ configstore/ disk_test. go#newcode113 configstore/ disk_test. go:113: c.Assert( info.Location( ),
environs/
gc.Matches, ".*/someenv.jenv")
if we know the full path, which we should, I'd really like to assert it
specifically
https:/ /codereview. appspot. com/71490045/ diff/100001/ environs/ configstore/ mem.go configstore/ mem.go (right):
File environs/
https:/ /codereview. appspot. com/71490045/ diff/100001/ environs/ configstore/ mem.go# newcode76 configstore/ mem.go: 76: return "memory"
environs/
ah. hmm. I see. I think I'd really rather have Path() (string, error);
return ("", ErrNotADiskStore); and handle that error in the command.
overkill?
https:/ /codereview. appspot. com/71490045/