search

temporalio / helm-charts

Temporal Helm charts
MIT License
276 stars 315 forks source link

Setup for `ci install`. #522

Closed robholland closed 3 days ago

robholland commented 4 days ago

Adds a test so that helm test now checks that the cluster is healthy after the system is deployed.

Refactors server-job to remove the use of helm hooks which cause lot of users pain and don't work with --wait, or the mechanism that helm test uses. This should allow deployment to work when done via Terraform/Pulumi and other similar system which use a wait-style approach.

Improve the handling of elasticsearch by treating it just like the other drivers. Also uses a secret for Elasticsearch password rather than having it in plain text, inline with the other drivers.

Correctly handle the schema.createDatabase setting which previously had edge cases.

Note: This PR does not enable ct install to run in any PR or release workflows, but that will be done soon in another PR.

robholland commented 4 days ago

Fixes: #506.

axfelix commented 4 days ago 
$ helm test temporaltest
NAME: temporaltest
LAST DEPLOYED: Wed Jul  3 09:41:29 2024
NAMESPACE: default
STATUS: deployed
REVISION: 1
TEST SUITE:     temporaltest-opsyo-test
Last Started:   Wed Jul  3 09:48:30 2024
Last Completed: Wed Jul  3 09:48:33 2024
Phase:          Succeeded
TEST SUITE:     temporaltest-test-wfl
Last Started:   Wed Jul  3 09:48:33 2024
Last Completed: Wed Jul  3 09:48:35 2024
Phase:          Succeeded
NOTES:
To verify that Temporal has started, run:

  kubectl --namespace=default get pods -l "app.kubernetes.io/instance=temporaltest"

lgtm