-
Notifications
You must be signed in to change notification settings - Fork 14
Open
Description
CRDs are duplicated between helm-charts/crds/ and terraform/manifests/. Every sync from pulp-operator requires updating both, and they can drift.
The terraform copies don't use template variables - they're static YAML. We could have terraform read directly from helm-charts/crds/ instead:
locals {
manifests = fileset(path.module, "manifests/*.yaml")
crds = fileset("${path.module}/../helm-charts/crds", "*.yaml")
}
resource "kubernetes_manifest" "crds" {
for_each = local.crds
manifest = yamldecode(file("${path.module}/../helm-charts/crds/${each.value}"))
}
resource "kubernetes_manifest" "roles" {
for_each = local.manifests
manifest = yamldecode(templatefile("${each.value}", { ... }))
depends_on = [kubernetes_manifest.crds]
}Then delete the CRDs from terraform/manifests/.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels