This message was deleted.
# contributorsb
broad-zoo-34077
07/23/2021, 12:22 PMThis message was deleted.
l
little-author-61621
07/23/2021, 1:24 PM
Looks cool, I haven't used either before but will have a look in more detail. Have you used the consul-helm method before?
little-author-61621
07/23/2021, 1:25 PM
There's also https://helm.sh/docs/topics/chart_tests/ we should look at, but this is more smoke tests
👀 1
d
damp-ambulance-61372
07/23/2021, 1:25 PMYeah I've used both before. Actually started with golang unit tests but switched to bat method like consul-helm (since golang unit tests was getting unnecessarily complex)
damp-ambulance-61372
07/23/2021, 1:29 PMThis helm chart tests (https://helm.sh/docs/topics/chart_tests/) is good for integration or e2e testing as they require a running k8s cluster. I think we can also have them as the next step (e.g. to run only on release tags, as they might be expensive)
l
little-author-61621
07/23/2021, 1:40 PM
Yeah that makes sense
little-author-61621
07/23/2021, 1:41 PM
The bash ones look pretty good actually. Have you tried these ones before? https://github.com/anikin-aa/helm-unittest
d
damp-ambulance-61372
07/23/2021, 3:06 PMNice. No I haven't. The last commit seems to be for 2019 and the main repo (which this is forked from) is no longer maintained. IDK if it supports helm3 or not. Maybe that won't be an issue for us and will work for us as is, or maybe we can fork it and make it work with a few tweaks. Needs a closer investigation.
l
little-author-61621
07/23/2021, 3:12 PM
This seems to be the most maintained fork: https://github.com/quintush/helm-unittest
Apache Airflow switched to Python for the tests as well, more info here: https://github.com/apache/airflow/issues/11657
👍 1
9 Views