https://codereview.appspot.com/13908044/diff/1/state/apiserver/uniter/uniter.go#oldcode797
state/apiserver/uniter/uniter.go:797: // All relation settings should be
strings.
Hmm. This is almost legitimate-panic territory. Best not, since we're
running in the API server and will cause a lot of collateral damage, but
I think we should at least error out -- if it happens, we're clearly not
in Kansas any more.
LGTM with an error return on unexpected relation settings.
https:/ /codereview. appspot. com/13908044/ diff/1/ state/api/ params/ internal. go params/ internal. go (right):
File state/api/
https:/ /codereview. appspot. com/13908044/ diff/1/ state/api/ params/ internal. go#newcode138 params/ internal. go:138: // EnvironConfig holds an environment
state/api/
configuration.
Man, our vocabulary is still kinda screwed up here... "Config" means too
many things. Not much we can do about it right now though.
https:/ /codereview. appspot. com/13908044/ diff/1/ state/api/ uniter/ unit.go uniter/ unit.go (right):
File state/api/
https:/ /codereview. appspot. com/13908044/ diff/1/ state/api/ uniter/ unit.go# newcode143 uniter/ unit.go: 143: return interface{ }(result. Settings) , nil result. Settings) ?
state/api/
map[string]
can you not do charm.Settings(
https:/ /codereview. appspot. com/13908044/ diff/1/ state/apiserver /uniter/ uniter. go /uniter/ uniter. go (left):
File state/apiserver
https:/ /codereview. appspot. com/13908044/ diff/1/ state/apiserver /uniter/ uniter. go#oldcode797 /uniter/ uniter. go:797: // All relation settings should be
state/apiserver
strings.
Hmm. This is almost legitimate-panic territory. Best not, since we're
running in the API server and will cause a lot of collateral damage, but
I think we should at least error out -- if it happens, we're clearly not
in Kansas any more.
https:/ /codereview. appspot. com/13908044/