Skip to content

Instantly share code, notes, and snippets.

@mbruzek

mbruzek/pull-request.md

Last active August 29, 2015 14:19
Show Gist options
  • Save mbruzek/acaaa5fd69361eafc4c7 to your computer and use it in GitHub Desktop.
Save mbruzek/acaaa5fd69361eafc4c7 to your computer and use it in GitHub Desktop.
Download ZIP
Raw
pull-request.md

This is a follow up to the original Juju pull request: kubernetes/kubernetes#5414

On @erictune's suggestion we are adding the charms and a bundle to keep in better sync with the kubernetes project.

cluster/kube-up.sh will deploy the local charms and the bundle from the kubernetes repository and stand up a cluster.

We have done a lot of work to update the cluster/juju/util.sh to the v0.15.0 release that changed the API and a lot of things.

  • Added the Juju charms to cluster/juju/charms directory.
  • Added a Juju bundle to cluster/juju/bundles directory.
  • Changing cluster/juju/util.sh script uses the bundle to deploy the charms from the kubernetes repository.
  • Updated the charms and bundle to work with v0.15.0 changes.
  • Updated the charms to use v1beta3 which is default in v0.15.0.

We are still seeing some issues with the validate-cluster.sh script ('Error: no resource "cs" has been defined') but I don't think that is related to our changes in this pull request. Manual testing shows that we can create objects in the kubernetes cluster.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment