juju-core

Code review comment for lp://staging/~fwereade/juju-core/config-5-state-service-config-yaml

  1. config-5-state-service-config-yaml
  2. Merge into trunk
Revision history for this message
Tim Penhey (thumper) wrote :

LGTM - comment about a bug maybe fixed

https://codereview.appspot.com/10172045/diff/1/state/service.go
File state/service.go (right):

https://codereview.appspot.com/10172045/diff/1/state/service.go#newcode778
state/service.go:778: charm, _, err := s.Charm()
Is lp:1167465 being addressed by this?

https://codereview.appspot.com/10172045/

« Back to merge proposal