[FLINK-37669][HELM] Add Helm chart unit tests to flink kubernetes operator

helm/flink-kubernetes-operator/.helmignore

@@ -1,7 +1,34 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contributor license agreements.  See the NOTICE file distributed with
+# this work for additional information regarding copyright ownership.
+# The ASF licenses this file to You under the Apache License, Version 2.0
+# (the "License"); you may not use this file except in compliance with
+# the License.  You may obtain a copy of the License at
+#
+#    http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+#
+
 # Patterns to ignore when building packages.
 # This supports shell glob matching, relative path matching, and
 # negation (prefixed with !). Only one pattern per line.
-.DS_Store
+
+ci/
+
+# helm-unittest
+./tests
+.debug
+__snapshot__
+
+# helm-docs
+README.md.gotmpl
+
 # Common VCS dirs
 .git/
 .gitignore
@@ -10,14 +37,19 @@
 .hg/
 .hgignore
 .svn/
+
 # Common backup files
 *.swp
 *.bak
 *.tmp
 *.orig
 *~
+
 # Various IDEs
 .project
 .idea/
 *.tmproj
 .vscode/
+
+# MacOS
+.DS_Store

helm/flink-kubernetes-operator/Chart.yaml

@@ -16,11 +16,18 @@
 # limitations under the License.
 ################################################################################
 
----
 apiVersion: v2
+
 name: flink-kubernetes-operator
+
 description: A Helm chart for the Apache Flink Kubernetes Operator
+
 type: application
+
 version: 1.12-SNAPSHOT
+
 appVersion: 1.12-SNAPSHOT
+
+home: https://github.com/apache/flink-kubernetes-operator
+
 icon: https://flink.apache.org/img/logo/png/50/color_50.png

helm/flink-kubernetes-operator/templates/_helpers.tpl

@@ -1,20 +1,20 @@
-################################################################################
-#  Licensed to the Apache Software Foundation (ASF) under one
-#  or more contributor license agreements.  See the NOTICE file
-#  distributed with this work for additional information
-#  regarding copyright ownership.  The ASF licenses this file
-#  to you under the Apache License, Version 2.0 (the
-#  "License"); you may not use this file except in compliance
-#  with the License.  You may obtain a copy of the License at
-#
-#      http://www.apache.org/licenses/LICENSE-2.0
-#
-#  Unless required by applicable law or agreed to in writing, software
-#  distributed under the License is distributed on an "AS IS" BASIS,
-#  WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-#  See the License for the specific language governing permissions and
-# limitations under the License.
-################################################################################
+{{- /*
+Licensed to the Apache Software Foundation (ASF) under one
+or more contributor license agreements.  See the NOTICE file
+distributed with this work for additional information
+regarding copyright ownership.  The ASF licenses this file
+to you under the Apache License, Version 2.0 (the
+"License"); you may not use this file except in compliance
+with the License.  You may obtain a copy of the License at
+
+    http://www.apache.org/licenses/LICENSE-2.0
+
+Unless required by applicable law or agreed to in writing, software
+distributed under the License is distributed on an "AS IS" BASIS,
+WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+See the License for the specific language governing permissions and
+limitations under the License.
+*/ -}}
 
 {{/*
 Expand the name of the chart.
@@ -78,73 +78,6 @@ Create the path of the operator image to use
 {{- end }}
 {{- end }}
 
-{{/*
-Create the name of the operator role to use
-*/}}
-{{- define "flink-operator.roleName" -}}
-{{- if .Values.rbac.operatorRole.create }}
-{{- default (include "flink-operator.fullname" .) .Values.rbac.operatorRole.name }}
-{{- else }}
-{{- default "default" .Values.rbac.operatorRole.name }}
-{{- end }}
-{{- end }}
-
-{{/*
-Create the name of the operator role binding to use
-*/}}
-{{- define "flink-operator.roleBindingName" -}}
-{{- if .Values.rbac.operatorRoleBinding.create }}
-{{- default (include "flink-operator.fullname" .) .Values.rbac.operatorRoleBinding.name }}
-{{- else }}
-{{- default "default" .Values.rbac.operatorRoleBinding.name }}
-{{- end }}
-{{- end }}
-
-{{/*
-Create the name of the job role to use
-*/}}
-{{- define "flink-operator.jobRoleName" -}}
-{{- if .Values.rbac.jobRoleBinding.create }}
-{{- default (include "flink-operator.fullname" .) .Values.rbac.jobRole.name }}
-{{- else }}
-{{- default "default" .Values.rbac.jobRole.name }}
-{{- end }}
-{{- end }}
-
-{{/*
-Create the name of the job role to use
-*/}}
-{{- define "flink-operator.jobRoleBindingName" -}}
-{{- if .Values.rbac.jobRole.create }}
-{{- default (include "flink-operator.fullname" .) .Values.rbac.jobRoleBinding.name }}
-{{- else }}
-{{- default "default" .Values.rbac.jobRoleBinding.name }}
-{{- end }}
-{{- end }}
-
-
-{{/*
-Create the name of the operator service account to use
-*/}}
-{{- define "flink-operator.serviceAccountName" -}}
-{{- if .Values.operatorServiceAccount.create }}
-{{- default (include "flink-operator.fullname" .) .Values.operatorServiceAccount.name }}
-{{- else }}
-{{- default "default" .Values.operatorServiceAccount.name }}
-{{- end }}
-{{- end }}
-
-{{/*
-Create the name of the job service account to use
-*/}}
-{{- define "flink-operator.jobServiceAccountName" -}}
-{{- if .Values.jobServiceAccount.create }}
-{{- default (include "flink-operator.fullname" .) .Values.jobServiceAccount.name }}
-{{- else }}
-{{- default "default" .Values.jobServiceAccount.name }}
-{{- end }}
-{{- end }}
-
 {{/*
 Determine role scope based on name
 */}}
@@ -155,43 +88,3 @@ Determine role scope based on name
 {{- printf "Role" }}
 {{- end }}
 {{- end }}
-
-{{- define "flink-operator.validating-webhook-enabled" -}}
-{{- if hasKey .Values.webhook "validator" }}
-{{- if .Values.webhook.validator.create }}
-{{- printf "true" }}
-{{- else }}
-{{- printf "false" }}
-{{- end }}
-{{- else }}
-{{- if or (.Values.webhook.create) }}
-{{- printf "true" }}
-{{- else }}
-{{- printf "false" }}
-{{- end }}
-{{- end }}
-{{- end }}
-
-{{- define "flink-operator.mutating-webhook-enabled" -}}
-{{- if hasKey .Values.webhook "mutator" }}
-{{- if .Values.webhook.mutator.create }}
-{{- printf "true" }}
-{{- else }}
-{{- printf "false" }}
-{{- end }}
-{{- else }}
-{{- if or (.Values.webhook.create) }}
-{{- printf "true" }}
-{{- else }}
-{{- printf "false" }}
-{{- end }}
-{{- end }}
-{{- end }}
-
-{{- define "flink-operator.webhook-enabled" -}}
-{{- if or (eq (include "flink-operator.validating-webhook-enabled" .) "true") (eq (include "flink-operator.mutating-webhook-enabled" .) "true") }}
-{{- printf "true" }}
-{{- else }}
-{{- printf "false" }}
-{{- end }}
-{{- end }}

helm/flink-kubernetes-operator/templates/certmanager/_helpers.tpl

@@ -0,0 +1,17 @@
+{{- /*
+Licensed to the Apache Software Foundation (ASF) under one
+or more contributor license agreements.  See the NOTICE file
+distributed with this work for additional information
+regarding copyright ownership.  The ASF licenses this file
+to you under the Apache License, Version 2.0 (the
+"License"); you may not use this file except in compliance
+with the License.  You may obtain a copy of the License at
+
+    http://www.apache.org/licenses/LICENSE-2.0
+
+Unless required by applicable law or agreed to in writing, software
+distributed under the License is distributed on an "AS IS" BASIS,
+WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+See the License for the specific language governing permissions and
+limitations under the License.
+*/ -}}

helm/flink-kubernetes-operator/templates/certmanager/certificate.yaml

@@ -0,0 +1,45 @@
+{{- /*
+Licensed to the Apache Software Foundation (ASF) under one
+or more contributor license agreements.  See the NOTICE file
+distributed with this work for additional information
+regarding copyright ownership.  The ASF licenses this file
+to you under the Apache License, Version 2.0 (the
+"License"); you may not use this file except in compliance
+with the License.  You may obtain a copy of the License at
+
+    http://www.apache.org/licenses/LICENSE-2.0
+
+Unless required by applicable law or agreed to in writing, software
+distributed under the License is distributed on an "AS IS" BASIS,
+WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+See the License for the specific language governing permissions and
+limitations under the License.
+*/ -}}
+
+apiVersion: cert-manager.io/v1
+kind: Certificate
+metadata:
+  name: flink-operator-serving-cert
+  namespace: {{ .Release.Namespace }}
+spec:
+  dnsNames:
+  - flink-operator-webhook-service.{{ .Release.Namespace }}.svc
+  - flink-operator-webhook-service.{{ .Release.Namespace }}.svc.cluster.local
+  keystores:
+    pkcs12:
+      create: true
+      passwordSecretRef:
+      {{- if .Values.webhook.keystore.useDefaultPassword }}
+        name: flink-operator-webhook-secret
+        key: password
+      {{- else }}
+        {{- with .Values.webhook.keystore.passwordSecretRef }}
+          {{- toYaml . | nindent 8 }}
+        {{- end }}
+      {{- end }}
+  issuerRef:
+    group: cert-manager.io
+    kind: Issuer
+    name: flink-operator-selfsigned-issuer
+  commonName: FlinkDeployment Validator
+  secretName: webhook-server-cert

