-
Notifications
You must be signed in to change notification settings - Fork 12
Conversation
main.tf
Outdated
yaml_body = each.value | ||
name = each.key | ||
namespace = "argocd" | ||
repository = "https://bedag.github.io/helm-charts/" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't know who owns this chart.
Can we copy the helm chart to a local directory and use the helm chart from directory instead of relying on this 3rd chart and also network access to the domain bedag.github.io
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good point the chart is pretty basic I’ll add a local copy
I've added the charts locally, works fine |
|
||
*/}} | ||
{{- if or $.Values.resources $.Values.templates -}} | ||
{{- $base := fromYaml (include "raw.labels" .) -}} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lets try to eliminate the need to include the library chart, is just for common labels.
Lets implement the commonLabels in this chart and remove the library, or delete this line and remove support for "commonLabels" for now
Fix #4
With this Terraform will only create a helm release, and helm will install manifests inside Kubernetes.
On next terraform apply, as the helm release already exists, there will be no updates.
To force Terraform recreate the helm release, it will need to be manually deleted in Kubernetes and reapply from terraform.
This is tested and seems very more reliable than the previous solution