Remove use of -init-type=sync as it has no effect

As part of hashicorp/consul-k8s#232 we removed the docs about -init-type=sync. If -init-type was set to sync it actually had no effect. Only -init-type=client had effect so this change stops setting that flag in the case of sync.
hashicorp · Mar 23, 2020 · fda58cb · fda58cb
1 parent 1790a25
commit fda58cb
Show file tree

Hide file tree

Showing 6 changed files with 4 additions and 10 deletions.
diff --git a/templates/client-snapshot-agent-deployment.yaml b/templates/client-snapshot-agent-deployment.yaml
@@ -126,8 +126,7 @@ spec:
           - |
             consul-k8s acl-init \
               -secret-name="{{ template "consul.fullname" . }}-client-snapshot-agent-acl-token" \
-              -k8s-namespace={{ .Release.Namespace }} \
-              -init-type="sync"
+              -k8s-namespace={{ .Release.Namespace }}
         volumeMounts:
           - name: aclconfig
             mountPath: /consul/aclconfig

diff --git a/templates/connect-inject-deployment.yaml b/templates/connect-inject-deployment.yaml
@@ -190,8 +190,7 @@ spec:
           - |
             consul-k8s acl-init \
               -secret-name="{{ template "consul.fullname" . }}-connect-inject-acl-token" \
-              -k8s-namespace={{ .Release.Namespace }} \
-              -init-type="sync"
+              -k8s-namespace={{ .Release.Namespace }}
       {{- end }}
       {{- if .Values.connectInject.nodeSelector }}
       nodeSelector:

diff --git a/templates/enterprise-license-job.yaml b/templates/enterprise-license-job.yaml
@@ -109,8 +109,7 @@ spec:
           - |
             consul-k8s acl-init \
               -secret-name="{{ template "consul.fullname" . }}-enterprise-license-acl-token" \
-              -k8s-namespace={{ .Release.Namespace }} \
-              -init-type="sync"
+              -k8s-namespace={{ .Release.Namespace }}
       {{- end }}
 {{- end }}
 {{- end }}
diff --git a/templates/mesh-gateway-deployment.yaml b/templates/mesh-gateway-deployment.yaml
@@ -112,7 +112,6 @@ spec:
                 consul-k8s acl-init \
                   -secret-name="{{ template "consul.fullname" . }}-mesh-gateway-acl-token" \
                   -k8s-namespace={{ .Release.Namespace }} \
-                  -init-type="sync" \
                   -token-sink-file=/consul/service/acl-token
                 {{ end }}
 

diff --git a/templates/sync-catalog-deployment.yaml b/templates/sync-catalog-deployment.yaml
@@ -175,8 +175,7 @@ spec:
           - |
             consul-k8s acl-init \
               -secret-name="{{ template "consul.fullname" . }}-catalog-sync-acl-token" \
-              -k8s-namespace={{ .Release.Namespace }} \
-              -init-type="sync"
+              -k8s-namespace={{ .Release.Namespace }}
       {{- end }}
       {{- if .Values.syncCatalog.nodeSelector }}
       nodeSelector:

diff --git a/test/unit/mesh-gateway-deployment.bats b/test/unit/mesh-gateway-deployment.bats
@@ -632,7 +632,6 @@ consul services register \
   exp='consul-k8s acl-init \
   -secret-name="release-name-consul-mesh-gateway-acl-token" \
   -k8s-namespace=default \
-  -init-type="sync" \
   -token-sink-file=/consul/service/acl-token
 
 consul-k8s service-address \