-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Containers should not be allowed to directly create K8s objects #254
Comments
Ark-kun
assigned yebrahim, IronPan, gaoning777, paveldournov, qimingj, vicaire and rileyjbauer
Nov 14, 2018
Isn't this what RBAC and namespaces are for? What does it mean to send a CRD to some backend? Why not just let pipelines run in a user selected namespace with suitable, user controlled RBAC permissions to limit what pipelines can do? |
+1 on @jlewi's comment. I am not in favor of containers calling the Pipeline API Server. I think we have two options:
|
Resolving since there is no outstanding item. |
vicaire
unassigned yebrahim, IronPan, gaoning777, paveldournov, vicaire and rileyjbauer
Mar 27, 2019
magdalenakuhn17
pushed a commit
to magdalenakuhn17/pipelines
that referenced
this issue
Oct 22, 2023
Now that we have knative 0.7 the workaround is no longer needed!
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Currently our containers are not really isolated. They have full access to network, GCS, other GCP services and can even create, delete and modify other K8s objects as in #231
This was discussed during a design meeting and as far as I remember the consensus was that it's too dangerous and insecure. Worse still, its really hard to put this genie back into the bottle once it's out and people start using this.
As we discussed, all CRDs should be created through the pipelines system backend endpoint or admission controller and the Pod needs to identify itself.
Technical solution:
gcr.io/ml-pipelines/
container image bucket).command
property of the container should not be set.The text was updated successfully, but these errors were encountered: