Skip to content
This repository has been archived by the owner on Nov 30, 2021. It is now read-only.

RBAC support #812

Merged
merged 1 commit into from
May 31, 2017
Merged

RBAC support #812

merged 1 commit into from
May 31, 2017

Conversation

Bregor
Copy link
Contributor

@Bregor Bregor commented May 12, 2017

@vdice vdice requested review from mboersma and vdice May 12, 2017 21:58
@vdice vdice added this to the v2.15 milestone May 12, 2017
@vdice
Copy link
Member

vdice commented May 12, 2017

edit: latest controller-pr chart version updated

A note for how to install the entire Workflow chart with relevant RBAC-ified sub-chart versions added:

If you haven't done so, add the necessary chart repos for all affected components to helm:

$ for repo in workflow-pr fluentd-pr router-pr monitor-pr controller-pr builder-pr; do \
    helm repo add $repo https://charts.deis.com/$repo; \
done

Next, fetch the Workflow PR chart from this commit:

$ helm fetch --untar workflow-pr/workflow --version v2.14.1-20170512231142-sha.356374f

Substitute the following requirements.yaml in place of the existing workflow/requirements.yaml:

dependencies:
  - name: builder
    version: "v2.10.2-20170512171907-sha.c891804"
    repository: https://charts.deis.com/builder-pr
  - name: slugbuilder
    version: ">=v2.4.12-dev"
    repository: https://charts.deis.com/slugbuilder-dev
  - name: dockerbuilder
    version: ">=v2.7.2-dev"
    repository: https://charts.deis.com/dockerbuilder-dev
  - name: controller
    version: "v2.14.2-20170526164617-sha.c321669"
    repository: https://charts.deis.com/controller-pr
  - name: slugrunner
    version: ">=v2.3.0-dev"
    repository: https://charts.deis.com/slugrunner-dev
  - name: database
    version: ">=v2.5.3-dev"
    repository: https://charts.deis.com/database-dev
  - name: fluentd
    version: "v2.9.1-20170512163817-sha.464bbd9"
    repository: https://charts.deis.com/fluentd-pr
  - name: redis
    version: ">=v2.2.6-dev"
    repository: https://charts.deis.com/redis-dev
  - name: logger
    version: ">=v2.4.3-dev"
    repository: https://charts.deis.com/logger-dev
  - name: minio
    version: ">=v2.3.5-dev"
    repository: https://charts.deis.com/minio-dev
  - name: monitor
    version: "v2.9.1-20170512172938-sha.ff4058d"
    repository: https://charts.deis.com/monitor-pr
  - name: nsqd
    version: ">=v2.2.7-dev"
    repository: https://charts.deis.com/nsqd-dev
  - name: registry
    version: ">=v2.4.0-dev"
    repository: https://charts.deis.com/registry-dev
  - name: registry-proxy
    version: ">=v1.3.0-dev"
    repository: https://charts.deis.com/registry-proxy-dev
  - name: registry-token-refresher
    version: ">=v1.1.2-dev"
    repository: https://charts.deis.com/registry-token-refresher-dev
  - name: router
    version: "v2.12.2-20170516193608-sha.c9769fe"
    repository: https://charts.deis.com/router-pr
  - name: workflow-manager
    version: ">=v2.5.0-dev"
    repository: https://charts.deis.com/workflow-manager-dev

Finally, update with the specified dependencies and install the chart:

$ helm dependency update workflow
...
$ helm install workflow --name deis-workflow --namespace deis

If cluster ready for RBAC (note this PR adds docs around this process):

$ helm install workflow --name deis-workflow --namespace deis --set global.use_rbac=true

@vdice
Copy link
Member

vdice commented May 16, 2017

Note: router chart version updated in requirements.yaml from above w/ latest changes pushed to deis/router#344

@vdice
Copy link
Member

vdice commented May 25, 2017

Closes #809

@vdice
Copy link
Member

vdice commented May 26, 2017

latest controller chart version added to #812 (comment)

@vdice
Copy link
Member

vdice commented May 31, 2017

Thanks so much for the contribution @Bregor ! This was a huge effort and very much appreciated. Merging this and associated PRs and we can do a final round of testing once we have a 2.15 release candidate chart...

@Bregor
Copy link
Contributor Author

Bregor commented May 31, 2017

@vdice great! Thanks for you help and patience!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants