New charm proposal: txstatsd

Bug #886362 reported by James Westby
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juju Charms Collection
New
Wishlist
Unassigned

Bug Description

Hi,

Here's my proposed first charm: txstatsd.

I'm sure it needs a lot of work, and it should be packaged, but I wanted to push it up for review.

  lp:~james-w/charm/oneiric/txstatsd/initial

It requires graphite as well for the "carbon" relation. I have an even crappier version of that charm
here.

Thanks,

James

Revision history for this message
Mark Mims (mark-mims) wrote :

stopper:
- needs some sort of license (I just grab them from other charms)

maybe a stopper(?):
- metadata says requires carbon, but there're no hooks for that relation. Sorry, I don't know if that's a stopper for actual promulgation or not. Does the charm do a whole lot without that hook?

recommendations/discussion:

- metadata:description... we should copy the summary again at the very least. maybe even blank?

- no need to open internal ports (8125) unless you intend for them to be external someday. Everything's open internally for the foreseeable future.

- maybe use an upstart script for your service to survive reboots?

- maybe pick some standard place to install the service (/usr/local, /opt, or even /usr)? It'd make it easier for someone else to come along and see the branch, scripts, pid-files, logs, etc when debugging.

Jorge Castro (jorge)
Changed in charm:
status: New → Incomplete
Revision history for this message
Mark Mims (mark-mims) wrote :

removing new-charm tag

please add this back when the charm is updated and it'll automatically get put in the review queue again.

Thanks!

tags: removed: new-charm
Revision history for this message
Jorge Castro (jorge) wrote :

Hi James, any update on this? Need help with anything?

Revision history for this message
James Westby (james-w) wrote : Re: [Bug 886362] Re: New charm proposal: txstatsd

On Mon, 19 Dec 2011 18:31:46 -0000, "Jorge O. Castro" <email address hidden> wrote:
> Hi James, any update on this? Need help with anything?

Hi,

I hear u1 have this puppetised now, so I was thinking about reworking
these charms to be based on that.

I'll take some time over the vacation to pick these up again and try and
get them to an acceptable state.

Thanks,

James

Revision history for this message
Launchpad Janitor (janitor) wrote :

[Expired for Juju Charms Collection because there has been no activity for 60 days.]

Changed in charms:
status: Incomplete → Expired
Revision history for this message
James Westby (james-w) wrote :

Hi,

I've done another pass at this charm and fixed most of the things that Mark suggested,
as well as fixing it for upstream changes and adding some other improvements.

Thanks,

James

Changed in charms:
status: Expired → New
Revision history for this message
James Westby (james-w) wrote :

Hi,

My improvements are actually at

  lp:~james-w/charms/precise/txstatsd/trunk

Thanks,

James

Jorge Castro (jorge)
Changed in charms:
importance: Undecided → Wishlist
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.