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

chore: Add securitycontext for PSS PoC (rootless Kubeflow) #11462

Merged
merged 2 commits into from
Dec 20, 2024

Conversation

juliusvonkohout
Copy link
Member

@juliusvonkohout juliusvonkohout commented Dec 12, 2024

Description of your changes:

upstream what we have in https://github.com/kubeflow/manifests/tree/master/contrib/security/PSS to make PSS enforcable
and enjoy a rootless Kubeflow kubeflow/manifests#2528

Checklist:

@juliusvonkohout juliusvonkohout marked this pull request as draft December 12, 2024 16:52
@juliusvonkohout juliusvonkohout changed the title chore: Update ml-pipeline-persistenceagent-deployment.yaml with securitycontext for PSS chore: Add securitycontext for PSS PoC Dec 12, 2024
@juliusvonkohout juliusvonkohout changed the title chore: Add securitycontext for PSS PoC chore: Add securitycontext for PSS PoC (rootless Kubeflow) Dec 12, 2024
@google-oss-prow google-oss-prow bot added size/M and removed size/S labels Dec 15, 2024
@biswassri biswassri force-pushed the patch-26 branch 2 times, most recently from 580147a to df2c7fc Compare December 15, 2024 07:50
@juliusvonkohout
Copy link
Member Author

juliusvonkohout commented Dec 16, 2024

@biswassri

Thank you.

It is essential, that you sign your commits or you will break the DCO test and we cannot merge
https://github.com/kubeflow/pipelines/pull/11462/checks?check_run_id=34430161221

Remaining patches for pipelines from https://github.com/kubeflow/manifests/tree/master/contrib/security/PSS/patches:

  • minio
  • metadata-*
  • mysql
  • metacontroller
  • workflow-controller

Copy link

Approvals successfully granted for pending runs.

@juliusvonkohout juliusvonkohout force-pushed the patch-26 branch 3 times, most recently from 9b15e90 to cdb5c46 Compare December 16, 2024 14:51
@google-oss-prow google-oss-prow bot added size/L and removed size/M labels Dec 17, 2024
@biswassri
Copy link
Contributor

@juliusvonkohout I think I got in most of the patches in. Please let me know if I got something wrong. Also I wasn't sure where to update metacontroller patch I wasn't sure about which file.

@juliusvonkohout
Copy link
Member Author

juliusvonkohout commented Dec 17, 2024

@biswassri please check out https://github.com/juliusvonkohout/pipelines/blob/patch-26/manifests/kustomize/third-party/metacontroller/base/stateful-set.yaml. We should also update it to the latest release in a separate PR https://github.com/metacontroller/metacontroller/releases/tag/v4.11.21. So please modify in a separate PR https://github.com/juliusvonkohout/pipelines/blob/5587d9acc25dd96b596a491af3f38f7c3f885469/manifests/kustomize/third-party/metacontroller/base/stateful-set.yaml#L42 from docker.io/metacontrollerio/metacontroller:v2.0.4 to ghcr.io/metacontroller/metacontroller:v4.11.21

@juliusvonkohout juliusvonkohout marked this pull request as ready for review December 17, 2024 11:38
@juliusvonkohout
Copy link
Member Author

/ok-to-test
/retest

@juliusvonkohout
Copy link
Member Author

@rimolive @HumairAK we need to update metacontroller since dockerhub will be removed in the future according to https://github.com/metacontroller/metacontroller/releases/tag/v4.11.21

@biswassri biswassri force-pushed the patch-26 branch 2 times, most recently from 4aed65a to 7242a2f Compare December 19, 2024 06:47
@biswassri
Copy link
Contributor

biswassri commented Dec 19, 2024

@juliusvonkohout I updated the metacontroller security patch as well. Created a separate PR for the image update. #11474

@hbelmiro
Copy link
Contributor

/ok-to-test

Copy link

Approvals successfully granted for pending runs.

Signed-off-by: biswassri <[email protected]>

Update ml-pipeline-scheduledworkflow-deployment.yaml

Signed-off-by: juliusvonkohout <[email protected]>

Update ml-pipeline-persistenceagent-deployment.yaml

Upstreaming off pss patches

Signed-off-by: Julius von Kohout <[email protected]>

Updating server,ui,visualization,veiwercrd deployment yaml

Signed-off-by: biswassri <[email protected]>
Signed-off-by: juliusvonkohout <[email protected]>

Updating remaining PSS patches

Signed-off-by: biswassri <[email protected]>
@juliusvonkohout
Copy link
Member Author

juliusvonkohout commented Dec 19, 2024

@biswassri I have squashed all commits and now the DCO is satisfied :-D

@juliusvonkohout
Copy link
Member Author

@HumairAK @rimolive for approval

@juliusvonkohout
Copy link
Member Author

/assign @HumairAK

@juliusvonkohout
Copy link
Member Author

/assign @hbelmiro

@juliusvonkohout
Copy link
Member Author

juliusvonkohout commented Dec 19, 2024

@rimolive this is a blocker for kubeflow/manifests 1.10 release / roadmap and kubeflow/manifests#2528

Signed-off-by: juliusvonkohout <[email protected]>
@HumairAK
Copy link
Collaborator

/lgtm
/approve

Thanks @juliusvonkohout !

@google-oss-prow google-oss-prow bot added the lgtm label Dec 20, 2024
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: HumairAK

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 /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@google-oss-prow google-oss-prow bot merged commit f3cb244 into kubeflow:master Dec 20, 2024
29 checks passed
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