From fceda8451461ee5e623815414f76885df77b7217 Mon Sep 17 00:00:00 2001 From: Felix Scheinost <31761492+felixscheinost@users.noreply.github.com> Date: Fri, 10 Nov 2023 03:02:55 +0100 Subject: [PATCH] Fix default of `helm.releases.<>.apiVersions` (#42) The previous value included only the `.`, e.g. `monitoring.coreos.com/v1`. But Helm charts actually also check for e.g. `monitoring.coreos.com/v1/ServiceMonitor`. For this reason we need both. --- modules/helm.nix | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/modules/helm.nix b/modules/helm.nix index 10a5981..52db0de 100644 --- a/modules/helm.nix +++ b/modules/helm.nix @@ -106,7 +106,12 @@ in If you use `kubernetes.customTypes` to make kubenix aware of CRDs, it will include those as well by default. ''; type = types.listOf types.str; - default = builtins.map (customType: "${customType.group}/${customType.version}") + default = builtins.concatMap + (customType: + [ + "${customType.group}/${customType.version}" + "${customType.group}/${customType.version}/${customType.kind}" + ]) (builtins.attrValues globalConfig.kubernetes.customTypes); };