CI mgmt-drift-correction test should be failed for drift detection errors
Summary
Observation with the following pipeline : https://gitlab.com/sylva-projects/sylva-core/-/pipelines/2005275766
The mgmt-drit-correction test is getting passed despite of errors related to drift in helm-controller.
{"level":"debug","ts":"2025-08-26T21:47:35.711Z","msg":"resource modified","controller":"helmrelease","controllerGroup":"helm.toolkit.fluxcd.io","controllerKind":"HelmRelease","HelmRelease":{"name":"loki","namespace":"sylva-system"},"namespace":"sylva-system","name":"loki","reconcileID":"9ffa4dac-e207-4875-bf03-4c1732319e9b","resource":"PrometheusRule/loki/loki-loki-alerts","patch":[{"value":"true","op":"add","path":"/metadata/annotations/prometheus-operator-validated"}]}
{"level":"debug","ts":"2025-08-26T21:47:35.711Z","msg":"resource modified","controller":"helmrelease","controllerGroup":"helm.toolkit.fluxcd.io","controllerKind":"HelmRelease","HelmRelease":{"name":"loki","namespace":"sylva-system"},"namespace":"sylva-system","name":"loki","reconcileID":"9ffa4dac-e207-4875-bf03-4c1732319e9b","resource":"PrometheusRule/loki/loki-loki-rules","patch":[{"value":"true","op":"add","path":"/metadata/annotations/prometheus-operator-validated"}]}
{"level":"debug","ts":"2025-08-26T21:47:35.711Z","logger":"events","msg":"Cluster state of release loki/loki.v1 has drifted from the desired state:\nPrometheusRule/loki/loki-loki-alerts changed (1 additions, 0 changes, 0 removals)\nPrometheusRule/loki/loki-loki-rules changed (1 additions, 0 changes, 0 removals)","type":"Warning","object":{"kind":"HelmRelease","namespace":"sylva-system","name":"loki","uid":"2cf56def-30e8-49e9-87bd-b262f73e0adb","apiVersion":"helm.toolkit.fluxcd.io/v2","resourceVersion":"53504"},"reason":"DriftDetected"}
{"level":"debug","ts":"2025-08-26T21:47:35.794Z","logger":"events","msg":"Cluster state of release loki/loki.v1 has been corrected:\nPrometheusRule/loki/loki-loki-alerts configured\nPrometheusRule/loki/loki-loki-rules configured","type":"Normal","object":{"kind":"HelmRelease","namespace":"sylva-system","name":"loki","uid":"2cf56def-30e8-49e9-87bd-b262f73e0adb","apiVersion":"helm.toolkit.fluxcd.io/v2","resourceVersion":"53504"},"reason":"DriftCorrected"}
Summary:
PrometheusRule/loki/loki-loki-alerts changed (1 additions, 0 changes, 0 removals)
PrometheusRule/loki/loki-loki-rules changed (1 additions, 0 changes, 0 removals)
These drift logs are related to loki PrometheusRule in loki namespace. The CI test mgmt-drit-correction seems not able to detect errors as seen in the Job
Job Reference for Logs: https://gitlab.com/sylva-projects/sylva-core/-/jobs/11147416206
related references
Details
Edited by Nitin Sharma