fix(otel): Make cloud.account.id assertion unconditional

vaishnavi-30-beep · vaishnavi-30-beep · commit bbb354aec00d · 2026-07-27T13:35:31.000+01:00
cloud.account.id is always present on Karpenter metrics. Replace
conditional check with require.True assertion to fail explicitly
if the attribute is ever missing.
diff --git a/test/otel/solutions/karpenter/karpenter_test.go b/test/otel/solutions/karpenter/karpenter_test.go
@@ -306,9 +306,8 @@ func TestKarpenterCloudAccountID(t *testing.T) {
 			require.NotEmpty(t, results, "%s not available", metricName)
 			for _, r := range results {
 				acctID, ok := r.Labels.Resource["cloud.account.id"]
-				if ok {
-					require.Equal(t, cfg.AccountID, acctID, "%s cloud.account.id", metricName)
-				}
+				require.True(t, ok, "%s missing @resource.cloud.account.id", metricName)
+				require.Equal(t, cfg.AccountID, acctID, "%s cloud.account.id", metricName)
 			}
 		})
 	}

Original file line number	Diff line number	Diff line change
`@@ -306,9 +306,8 @@ func TestKarpenterCloudAccountID(t *testing.T) {`
`306`	`306`	`require.NotEmpty(t, results, "%s not available", metricName)`
`307`	`307`	`for _, r := range results {`
`308`	`308`	`acctID, ok := r.Labels.Resource["cloud.account.id"]`
`309`		`- if ok {`
`310`		`- require.Equal(t, cfg.AccountID, acctID, "%s cloud.account.id", metricName)`
`311`		`- }`
	`309`	`+ require.True(t, ok, "%s missing @resource.cloud.account.id", metricName)`
	`310`	`+ require.Equal(t, cfg.AccountID, acctID, "%s cloud.account.id", metricName)`
`312`	`311`	`}`
`313`	`312`	`})`
`314`	`313`	`}`