/juju

Allow deployment of metered charms that do not require a plan. #6549

Merged
merged 1 commit into from Nov 9, 2016

Conversation

Reviewers

@mattyw mattyw

@alesstimec alesstimec

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
4 participants
@cmars @jujubot @mattyw @alesstimec
@cmars
Owner

cmars commented Nov 9, 2016

QA steps: Should be able to deploy a charm that declares metrics. See
https://bugs.launchpad.net/juju-core/+bug/1639855 for a test case.

Fixes LP:#1639855, depends on juju/charms#225
@cmars
Allow deployment of metered charms that do not require a plan. 
QA steps: Should be able to deploy a charm that declares metrics. See
https://bugs.launchpad.net/juju-core/+bug/1639855 for a test case.

Fixes LP:#1639855, depends on juju/charms#225
f3dda83
@alesstimec

alesstimec approved these changes Nov 9, 2016
View changes

LGTM

@cmars
Owner

cmars commented Nov 9, 2016

$$merge$$
@jujubot
Contributor

jujubot commented Nov 9, 2016

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju

@jujubot jujubot merged commit 0ef630d into juju:1.25 Nov 9, 2016
1 check passed

1 check passed

github-check-merge-juju Built PR, ran unit tests, and tested LXD deploy. Use !!.*!! to request another build. IE, !!build!!, !!retry!!
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment