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

feat: Conditional automount at execution #17

Closed
k-kinzal opened this issue Dec 16, 2018 · 6 comments
Closed

feat: Conditional automount at execution #17

k-kinzal opened this issue Dec 16, 2018 · 6 comments
Labels

Comments

@k-kinzal
Copy link
Owner

I had a problem.

/usr/local/bin/kubectl:
  image: chatwork/kubectl
  tag: 1.11.2
  env:
    KUBECONFIG: $KUBECONFIG
  volume:
  - $HOME/.kube:/root/.kube
  - $PWD:/kube
  workdir: /kube
$ KUBECONFIG=$HOME/Downloads/kube-config.cfg kubectl config view --flatten

it does not work because it is not mounted $HOME/Downloads/kube-config.cfg.

it possible to solve this problem if it can be automatically mounted under conditions at execution.

@k-kinzal
Copy link
Owner Author

There is a problem with conditional automount at execution.

The first problem is that multiple things are injected at execution.
Can we solve the injection of Environment Variable, command arguments, configuration file of command?
Should we support what changes with commands?

The second problem is that scripts that use are complicated.
In order to make debugging easier, I want to prevent complication of scripts.

@k-kinzal
Copy link
Owner Author

The third problem seems to be a problem with resolution of dependency.
If there is command dependence, the execution environment will be on docker.
In this case, we can not mount because the host path is unknown.

@k-kinzal
Copy link
Owner Author

Hope for a solution in the future

@k-kinzal
Copy link
Owner Author

Usecases

  • Symbolic link in mount dir
  • Environment variable of FOO=bar cmd
  • Arguments of curl ... -o /path/to

Repository owner deleted a comment from cw-ozaki Dec 20, 2018
@k-kinzal
Copy link
Owner Author

We will present a provisional solution to this problem in README.md.

@k-kinzal
Copy link
Owner Author

k-kinzal commented Jan 6, 2019

It closed because it derived from another issue.

@k-kinzal k-kinzal closed this as completed Jan 6, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant