Code review comment for lp:~thumper/juju-core/juju-mongodb

Revision history for this message
Ian Booth (wallyworld) wrote :

LGTM but there's no test for the trusty vs non trusty specific content
of cloud init. I think we need that added.

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

https://codereview.appspot.com/77300044/diff/1/agent/mongo/mongo.go#newcode75
agent/mongo/mongo.go:75: mongodPath :=
MongodPathForSeries("some-series")
we have version.readSeries()
Why not use that (after the necessary tweaking to expose the
functionality)?

https://codereview.appspot.com/77300044/

« Back to merge proposal