/juju

all: use v5 of the charmstore API #4807

Merged
merged 1 commit into from Mar 21, 2016

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
4 participants
@rogpeppe @jujubot @urosj @natefinch
@rogpeppe
Owner

rogpeppe commented Mar 20, 2016

This implies the new publishing model, so charms need to be published
before they are available in the default (stable) channel.

Because the new ussologin VisitWebPage API needed changing
in many places, we factor out everywhere that uses it into
a single place rather than just change it everywhere.

(Review request: http://reviews.vapour.ws/r/4249/)
@rogpeppe
Owner

rogpeppe commented Mar 20, 2016

$$merge$$
@jujubot
Contributor

jujubot commented Mar 20, 2016

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

jujubot commented Mar 20, 2016

Build failed: Does not match ['fixes-1559706']
build url: http://juju-ci.vapour.ws:8080/job/github-merge-juju/6943
@urosj
Owner

urosj commented Mar 21, 2016

👍
@natefinch
Contributor

natefinch commented Mar 21, 2016

👍
@rogpeppe
Owner

rogpeppe commented Mar 21, 2016

$$merge$$
@jujubot
Contributor

jujubot commented Mar 21, 2016

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

jujubot commented Mar 21, 2016

Build failed: Generating tarball failed
build url: http://juju-ci.vapour.ws:8080/job/github-merge-juju/6970
@rogpeppe
Owner

rogpeppe commented Mar 21, 2016

$$merge$$
@jujubot
Contributor

jujubot commented Mar 21, 2016

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju
@rogpeppe
all: use v5 of the charmstore API 
This implies the new publishing model, so charms need to be published
before they are available in the default (stable) channel.
27caec8
@jujubot
Contributor

jujubot commented Mar 21, 2016

Build failed: Generating tarball failed
build url: http://juju-ci.vapour.ws:8080/job/github-merge-juju/6974
@rogpeppe
Owner

rogpeppe commented Mar 21, 2016

$$merge$$
@jujubot
Contributor

jujubot commented Mar 21, 2016

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

jujubot added a commit that referenced this pull request Mar 21, 2016

@jujubot
Merge pull request #4807 from rogpeppe/070-use-charmstore-v5-api 
all: use v5 of the charmstore API

This implies the new publishing model, so charms need to be published
before they are available in the default (stable) channel.

Because the new ussologin VisitWebPage API needed changing
in many places, we factor out everywhere that uses it into
a single place rather than just change it everywhere.

(Review request: http://reviews.vapour.ws/r/4249/)
b20754a

@jujubot jujubot merged commit b20754a into juju:master Mar 21, 2016

cherylj pushed a commit to cherylj/juju that referenced this pull request Mar 22, 2016

Add back github.com/gabriel-samfira/sys dep 
PR #4807 removed github.com/gabriel-samfira/sys from
dependencies.tsv, causing build failures on master.

Re-adding it back in.
7277fb0

@cherylj cherylj referenced this pull request Mar 22, 2016

Merged

Add back github.com/gabriel-samfira/sys dep #4852

jujubot added a commit that referenced this pull request Mar 22, 2016

@jujubot
Merge pull request #4852 from cherylj/add-gabriel-samfira-dep 
Add back github.com/gabriel-samfira/sys dep

PR #4807 removed github.com/gabriel-samfira/sys from
dependencies.tsv, causing build failures on master.

Re-adding it back in.

(Review request: http://reviews.vapour.ws/r/4299/)
0ee1f42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment