[manifests]: Add missing Pod Name (#31)

bedag · Jan 25, 2021 · ba60447 · ba60447
1 parent e31407a
commit ba60447
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 4 deletions.
diff --git a/charts/manifests/Chart.yaml b/charts/manifests/Chart.yaml
@@ -2,7 +2,7 @@ apiVersion: v2
 name: manifests
 description: "Bedag's Manifest chart. Library full of basic kubernetes manifests."
 type: library
-version: 0.4.9
+version: 0.4.10
 icon: "https://www.bedag.ch/wGlobal/wGlobal/layout/images/logo.svg"
 keywords:
 - Bedag
@@ -22,4 +22,4 @@ annotations:
   artifacthub.io/prerelease: "true"
   artifacthub.io/license: Apache-2.0
   artifacthub.io/changes: |
-    - "[Fixed]: Pod twice kind declration & nindent"
+    - "[Fixed]: Pod missing name in metadata"
diff --git a/charts/manifests/README.md b/charts/manifests/README.md
@@ -1,6 +1,6 @@
 # Manifests Library
 
-![Version: 0.4.9](https://img.shields.io/badge/Version-0.4.9-informational?style=flat-square) ![Type: library](https://img.shields.io/badge/Type-library-informational?style=flat-square)
+![Version: 0.4.10](https://img.shields.io/badge/Version-0.4.10-informational?style=flat-square) ![Type: library](https://img.shields.io/badge/Type-library-informational?style=flat-square)
 
 This library's purpose is to have more flexibility as chart author but at the same time have kubernetes manifests managed in a central library. This way you can avoid big surprises when Kubernetes has breaking changes in any of their APIs. Currently we support a base set of resources. Resources may be added as soon as we see or get a request that there's a need for it. This chart is still under development and testing, since it's rather complex. Feel free to use it. Our goal is to get it as reliable as possible.
 

diff --git a/charts/manifests/templates/manifests/_pod.tpl b/charts/manifests/templates/manifests/_pod.tpl
@@ -29,7 +29,9 @@ apiVersion: {{ $pod.apiVersion }}
     {{- else }}
 apiVersion: v1
     {{- end }}
-    {{- include "bedag-lib.template.pod" (set . "pod" $pod) | nindent 0 }}
+    {{- $podTpl := fromYaml (include "bedag-lib.template.pod" (set . "pod" $pod)) }}
+    {{- $_ := set $podTpl.metadata "name" (include "bedag-lib.utils.common.fullname" .) }}
+    {{- toYaml $podTpl | nindent 0 }}
   {{- else }}
     {{- fail "Template requires '.context' as arguments" }}
   {{- end }}