Skip to content

Commit c8406b6

Browse files
committed
Ignore risks from e2e testing
This is to follow up a recent finding [1]. Further digging shows that the related risks to our e2e tests are `TestAlertFeatureE2ETestOTA1813`, `SyntheticRiskA`. This pull ignores the risks with prefix `TestAlertFeatureE2ETest` similar to [2]. I will fix `SyntheticRiskA` by modifying "fauxinnati" so that the risks coming from it have the prefix too. This will recover the health of the TP-enabled jobs in CI. [1]. #1367 (comment) [2]. openshift/origin#30929
1 parent e9c1c39 commit c8406b6

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

install/0000_90_cluster-version-operator_02_prometheusrule_servicemonitor.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ spec:
2020
description: The conditional update risk {{ "{{ $labels.risk }}" }} might apply to the cluster because of {{ "{{ $labels.reason }}" }}, and the cluster update to a version exposed to the risk is not recommended. For more information refer to 'oc adm upgrade'.
2121
runbook_url: https://github.com/openshift/runbooks/blob/master/alerts/cluster-version-operator/OpenShiftUpdateRiskMightApply.md
2222
expr: |
23-
max by (namespace, risk, reason) (last_over_time(cluster_version_risk_conditions{job="cluster-version-operator", condition="Applies"}[5m]) != 0)
23+
max by (namespace, risk, reason) (last_over_time(cluster_version_risk_conditions{job="cluster-version-operator", condition="Applies", risk!~"TestAlertFeatureE2ETest.*"}[5m]) != 0)
2424
for: 10m
2525
labels:
2626
severity: warning

0 commit comments

Comments
 (0)