-
Notifications
You must be signed in to change notification settings - Fork 917
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
chore: Removing PSS patches #2986
Conversation
Thank you. you need to sigh for the DCO
|
Signed-off-by: biswassri <srijoni.biswas1994@gmail.com>
9659683
to
2e1ec60
Compare
can you check which pod does not have the full security context? Maybe jwks-proxy? |
Yes i think https://github.com/kubeflow/manifests/blob/master/common/oauth2-proxy/components/cluster-jwks-proxy/cluster-jwks-proxy.yaml needs the securitycontext.
|
@biswassri do you want to continue here? |
Signed-off-by: biswassri <srijoni.biswas1994@gmail.com>
@juliusvonkohout Apologies for the delay. Updated the securitycontext for the said cluster. Waiting for the CI to pass now. |
Thank you very much. The next step then is to fix the seccomprofile as described in the other issues and PRs. To fix it for the istio-ingressgateway as well. |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: juliusvonkohout The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Pull Request Template for Kubeflow manifests Issues
✏️ A brief description of the changes
Removed the PSS security patches that are no longer needed based on description in this issue #2970
📦 List any dependencies that are required for this change
🐛 If this PR is related to an issue, please put the link to the issue here.
#2970
✅ Contributor checklist
DCO
check)