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

release kargo-render binary #284

Closed
joaoestrela opened this issue May 7, 2024 · 4 comments
Closed

release kargo-render binary #284

joaoestrela opened this issue May 7, 2024 · 4 comments

Comments

@joaoestrela
Copy link
Contributor

Currently, only the Docker image is published. It would be great to also provide a binary release. Currently, it's challenging to pass through Helm credentials in a GitHub Action because it's within a container with its own Helm installation.

Having a binary release would allow the kargo-render binary to be used within the workflow authentication context.

@krancour
Copy link
Member

Currently, it's challenging to pass through Helm credentials in a GitHub Action because it's within a container with its own Helm installation.

Kargo proper just did something about this a few days ago: akuity/kargo#1977

I had been imagining that we'd soon create a native way for chart repo creds to be passed directly to Kargo Render as well, but hadn't yet put much thought into how.

On some level, I'd prefer solving that over simply publishing the binary, since there is a reason that we don't. (Packaging it in an image with git, helm, and kustomize binaries is the easiest way to guarantee Kargo Render has compatible versions of those binaries available to it.)

I'll start thinking about a clean way of passing the creds through.

@joaoestrela
Copy link
Contributor Author

Fair points.
My suggestion was intended as an additional option for those who might prefer a more customised approach, but I guess that compiling it themselves is a viable solution.
Thanks!

@krancour
Copy link
Member

compiling it themselves is a viable solution

You can also copy it out of a container as long as you're looking for linux/amd64 (or linux/arm64). Presumably in an Action, linux/amd64 would be exactly what you need.

@krancour
Copy link
Member

To put some closure on this, solving for #226 will obviate the need for this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants