azimuth-cloud / azimuth-caas-operator

K8s operator to create ansible based clusters using K8s CRDs
Apache License 2.0
1 stars 2 forks source link

Add deploy keys for Slurm appliance #54

Closed mkjpryor closed 1 year ago

JohnGarbutt commented 1 year ago

I am worried by the functional tests breaking here, do you know why?

The logs suggest it hits this error:

RuntimeError: CONSUL_HTTP_ADDR is not set

ah, that is because you removed the defaults:

https://github.com/stackhpc/azimuth-caas-operator/pull/54/files#diff-76b8f256d67e8c0589c7f0e9b1fd75a4e18f346fa8986a18616b4e1be6dc8e0fR120

mkjpryor commented 1 year ago

Yeah - I didn’t like those defaults as they almost never apply 🤣 Better to be explicit I think.