Address review comments

Signed-off-by: Antonin Bas <abas@vmware.com>
antrea-io · Apr 8, 2022 · 90f738c · 90f738c
1 parent 165bb1b
commit 90f738c
Show file tree

Hide file tree

Showing 6 changed files with 15 additions and 36 deletions.
diff --git a/.github/workflows/go.yml b/.github/workflows/go.yml
@@ -224,6 +224,15 @@ jobs:
       run: |
         sudo npm install -g markdownlint-cli@0.31.1
         make markdownlint
+    - name: Checking whether autogenerated Helm chart documentation is up-to-date
+      working-directory: build/charts/
+      run: |
+        make helm-docs
+        DIFF=$(git diff .)
+        if [ -n "$DIFF" ]; then
+          echo "The Helm chart documentation is out-of-date; please run 'make helm-docs' in 'build/charts/' and commit the changes"
+          exit 1
+        fi
 
   benchmark:
     needs: check-changes

diff --git a/.github/workflows/helm_docs.yml b/.github/workflows/helm_docs.yml
diff --git a/build/charts/antrea/conf/antrea-controller.conf b/build/charts/antrea/conf/antrea-controller.conf
@@ -37,7 +37,7 @@ enablePrometheusMetrics: {{ .Values.controller.enablePrometheusMetrics }}
 #   ca.crt: <CA certificate>
 #   tls.crt: <TLS certificate>
 #   tls.key: <TLS private key>
-selfSignedCert: {{ .Values.controller.selfSigned }}
+selfSignedCert: {{ .Values.controller.selfSignedCert }}
 
 # Comma-separated list of Cipher Suites. If omitted, the default Go Cipher Suites will be used.
 # https://golang.org/pkg/crypto/tls/#pkg-constants

diff --git a/build/charts/antrea/templates/agent/daemonset.yaml b/build/charts/antrea/templates/agent/daemonset.yaml
@@ -244,9 +244,6 @@ spec:
           args:
             - "--log_file_max_size={{ .Values.agent.antreaOVS.logFileMaxSize }}"
             - "--log_file_max_num={{ .Values.agent.antreaOVS.logFileMaxNum }}"
-            {{- if .Values.logVerbosity }}
-            - "--v={{ .Values.logVerbosity }}"
-            {{- end }}
             {{- if .Values.hwOffload }}
             - "--hw-offload"
             {{- end }}

diff --git a/build/charts/antrea/values.yaml b/build/charts/antrea/values.yaml
@@ -194,7 +194,7 @@ controller:
   # -- Indicates whether to use auto-generated self-signed TLS certificates. If
   # false, a Secret named "antrea-controller-tls" must be provided with the
   # following keys: ca.crt, tls.crt, tls.key.
-  selfSigned: true
+  selfSignedCert: true
   # -- Tolerations for the antrea-controller Pod.
   tolerations:
     # Mark it as a critical add-on.

diff --git a/hack/generate-standard-manifests.sh b/hack/generate-standard-manifests.sh
@@ -100,6 +100,10 @@ fi
 
 THIS_DIR="$( cd "$( dirname "${BASH_SOURCE[0]}" )" >/dev/null 2>&1 && pwd )"
 
+# Avoid potential Helm warnings about invalid permissions for Kubeconfig file.
+# The Kubeconfig does not matter for "helm template".
+unset KUBECONFIG
+
 source $THIS_DIR/verify-helm.sh
 
 if [ -z "$HELM" ]; then