please include transcode charm in charm store

Bug #1342843 reported by Dustin Kirkland 
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juju Charms Collection
Fix Released
Undecided
Unassigned

Bug Description

I created a video transcoding charm, as described in detail at:

http://blog.dustinkirkland.com/2014/07/scalable-parallel-video-transcoding-on.html

I would it to be included in the charm store. Please.

Revision history for this message
Dustin Kirkland  (kirkland) wrote :

mkdir precise
cd precise
bzr branch lp:~kirkland/charms/precise/transcode/trunklp:~kirkland/charms/precise/transcode/trunk transcode
juju boot-strap
juju deploy nfs
juju deploy --repository .. -n 6 local:precise/transcode
juju add-relation transcode nfs
juju set transcode input_url=http://download.blender.org/demo/old_demos/diditdoneit.mpg
### point browser to any transcode node
uju set transcode input_url=http://download.blender.org/peach/bigbuckbunny_movies/big_buck_bunny_1080p_stereo.ogg output_size=960x540
### point browser to any transcode node

Revision history for this message
Tim Van Steenburgh (tvansteenburgh) wrote :

Hey Dustin,

Cool charm, thanks for making this! I tested on EC2 and it works great, with a few minor caveats:

Bugs:

- The charm needs to `open-port 80` at some point to allow access to the web interface.

Suggestions:

- The config-changed hook will fail (exit 5) if you pass an https input_url. This isn't a deal-breaker, but it would be nice to note this limitation in the README.
- I believe this charm won't work on lxc because NFS won't work - it would be nice to note this limitation in the README as well.

Thanks again for the charm! If you have time to make these small corrections I think we can get this into the store soon!

Tim

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

Dustin,

Thank you for the Transcode charm submission. I've validated the charm, and it appears you've addressed Tim's feedback based on the open-port(80) block in the start hook of the charm.

I see no apparent blockers for this charm's inclusion in the charm store, and have started the promulgation now. Please note I've included a quick commit to add the caveats outlined above to the charm's readme, that you may need to backport to your local branch for future updates to prevent branch divergence.

You can find the project page for your charm in the juju charm store here: https://launchpad.net/~charmers/charms/precise/transcode/trunk

with the bug tracker, here:
https://bugs.launchpad.net/charms/precise/+source/transcode

We appreciate your patience during the review cycle, and thank you for a great submission.

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.

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.