Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

RBAC support #195

Merged
merged 1 commit into from
May 31, 2017
Merged

RBAC support #195

merged 1 commit into from
May 31, 2017

Conversation

Bregor
Copy link
Contributor

@Bregor Bregor commented May 7, 2017

With this change deis-monitor-telegraf became available to work in RBAC-only clusters

Works with both Kubernetes 1.5 and 1.6 (see templates/_helpers.tmpl for details)
Actually tested with 1.5.7 and 1.6.2

ClusterRole allows deis-monitor-telegraf:

  • pods: get

@deis-admin
Copy link

Thanks for the contribution! Please ensure your commits follow our style guide. This code will be tested once a Deis maintainer reviews it.

@vdice vdice added this to the v2.15 milestone May 8, 2017
@vdice
Copy link
Member

vdice commented May 8, 2017

Jenkins, OK to test

@vdice
Copy link
Member

vdice commented May 9, 2017

Jenkins, add to whitelist

1 similar comment
@vdice
Copy link
Member

vdice commented May 9, 2017

Jenkins, add to whitelist

With this change deis-monitor-telegraf became available to work in RBAC-only clusters

Works with both Kubernetes 1.5 and 1.6 (see templates/_helpers.tmpl for details)
Actually tested with 1.5.7 and 1.6.2

ClusterRole allows deis-monitor-telegraf:
- pods: get
@Bregor Bregor mentioned this pull request May 12, 2017
@vdice vdice requested review from vdice and mboersma May 16, 2017 17:01
@vdice vdice added the LGTM1 label May 16, 2017
@vdice vdice merged commit b4b3aef into deis:master May 31, 2017
vdice pushed a commit to deis/workflow that referenced this pull request May 31, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants