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

kpt fn run should support running containerized fns on K8s without docker #2158

Open
jlewi opened this issue Jun 4, 2021 · 2 comments
Open
Labels
area/fn-runtime KRM function runtime customer deep engagement enhancement New feature or request p1 triaged Issue has been triaged by adding an `area/` label

Comments

@jlewi
Copy link

jlewi commented Jun 4, 2021

Is your feature request related to a problem? Please describe.

I would like to use kpt fn run inside CI/CD pipelines that run in a cluster. kpt fn run currently relies on shelling out to docker. As a result, when running in cluster one needs to run docker in docker as illustrated by the tekton example.

My understanding is that mounting the docker socket into a container creates security concerns and is generally not best practice.

Describe the solution you'd like

I'd like kpt fn run to support launching containerized fns on a cluster (e.g. by creating a Pod) without requiring docker. Ideally the experience would be similar to skaffold build in that kpt would start the pod, upload the YAMLs to be processed (e.g. output of kpt fn source) and then download the resulting output.

Describe alternatives you've considered

My current solution is to build my own binary for running my custom kpt functions which links in my functions so I don't need to run containerized.

I've also considered running docker in docker as described in the kpt docs
https://googlecontainertools.github.io/kpt/guides/consumer/function/export/tekton/

Originally filed as
kubernetes-sigs/kustomize#3958

@jlewi jlewi added the enhancement New feature or request label Jun 4, 2021
@mikebz mikebz modified the milestones: v1.0 m4, v1.1 Jun 4, 2021
@mikebz mikebz added customer deep engagement triaged Issue has been triaged by adding an `area/` label labels Jun 4, 2021
@mikebz mikebz removed this from the v1.1 milestone Jul 14, 2021
aodinokov added a commit to aodinokov/kustomize that referenced this issue Oct 20, 2021
aodinokov added a commit to aodinokov/kustomize that referenced this issue Oct 20, 2021
aodinokov added a commit to aodinokov/kustomize that referenced this issue Oct 20, 2021
aodinokov added a commit to aodinokov/kustomize that referenced this issue Oct 20, 2021
aodinokov added a commit to aodinokov/kustomize that referenced this issue Oct 20, 2021
aodinokov added a commit to aodinokov/kustomize that referenced this issue Oct 22, 2021
aodinokov added a commit to aodinokov/kustomize that referenced this issue Oct 22, 2021
aodinokov added a commit to aodinokov/kustomize that referenced this issue Oct 22, 2021
aodinokov added a commit to aodinokov/kustomize that referenced this issue Oct 22, 2021
aodinokov added a commit to aodinokov/kustomize that referenced this issue Oct 27, 2021
aodinokov added a commit to aodinokov/kustomize that referenced this issue Oct 28, 2021
aodinokov added a commit to aodinokov/kustomize that referenced this issue Oct 28, 2021
aodinokov added a commit to aodinokov/kustomize that referenced this issue Oct 28, 2021
aodinokov added a commit to aodinokov/kustomize that referenced this issue Oct 28, 2021
aodinokov added a commit to aodinokov/kustomize that referenced this issue Oct 28, 2021
aodinokov added a commit to aodinokov/kustomize that referenced this issue Oct 28, 2021
aodinokov added a commit to aodinokov/kustomize that referenced this issue Oct 29, 2021
aodinokov added a commit to aodinokov/kustomize that referenced this issue Oct 29, 2021
aodinokov added a commit to aodinokov/kustomize that referenced this issue Oct 29, 2021
aodinokov added a commit to aodinokov/kustomize that referenced this issue Oct 29, 2021
aodinokov added a commit to aodinokov/kustomize that referenced this issue Oct 29, 2021
aodinokov added a commit to aodinokov/kustomize that referenced this issue Oct 29, 2021
aodinokov added a commit to aodinokov/kustomize that referenced this issue Nov 2, 2021
aodinokov added a commit to aodinokov/kustomize that referenced this issue Nov 3, 2021
aodinokov added a commit to aodinokov/kustomize that referenced this issue Nov 3, 2021
aodinokov added a commit to aodinokov/kustomize that referenced this issue Nov 5, 2021
aodinokov added a commit to aodinokov/kustomize that referenced this issue Nov 5, 2021
@bgrant0607
Copy link
Contributor

See also #2469 and #2567

@droot droot added the area/fn-runtime KRM function runtime label May 27, 2022
@droot droot added p1 and removed area/hydrate labels May 27, 2022
@bygui86
Copy link

bygui86 commented Jun 28, 2022

hi guys, what's the status of this feature?

@mortent mortent added this to kpt Jan 21, 2023
@mortent mortent moved this to Backlog in kpt Jan 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/fn-runtime KRM function runtime customer deep engagement enhancement New feature or request p1 triaged Issue has been triaged by adding an `area/` label
Projects
None yet
Development

No branches or pull requests

6 participants