fix: Use $ to reference the values.

> > Co-authored-by: Danny__Wei <[email protected]> Co-authored-by: lichanghao.orange <[email protected]>
bytedance · Nov 12, 2024 · 1f358f6 · 1f358f6
1 parent 8cf89dd
commit 1f358f6
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 3 deletions.
diff --git a/internal/status/api/v1/status.go b/internal/status/api/v1/status.go
@@ -87,6 +87,7 @@ func (m *StatusManager) HandleProfileStatusUpdate(status varmortypes.ProfileStat
 
 	m.profileChangeCount.Add(ctx, 1, metric.WithAttributeSet(attrSet))
 }
+
 func (m *StatusManager) syncStatusMetricsLoop() {
 	ctx := context.Background()
 	for {
@@ -114,7 +115,6 @@ func (m *StatusManager) syncStatusMetricsLoop() {
 					m.profileStatusPerNode.Record(ctx, 0, metric.WithAttributeSet(attrSet)) // 0 mean failure
 				}
 			}
-
 		}
 	}
 }

diff --git a/manifests/varmor/templates/deployments/manager.yaml b/manifests/varmor/templates/deployments/manager.yaml
@@ -60,7 +60,7 @@ spec:
         {{- if .Values.metrics.enabled }}
         {{- with .Values.manager.metrics.args }}
           {{- toYaml . | nindent 8 }}
-        - --syncMetricsSecond={{ .Values.metrics.syncMetricsSecond }}
+        - --syncMetricsSecond={{ $.Values.metrics.syncMetricsSecond }}
         {{- end }}
         {{- end }}
         {{- end }}

diff --git a/manifests/varmor/values.yaml b/manifests/varmor/values.yaml
@@ -98,7 +98,6 @@ manager:
     args:
     - --enableMetrics
 
-
   resources:
     limits:
       cpu: 200m
-Original file line number
+Diff line change
@@ Expand Up @@
     	m.profileChangeCount.Add(ctx, 1, metric.WithAttributeSet(attrSet))
     }
     func (m *StatusManager) syncStatusMetricsLoop() {
     	ctx := context.Background()
     	for {
@@ Expand Down Expand Up / @@ -114,7 +115,6 @@ func (m *StatusManager) syncStatusMetricsLoop() { @@
     					m.profileStatusPerNode.Record(ctx, 0, metric.WithAttributeSet(attrSet)) // 0 mean failure
     				}
     			}
     		}
     	}
     }
@@ Expand Down @@