From 1c802e720fd56485c3b6dbe20ef2e5bdef87a1fd Mon Sep 17 00:00:00 2001
From: JieKe <jkfang@cn.ibm.com>
Date: Wed, 15 May 2019 10:57:28 +0800
Subject: [PATCH] I want to include this chart as a subchart of my helm, naming
 the ldap service name as just release name is a little confusing, so want to
 add more meaningful suffix, thanks!

---
 templates/openldap.yaml     | 14 +++++++-------
 templates/phpldapadmin.yaml | 10 +++++-----
 2 files changed, 12 insertions(+), 12 deletions(-)

diff --git a/templates/openldap.yaml b/templates/openldap.yaml
index 578224a..089a091 100644
--- a/templates/openldap.yaml
+++ b/templates/openldap.yaml
@@ -10,26 +10,26 @@ apiVersion: v1
 kind: Service
 metadata:
   labels:
-    app: {{.Release.Name}}
-  name: {{.Release.Name}}
+    app: {{.Release.Name}}-openldap
+  name: {{.Release.Name}}-openldap
 spec:
   ports:
     - port: 389
   selector:
-    app: {{.Release.Name}}
+    app: {{.Release.Name}}-openldap
 ---
 apiVersion: extensions/v1beta1
 kind: Deployment
 metadata:
-  name: {{.Release.Name}}
+  name: {{.Release.Name}}-openldap
   labels:
-    app: {{.Release.Name}}
+    app: {{.Release.Name}}-openldap
 spec:
   replicas: 1
   template:
     metadata:
       labels:
-        app: {{.Release.Name}}
+        app: {{.Release.Name}}-openldap
     spec:
       initContainers:
         - name: init-copy
@@ -42,7 +42,7 @@ spec:
           - name: config-storage
             mountPath: /config-storage
       containers:
-        - name: {{.Release.Name}}
+        - name: {{.Release.Name}}-openldap
           image: {{.Values.OpenLdap.Image}}:{{.Values.OpenLdap.ImageTag}}
           imagePullPolicy: {{.Values.OpenLdap.ImagePullPolicy}}
           args: ["--loglevel", "debug"]
diff --git a/templates/phpldapadmin.yaml b/templates/phpldapadmin.yaml
index 68dfb26..1e8b4bd 100644
--- a/templates/phpldapadmin.yaml
+++ b/templates/phpldapadmin.yaml
@@ -1,7 +1,7 @@
 apiVersion: v1
 kind: Service
 metadata:
-  name: {{.Release.Name}}-admin
+  name: {{.Release.Name}}-ldap-admin
 spec:
 {{- if .Values.PhpLdapAdmin.NodePort }}
   type: NodePort
@@ -13,21 +13,21 @@ spec:
     nodePort: 31080
 {{- end }}
   selector:
-    app: {{.Release.Name}}-admin
+    app: {{.Release.Name}}-ldap-admin
 ---
 apiVersion: extensions/v1beta1
 kind: Deployment
 metadata:
-  name: {{.Release.Name}}-admin
+  name: {{.Release.Name}}-ldap-admin
 spec:
   replicas: {{.Values.OpenLdap.Replicas}}
   template:
     metadata:
       labels:
-        app: {{.Release.Name}}-admin
+        app: {{.Release.Name}}-ldap-admin
     spec:
       containers:
-      - name: {{.Release.Name}}-admin
+      - name: {{.Release.Name}}-ldap-admin
         image: {{.Values.PhpLdapAdmin.Image}}:{{.Values.PhpLdapAdmin.ImageTag}}
         imagePullPolicy: {{.Values.PhpLdapAdmin.ImagePullPolicy}}
         ports: