Skip to content

Add projected volume plugin into correct SCCs #14147

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

Merged
merged 1 commit into from
May 13, 2017

Conversation

jpeeler
Copy link

@jpeeler jpeeler commented May 11, 2017

Adds the projected volume plugin into the correct SCCs (everything but
privileged, because that has access to all volumes).

Closes bugzilla bug 1448816.

@jpeeler
Copy link
Author

jpeeler commented May 11, 2017

[test]

@openshift-bot
Copy link
Contributor

Evaluated for origin test up to aadcd49

@derekwaynecarr
Copy link
Member

[merge]

@openshift-bot
Copy link
Contributor

Evaluated for origin merge up to aadcd49

@openshift-bot
Copy link
Contributor

continuous-integration/openshift-jenkins/test FAILURE (https://ci.openshift.redhat.com/jenkins/job/test_pull_request_origin/1362/) (Base Commit: 2628c77)

@derekwaynecarr
Copy link
Member

re [merge]

@openshift-bot
Copy link
Contributor

openshift-bot commented May 13, 2017

continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/merge_pull_request_origin/628/) (Base Commit: 0641fae) (Image: devenv-rhel7_6228)

@openshift-bot openshift-bot merged commit ccad8d4 into openshift:master May 13, 2017
k8s-github-robot pushed a commit to kubernetes/kubernetes that referenced this pull request May 20, 2017
Automatic merge from submit-queue (batch tested with PRs 45346, 45903, 45958, 46042, 45975)

examples/podsecuritypolicy/rbac: allow to use projected volumes in restricted PSP

**What this PR does / why we need it**:
This PR modifies `restricted` PSP to allow `projected` volume type. No need to modify `privileged` PSP because it already allows all volume types.

It should not add any harm because `projected` uses configmaps, downward API, and secrets that are already permitted.

**Special notes for your reviewer**:
This was inspired by similar change in the OpenShift: openshift/origin#14147

**Release note**:
```release-note
NONE
```

PTAL @pweil- @derekwaynecarr 
CC @mfojtik
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