Merge lp:~mbruzek/charms/bundles/kubernetes-cluster/bundle into lp:~kubernetes/charms/bundles/kubernetes-cluster/bundle

Proposed by Matt Bruzek
Status: Merged
Merged at revision: 17
Proposed branch: lp:~mbruzek/charms/bundles/kubernetes-cluster/bundle
Merge into: lp:~kubernetes/charms/bundles/kubernetes-cluster/bundle
Diff against target: 43 lines (+5/-4)
2 files modified
bundle.yaml (+2/-2)
bundles.yaml (+3/-2)
To merge this branch: bzr merge lp:~mbruzek/charms/bundles/kubernetes-cluster/bundle
Reviewer Review Type Date Requested Status
Charles Butler Pending
Review via email: mp+273996@code.launchpad.net

Description of the change

Updating the bundle to reference the next revision of the kubernetes and kuberentes-master charms that have finally passed CI. Those changes involve getting kubernetes and kuberentes-master charms in sync with github.

This proposal depends on the other two charm proposals landing, and should be done last.

Only merge this proposal if the following proposals have been merged.
https://code.launchpad.net/~mbruzek/charms/trusty/kubernetes/trunk/+merge/273993
https://code.launchpad.net/~mbruzek/charms/trusty/kubernetes-master/trunk/+merge/273994

To post a comment you must log in.
Revision history for this message
Charles Butler (lazypower) wrote :

I'm holding off on merging this until we get a green test run attached, however I merged the other two dependent branches.

22. By Matt Bruzek

Correcting misspelled charm uri in bundle.

Revision history for this message
Matt Bruzek (mbruzek) wrote :

Good point Chuck.

http://juju-ci.vapour.ws:8080/job/charm-bundle-test-aws/989/console

It looks like it passed the tests on AWS

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

+1 LGTM

Charles Butler <email address hidden> - Juju Charmer
Come see the future of datacenter orchestration: http://jujucharms.com

On Fri, Oct 9, 2015 at 8:44 PM, Matt Bruzek <email address hidden>
wrote:

> Good point Chuck.
>
> http://juju-ci.vapour.ws:8080/job/charm-bundle-test-aws/989/console
>
> It looks like it passed the tests on AWS
> --
>
> https://code.launchpad.net/~mbruzek/charms/bundles/kubernetes-cluster/bundle/+merge/273996
> You are requested to review the proposed merge of
> lp:~mbruzek/charms/bundles/kubernetes-cluster/bundle into
> lp:~kubernetes/charms/bundles/kubernetes-cluster/bundle.
>

Revision history for this message
Matt Bruzek (mbruzek) wrote :

Based on the tests passing I am going to merge this bundle.

http://reports.vapour.ws/charm-test-details/charm-bundle-test-parent-1138

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'bundle.yaml'
2--- bundle.yaml 2015-10-01 03:55:32 +0000
3+++ bundle.yaml 2015-10-09 22:15:49 +0000
4@@ -1,6 +1,6 @@
5 services:
6 "kubernetes-master":
7- charm: cs:~kubernetes/trusty/kubernetes-master-10
8+ charm: cs:~kubernetes/trusty/kubernetes-master-11
9 constraints: "cpu-cores=2 mem=2G"
10 annotations:
11 "gui-x": "600"
12@@ -22,7 +22,7 @@
13 "gui-x": "0"
14 "gui-y": "300"
15 kubernetes:
16- charm: cs:~kubernetes/trusty/kubernetes-10
17+ charm: cs:~kubernetes/trusty/kubernetes-11
18 annotations:
19 "gui-x": "300"
20 "gui-y": "300"
21
22=== modified file 'bundles.yaml'
23--- bundles.yaml 2015-10-01 03:55:32 +0000
24+++ bundles.yaml 2015-10-09 22:15:49 +0000
25@@ -1,7 +1,8 @@
26 kubernetes-cluster:
27 services:
28 "kubernetes-master":
29- charm: cs:~kubernetes/trusty/kubernetes-master-10
30+ charm: cs:~kubernetes/trusty/kubernetes-master-11
31+ constraints: "cpu-cores=2 mem=2G"
32 annotations:
33 "gui-x": "600"
34 "gui-y": "0"
35@@ -22,7 +23,7 @@
36 "gui-x": "0"
37 "gui-y": "300"
38 kubernetes:
39- charm: cs:~kubernetes/trusty/kubernetes-10
40+ charm: cs:~kubernetes/trusty/kubernetes-11
41 annotations:
42 "gui-x": "300"
43 "gui-y": "300"

Subscribers

People subscribed via source and target branches