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

update bootstrappolicy/dead addDeadClusterRole to include systemOnly annotation #15320

Merged
merged 1 commit into from
Jul 19, 2017
Merged

Conversation

liggitt
Copy link
Contributor

@liggitt liggitt commented Jul 18, 2017

Roles that have been replaced by kube controller roles should receive
the systemOnly annotation to ensure they are not visible to typical
end users (such as those who would use the web console).

  • updates bootstrappolicy/dead addDeadClusterRole
  • updates bootstrappolicy/web_console_role_test TestSystemOnlyRoles
    • still skips controller roles (maintenance simplicity), but will throw an error if the annotation is missing

…annotation

Roles that have been replaced by kube controller roles should receive
the systemOnly annotation to ensure they are not visible to typical
end users (such as those who would use the web console).

- updates bootstrappolicy/dead addDeadClusterRole
- updates bootstrappolicy/web_console_role_test TestSystemOnlyRoles
	- still skips controller roles (maintenance simplicity), but will throw an error if the annotation is missing
@liggitt
Copy link
Contributor Author

liggitt commented Jul 18, 2017

cherry-pick of #15247 onto 3.6

@liggitt
Copy link
Contributor Author

liggitt commented Jul 18, 2017

[test]

@openshift-bot
Copy link
Contributor

Evaluated for origin test up to f2263f6

@liggitt
Copy link
Contributor Author

liggitt commented Jul 18, 2017

[merge][severity:blocker]

@openshift-bot
Copy link
Contributor

Evaluated for origin merge up to f2263f6

@openshift-bot
Copy link
Contributor

openshift-bot commented Jul 18, 2017

continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/merge_pull_request_origin/1304/) (Base Commit: 20e72f7) (PR Branch Commit: f2263f6) (Extended Tests: blocker) (Image: devenv-rhel7_6451)

@openshift-bot openshift-bot merged commit 12575c5 into openshift:release-3.6 Jul 19, 2017
@openshift-bot
Copy link
Contributor

continuous-integration/openshift-jenkins/test SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pull_request_origin/3250/) (Base Commit: 20e72f7) (PR Branch Commit: f2263f6)

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

Successfully merging this pull request may close these issues.

3 participants