Skip to content

Commit d0c379f

Browse files
Gate per-node ContainerLogV2 coverage check behind PER_NODE_LOG_COVERAGE env var
1 parent 5f8cb4a commit d0c379f

2 files changed

Lines changed: 5 additions & 0 deletions

File tree

test/ginkgo-e2e/querylogs/querylogs_suite_test.go

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,7 @@ var LogsClient *azquery.LogsClient
1818
var AKSResourceId string
1919
var RetinaNetworkFlowLogsEnabled string
2020
var GenevaIntegrationEnabled string
21+
var PerNodeLogCoverageEnabled string
2122
var Cfg *rest.Config
2223

2324
func TestQuerylogs(t *testing.T) {
@@ -34,6 +35,7 @@ var _ = BeforeSuite(func() {
3435
RetinaNetworkFlowLogsEnabled, err = utils.IsRetinaNetworkFlowLogsEnabled(K8sClient, "kube-system", "component", "ama-logs-agent", "ama-logs")
3536
Expect(err).NotTo(HaveOccurred())
3637
GenevaIntegrationEnabled = os.Getenv("GENEVA_INTEGRATION")
38+
PerNodeLogCoverageEnabled = os.Getenv("PER_NODE_LOG_COVERAGE")
3739
LogsClient, err = utils.SetupLogsClient()
3840
Expect(err).NotTo(HaveOccurred())
3941
})

test/ginkgo-e2e/querylogs/querylogs_test.go

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -43,6 +43,9 @@ var _ = Describe("When querying the logs for the table", func() {
4343

4444
var _ = Describe("When querying ContainerLogV2 per node", func() {
4545
It("Every node hosting an ama-logs DaemonSet pod should have ContainerLogV2 rows", func() {
46+
if PerNodeLogCoverageEnabled != "true" {
47+
Skip("Per-node ContainerLogV2 coverage skipped because PER_NODE_LOG_COVERAGE is not set to 'true'")
48+
}
4649
if GenevaIntegrationEnabled == "true" {
4750
Skip("ContainerLogV2 per-node coverage skipped because GENEVA_INTEGRATION is set to 'true'")
4851
}

0 commit comments

Comments
 (0)