Create a new Apache Tomcat Juju Charm.

Bug #1295710 reported by Matt Bruzek
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juju Charms Collection
Fix Released
Undecided
Unassigned

Bug Description

This bug is going to track the progress of a new Apache Tomcat Juju Charm.

Please let me know if you find any issues or have suggestions for improvements.

Thanks!

Related branches

Revision history for this message
Charles Butler (lazypower) wrote :

Greetings Matt!

Thank you for the charm submission! I've pulled the charm and I have the following comments:

charm proof returns clean! excellent. Moving on.

I like how you have handled the hooks. It read very cleanly and was easy to follow.

I see tests! I've run the amulet tests and it completed successfully. This is a boon to the charm, and might I add, excellent work!

I've got one knit pick:

In tomcat-common there are several bursts of heredocs. We recommend that you use a templating language such as Cheetah or Jinja2 to build templated files instead of using heredocs. If you're adamant on using heredocs - its good practice to abstract them away into their own include.

Otherwise there is nothing glaringly wrong with this charm. Its of an extremely high quality, and is a welcome addition to the charm store!

Thank you for this submission. Without further delay I'm going to promulgate this charm and it should show up in the store in about 30 minutes. If you have any questions about this review feel free to contact us in #juju on irc.freenode.net or contact the mailing list <email address hidden>

Thanks again, and all the best

Changed in charms:
status: New → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.