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

Copy file from pod into gefyra container "--file-from" #501

Open
buschNT opened this issue Nov 6, 2023 · 2 comments
Open

Copy file from pod into gefyra container "--file-from" #501

buschNT opened this issue Nov 6, 2023 · 2 comments
Assignees
Labels
enhancement 🎉 New feature or request

Comments

@buschNT
Copy link
Collaborator

buschNT commented Nov 6, 2023

What is the new feature about?

Sometimes it is necessary to copy a file from an existing pod into the container run with gefyra. As this is similar to the exiting --env-from cli option, I imagine a similar option for copying files (directories?)?

Why would such a feature be important to you?

I am working with a setup which requires me to have a specific file from a pod/deployment which is created/mounted by k8s. Thus, I have to manually copy this file every time I create a new container with gefyra run . With an --file-from option, it would be possible to include this manual step within the run command.

Anything else we need to know?

No response

@buschNT buschNT added the enhancement 🎉 New feature or request label Nov 6, 2023
@crkurz
Copy link

crkurz commented Oct 2, 2024

+1 - doing the same via kubectl-exec-tar, which results in quite some work, as file-setup is different for every container

buschNT added a commit that referenced this issue Oct 23, 2024
- add --file-from flag to run command
- copies file/directory from cluster using tar
- writes tar into local container
buschNT added a commit that referenced this issue Oct 23, 2024
- remove del line, not sure why this was an issue
@buschNT
Copy link
Collaborator Author

buschNT commented Oct 23, 2024

I opened a draft PR with the --file-from flag #725. @SteinRobert / @Schille maybe you can take a look provide some feedback.

buschNT added a commit that referenced this issue Oct 25, 2024
- implemented feedback from PR
- add tests for parse_file_from
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement 🎉 New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants