Comment 1 for bug 1355987

Greetings Jorge,

I gave this charm a re-review and I came up with the following notes:

The README will need a thorough revamping. Ideally we like to see multiple sections such as deployment strategy with examples, covering the options in detail, upstream support locations to file bugs - to name a few. If you have charm-tools installed, execute `charm add readme` to generate a README.ex, and follow that format.

The make target for the tests i noticed installs a specific revision of python_apt from launchpad, and it failed for me locally. Removing that launchpad url and replacing it with python_apt worked a treat.

There is currently no Icon included for the service. If you need help generating an icon, it can be done in inkscape fairly quickly - and executing `charm add icon` will also generate a boilerplate SVG for you. https://juju.ubuntu.com/docs/authors-charm-icon.html is a great tutorial to get moving on creating a charm icon and the guidelines.

Barring the modifications requested above, you are very close to having the rsyslog charm prepared for Trusty inclusion. I'm going to move the status of this bug from NEW to INCOMPLETE. When you are ready for another review, simply move the status to FIX COMMITTED and someone will be along shortly to review your work.

If you have any questions/comments/concerns about the review contact us in #juju on irc.freenode.net or email the mailing list <email address hidden>, or ask a question tagged with "juju" on http://askubuntu.com.