Skip to content

add: helm binary download cmd #4526

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

Merged
merged 1 commit into from
Oct 19, 2022
Merged

add: helm binary download cmd #4526

merged 1 commit into from
Oct 19, 2022

Conversation

ericpaulsen
Copy link
Member

@ericpaulsen ericpaulsen commented Oct 13, 2022

adds a command for downloading the helm binary.

customers are typically installing the helm chart via a VM or other remote client, and it isn't entirely intuitive as to how to do this from the GitHub releases page.

@ericpaulsen ericpaulsen added the docs Area: coder.com/docs label Oct 13, 2022
@ericpaulsen ericpaulsen self-assigned this Oct 13, 2022
@sreya
Copy link
Collaborator

sreya commented Oct 13, 2022

@ericpaulsen i'll make you a deal, i'll implement some CI to push our helm chart to https://helm.coder.com and then we can update this to direct people to our helm repo.

cc @deansheather @kylecarbs any reservations with me pushing our v2 helm chart to our helm repo?

@ericpaulsen
Copy link
Member Author

@sreya now that's an offer I can't refuse

@deansheather
Copy link
Member

any reservations with me pushing our v2 helm chart to our helm repo?

What do you want to call the package? coderv2?

@sreya
Copy link
Collaborator

sreya commented Oct 13, 2022 via email

@deansheather
Copy link
Member

I think it's kinda weird tbh to call it oss when it's the enterprise binary

@sreya
Copy link
Collaborator

sreya commented Oct 13, 2022

It's still oss... @kylecarbs @ammario what should we call the helm repo for v2?

@ammario
Copy link
Member

ammario commented Oct 13, 2022

oss makes sense to me since it's consistent with our docs.

@ammario ammario removed their request for review October 17, 2022 15:26
Copy link
Member

@bpmct bpmct left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Gonna merge this for now until we have the hosted helm chart. We'll need to rewrite these docs anyways

@bpmct bpmct merged commit f9480ed into main Oct 19, 2022
@bpmct bpmct deleted the helm-binary branch October 19, 2022 16:39
@github-actions github-actions bot locked and limited conversation to collaborators Oct 19, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
docs Area: coder.com/docs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants