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

Bring back static binaries #838

Closed
RobKenis opened this issue May 25, 2022 · 2 comments · Fixed by #1223
Closed

Bring back static binaries #838

RobKenis opened this issue May 25, 2022 · 2 comments · Fixed by #1223

Comments

@RobKenis
Copy link

RobKenis commented May 25, 2022

Describe the bug/feature
We depend on the static binary for 0.30.0. Our use case is to use kapitan in combination with ArgoCD. We used to download the binary in an InitContainer and mount the binary in ArgoCD so it was able to execute Kapitan commands. Doesn't really work that great when using pip, because we don't have a Python environment inside the ArgoCD container.

To Reproduce
Steps to reproduce the behavior:

  1. Install the package with pip
  2. It's not a binary.
@ramaro
Copy link
Member

ramaro commented May 25, 2022

Hey @RobKenis, the binary was removed in 0.30.0 because it was simply not sustainable to make it work. See details in #740

I currently run kapitan 0.30.0 in production with ArgoCD by prepackaging kapitan as a pex myself. This is fairly easy:

pip install pex
pex kapitan==0.30.0 --python=python3 -m kapitan -o kapitan.pex

You can run kapitan.pex as long as the python version used to build it is the same when you run it. E.g. running kapitan.pex with python 3.8 requires kapitan.pex to be built with python 3.8

ArgoCD's repo server container image does include python by default so this will fairly trivial to implement.

We are looking into releasing pex binaries (or other similar approaches) in further releases but I don't know if we can commit to having a fully functional "proper" binary. Naturally, we are looking for ideas and contributions to make this work best for everyone.

@MatteoVoges
Copy link
Contributor

Closing due newer version in #923

@MatteoVoges MatteoVoges closed this as not planned Won't fix, can't repro, duplicate, stale Apr 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants