stardog-union / helm-charts

Stardog Helm Charts
Apache License 2.0
9 stars 12 forks source link

Modify CircleCI config to fix the tests #82

Closed ctffarley closed 1 year ago

ctffarley commented 1 year ago
ctffarley commented 1 year ago

Not sure what debugging I added is worth keeping, if any. The big changes were bumping the versions, fixing the license environment variable on circle (we'll have to do this monthly unless we set up a license server or other solution), and adding -E to the sudo minikube tunnel to make it open the tunnel correctly by passing in the environment variables.

ctffarley commented 1 year ago

i'd say all these logs make sense. I am not sure whether circle has a way to export log files, but i'd say that's good. question, why do you need the -E? I can see there was another environment variable being set: --set "replicaCount=${NUM_STARDOGS}" did that even work without the -E flag?

The command wouldn't work without -E. You would get this error from the tunnel since you have to run it with sudo: https://app.circleci.com/pipelines/github/stardog-union/helm-charts/250/workflows/bb6d29ff-6266-4fd3-b58e-260d6ab762bc/jobs/265?invite=true#step-109-3301