Add helm chart #322
Add helm chart #322
Conversation
|
Thanks @avantgardnerio. I will review this over the weekend. It would be helpful if you could add brief instructions in the user guide on how to deploy Ballista with Helm. That will make the review go quicker ... I haven't used helm in several years. |
helm/README.md
Outdated
| ## Prerequisites | ||
|
|
||
| ```shell | ||
| sudo apt install kubectl docker.io docker-compose |
There was a problem hiding this comment.
I tried this on Ubuntu 20.04.4 LTS and got:
$ sudo apt install kubectl docker.io docker-compose
[sudo] password for andy:
Reading package lists... Done
Building dependency tree
Reading state information... Done
No apt package "kubectl", but there is a snap with that name.
Try "snap install kubectl"
E: Unable to locate package kubectl
There was a problem hiding this comment.
We can just document this of course.
| ## Build | ||
|
|
||
| ```shell | ||
| dev/build-ballista-docker.sh |
There was a problem hiding this comment.
This fails for me with:
ERROR: Version in "./docker-compose.yml" is unsupported. You might be seeing this error because you're using the wrong Compose file version. Either specify a supported version (e.g "2.2" or "3.3") and place your service definitions under the `services` key, or omit the `version` key and place your service definitions at the root of the file to use version 1.
Maybe we can downgrade to 3.3 unless there are specific features we need from newer versions?
There was a problem hiding this comment.
I suppose there is a wider discussion we should have about what minimum versions of different operations systems we plan on supporting.
There was a problem hiding this comment.
PR to downgrade to version 3.3 - #329
| including all the major cloud providers. | ||
| For the purposes of this documentation, we will use Kubernetes-in-Docker (kind) to install locally. | ||
|
|
||
| ## Prerequisites |
There was a problem hiding this comment.
We need to install Helm as well. I used sudo snap install helm --classic
There was a problem hiding this comment.
This works really well. Thanks, @avantgardnerio!
I left some suggestions for the user guide content.
Co-authored-by: Andy Grove <[email protected]>
Which issue does this PR close?
Closes #321.
Rationale for this change
Described in the issue.
What changes are included in this PR?
A Helm chart.
Are there any user-facing changes?
They will be able to deploy a Ballista cluster with a single command.