juju-core

Code review comment for lp:~axwalk/juju-core/agent-setapihostports

  1. agent-setapihostports
  2. Merge into trunk
Revision history for this message
Dimiter Naydenov (dimitern) wrote :

LGTM

https://codereview.appspot.com/82520043/diff/1/agent/agent.go
File agent/agent.go (right):

https://codereview.appspot.com/82520043/diff/1/agent/agent.go#newcode432
agent/agent.go:432: return
Why return here rather than creating apiDetails?

https://codereview.appspot.com/82520043/

« Back to merge proposal