helm/flink-kubernetes-operator/templates/certmanager/issuer.yaml

@@ -0,0 +1,25 @@
+{{- /*
+Licensed to the Apache Software Foundation (ASF) under one
+or more contributor license agreements.  See the NOTICE file
+distributed with this work for additional information
+regarding copyright ownership.  The ASF licenses this file
+to you under the Apache License, Version 2.0 (the
+"License"); you may not use this file except in compliance
+with the License.  You may obtain a copy of the License at
+
+    http://www.apache.org/licenses/LICENSE-2.0
+
+Unless required by applicable law or agreed to in writing, software
+distributed under the License is distributed on an "AS IS" BASIS,
+WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+See the License for the specific language governing permissions and
+limitations under the License.
+*/ -}}
+
+apiVersion: cert-manager.io/v1
+kind: Issuer
+metadata:
+  name: flink-operator-selfsigned-issuer
+  namespace: {{ .Release.Namespace }}
+spec:
+  selfSigned: {}

helm/flink-kubernetes-operator/templates/controller/_helpers.tpl

@@ -0,0 +1,17 @@
+{{- /*
+Licensed to the Apache Software Foundation (ASF) under one
+or more contributor license agreements.  See the NOTICE file
+distributed with this work for additional information
+regarding copyright ownership.  The ASF licenses this file
+to you under the Apache License, Version 2.0 (the
+"License"); you may not use this file except in compliance
+with the License.  You may obtain a copy of the License at
+
+    http://www.apache.org/licenses/LICENSE-2.0
+
+Unless required by applicable law or agreed to in writing, software
+distributed under the License is distributed on an "AS IS" BASIS,
+WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+See the License for the specific language governing permissions and
+limitations under the License.
+*/ -}}

helm/flink-kubernetes-operator/templates/controller/configmap.yaml

@@ -0,0 +1,56 @@
+{{- /*
+Licensed to the Apache Software Foundation (ASF) under one
+or more contributor license agreements.  See the NOTICE file
+distributed with this work for additional information
+regarding copyright ownership.  The ASF licenses this file
+to you under the Apache License, Version 2.0 (the
+"License"); you may not use this file except in compliance
+with the License.  You may obtain a copy of the License at
+
+    http://www.apache.org/licenses/LICENSE-2.0
+
+Unless required by applicable law or agreed to in writing, software
+distributed under the License is distributed on an "AS IS" BASIS,
+WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+See the License for the specific language governing permissions and
+limitations under the License.
+*/ -}}
+
+{{- if .Values.defaultConfiguration.create }}
+apiVersion: v1
+kind: ConfigMap
+metadata:
+  name: flink-operator-config
+  namespace: {{ .Release.Namespace }}
+  labels:
+    {{- include "flink-operator.labels" . | nindent 4 }}
+data:
+  flink-conf.yaml: |+
+{{- if .Values.defaultConfiguration.append }}
+  {{- $.Files.Get "conf/flink-conf.yaml"  | nindent 4 -}}
+{{- end }}
+{{- if index (.Values.defaultConfiguration) "flink-conf.yaml" }}
+  {{- index (.Values.defaultConfiguration) "flink-conf.yaml" | nindent 4 -}}
+{{- end }}
+{{- if .Values.watchNamespaces }}
+    kubernetes.operator.watched.namespaces: {{ join "," .Values.watchNamespaces  }}
+{{- end }}
+{{- if index .Values "operatorHealth" }}
+    kubernetes.operator.health.probe.enabled: true
+    kubernetes.operator.health.probe.port: {{ .Values.operatorHealth.port }}
+{{- end }}
+  log4j-operator.properties: |+
+{{- if .Values.defaultConfiguration.append }}
+  {{- $.Files.Get "conf/log4j-operator.properties"  | nindent 4 -}}
+{{- end }}
+{{- if index (.Values.defaultConfiguration) "log4j-operator.properties" }}
+  {{- index (.Values.defaultConfiguration) "log4j-operator.properties" | nindent 4 -}}
+{{- end }}
+  log4j-console.properties: |+
+{{- if .Values.defaultConfiguration.append }}
+  {{- $.Files.Get "conf/log4j-console.properties"  | nindent 4 -}}
+{{- end }}
+{{- if index (.Values.defaultConfiguration) "log4j-console.properties" }}
+  {{- index (.Values.defaultConfiguration) "log4j-console.properties" | nindent 4 -}}
+{{- end }}
+{{- end }}