new-charm Terracotta
Bug #992572 reported by
Robert Ayres
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Juju Charms Collection |
Fix Released
|
Undecided
|
Unassigned |
Bug Description
New Terracotta charm based on Terracotta 3.6.2 - lp:~robert-ayres/charms/precise/terracotta/trunk.
I've successfully tested it on LXC and EC2.
Changed in charms: | |
assignee: | nobody → James Page (james-page) |
To post a comment you must log in.
Hi Robert
I've reviewed your charm - looks pretty good - I was able to get a terracotta cluster up and running using the LXC provider really quickly.
I have two comments:
1) hooks/terracotta - configDefaults function
I would use default-java rather that the explicit openjdk-6 path - the next release of Ubuntu will ship openjdk-7 as default so using this link will mean you get auto-upgraded :-)
2) Source of distribution
I like the way you are validating the checksums on the distribution of terracotta - and the fact that this is configuration.
Around 12 months ago we did some work to package terracotta from the upstream binary distribution:
https:/ /launchpad. net/~terracotta -ubuntu
This was getting some limited use based on feedback I had from users but it would be a nice complement to this charm and would move the stuff I would expect to see in packaging out of the charm.
Note that this point won't block entry to official charm status - but would be nice to discuss at UDS next week.
3) metadata.yaml - peer relationship
The way you have your peer relationship defined in metadata.yaml works fine (although I'm not sure why):
peers:
server-array: terracotta-server
However its not formed as per spec:
https:/ /juju.ubuntu. com/docs/ charm.html# sample- metadata- yaml-files
So I'm a little concerned that it might break at a later date is something in juju changes - I think this is correct:
peers:
server-array:
interface: terracotta-server
Marking 'Incomplete' pending fixup of points 1) and 3).
Cheers
James