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

UPSTREAM: 47822: Separate serviceaccount and secret storage config #14838

Conversation

php-coder
Copy link
Contributor

Backport of kubernetes/kubernetes#47822 that is required for resources encryption in etcd.

PTAL @liggitt @smarterclayton
CC @simo5

@liggitt
Copy link
Contributor

liggitt commented Jun 22, 2017

LGTM [merge][severity:blocker]

@openshift-bot
Copy link
Contributor

[Test]ing while waiting on the merge queue

@openshift-bot
Copy link
Contributor

Evaluated for origin test up to ab93dcd

@openshift-bot
Copy link
Contributor

continuous-integration/openshift-jenkins/test SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pull_request_origin/2521/) (Base Commit: 4423ff5) (PR Branch Commit: ab93dcd)

@php-coder
Copy link
Contributor Author

@0xmichalis
Copy link
Contributor

[merge][severity:blocker]

@php-coder
Copy link
Contributor Author

php-coder commented Jul 7, 2017

Logs are unavailable now. I'd suggest to rerun merge process.

@openshift-bot
Copy link
Contributor

Evaluated for origin merge up to ab93dcd

@openshift-bot
Copy link
Contributor

openshift-bot commented Jul 7, 2017

continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/merge_pull_request_origin/1250/) (Base Commit: 06c65e8) (PR Branch Commit: ab93dcd) (Extended Tests: blocker) (Image: devenv-rhel7_6427)

@openshift-bot openshift-bot merged commit e0c904f into openshift:master Jul 7, 2017
@php-coder php-coder deleted the backport_sa_and_secrets_storage_config branch July 7, 2017 17:43
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.

4 participants