Skip to content

Commit

Permalink
Add docs, import cleanup.
Browse files Browse the repository at this point in the history
Described `helm test` command.
  • Loading branch information
aLekSer committed Apr 21, 2020
1 parent fb5c975 commit 21d38e7
Show file tree
Hide file tree
Showing 2 changed files with 23 additions and 2 deletions.
3 changes: 1 addition & 2 deletions examples/crd-client/main.go
Original file line number Diff line number Diff line change
Expand Up @@ -18,11 +18,10 @@ import (
"strings"
"time"

"github.com/sirupsen/logrus"

agonesv1 "agones.dev/agones/pkg/apis/agones/v1"
"agones.dev/agones/pkg/client/clientset/versioned"
"agones.dev/agones/pkg/util/runtime"
"github.com/sirupsen/logrus"
"github.com/spf13/pflag"
"github.com/spf13/viper"
corev1 "k8s.io/api/core/v1"
Expand Down
22 changes: 22 additions & 0 deletions site/content/en/docs/Installation/Install Agones/helm.md
Original file line number Diff line number Diff line change
Expand Up @@ -207,6 +207,28 @@ $ helm install --name my-release --namespace agones-system -f values.yaml agones
You can use the default {{< ghlink href="install/helm/agones/values.yaml" >}}values.yaml{{< /ghlink >}}
{{< /alert >}}

{{% feature publishVersion="1.6.0" %}}
Check the Agones installation by running the following command:
```bash
$ helm test my-release --cleanup
RUNNING: agones-test
PASSED: agones-test
```

This test would create a `GameServer` resource and delete it afterwards.

{{< alert title="Tip" color="info">}}
If you receive the following error:
```
RUNNING: agones-test
ERROR: pods "agones-test" already exists
Error: 1 test(s) failed
```
That mean that you skiped `--cleanup` flag and you should either delete `agones-test` pod manually or run with `--cleanup` the test two more times.
{{< /alert >}}

{{% /feature %}}

## TLS Certificates

By default agones chart generates tls certificates used by the admission controller, while this is handy, it requires the agones controller to restart on each `helm upgrade` command.
Expand Down

0 comments on commit 21d38e7

Please sign in to comment.