New Charm: npm-app

Bug #1572326 reported by Alan Meekins
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juju Charms Collection
In Progress
Undecided
Unassigned

Bug Description

Just following the directions for charm ingestion at: https://jujucharms.com/docs/1.25/charms-bundles to get my package included in the charm store.

https://code.launchpad.net/~alan-meekins/charms/trusty/npm-app/trunk

This charm is a fork of node-app which unlike node-app uses `npm start` to execute services in a way more in line with how npm packages are actually used.

summary: - npm-app needs to be indexed into charm store
+ New Charm: npm-app
description: updated
Revision history for this message
Review Queue (review-queue) wrote : AWS Test Results: New Charm: npm-app

This item has failed automated testing! Results available here http://juju-ci.vapour.ws:8080/job/charm-bundle-test-aws/3955/

Revision history for this message
Review Queue (review-queue) wrote : LXC Test Results: New Charm: npm-app

This item has failed automated testing! Results available here http://juju-ci.vapour.ws:8080/job/charm-bundle-test-lxc/3910/

Revision history for this message
Cory Johns (johnsca) wrote :

Alan,

Thank you for your contribution! If this is the more correct way of starting the node apps, would you prefer to submit this as a Merge Proposal against node-app (https://code.launchpad.net/~charmers/charms/precise/node-app/trunk) instead? If you wish to submit this as a replacement, is there a significant reason to change the name from node-app to npm-app?

Additionally, if you only want a charm to be available for deployment from the store, you can publish it in your own namespace (see the instructions for the new publish workflow here: https://jujucharms.com/docs/devel/authors-charm-store). It would then be available for deploy with:

    juju deploy cs:~alan-meekins/trusty/npm-app

Submitting a charm for review is only required if you then want it to be promulgated to the top-level namespace, i.e., cs:trusty/npm-app

Revision history for this message
Pen Gale (pengale) wrote :

Hi Alan,

Please let us know how you'd like to proceed, per Cory's comment above.

Changed in charms:
status: New → In Progress
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.