Closed simonpasquier closed 1 month ago
Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all
Tested with a live cluster after triggering deployment failures.
Before this PR
UpdatingTelemeterClient: reconciling Telemeter client Deployment failed: updating Deployment object failed: waiting for DeploymentRollout of openshift-monitoring/telemeter-client: context deadline exceeded,
UpdatingPrometheus: client rate limiter Wait returned an error: context deadline exceeded
After this PR
UpdatingTelemeterClient: reconciling Telemeter client Deployment failed: updating Deployment object failed: waiting for DeploymentRollout of openshift-monitoring/telemeter-client: context deadline exceeded: got 1 unavailable replicas,
UpdatingPrometheus: StatefulSetNotFound: shard 0: statefulset openshift-monitoring/prometheus-k8s not found
/skip
@simonpasquier: This pull request references Jira Issue OCPBUGS-37595, which is valid. The bug has been moved to the POST state.
Requesting review from QA contact: /cc @juzhao
The bug has been updated to refer to the pull request using the external bug tracker.
@simonpasquier: This pull request references Jira Issue OCPBUGS-37595, which is valid.
Requesting review from QA contact: /cc @juzhao
/label qe-approved
@simonpasquier: This pull request references Jira Issue OCPBUGS-17506, which is invalid:
Comment /jira refresh
to re-evaluate validity if changes to the Jira bug are made, or edit the title of this pull request to link to a different bug.
The bug has been updated to refer to the pull request using the external bug tracker.
/jira refresh
@simonpasquier: This pull request references Jira Issue OCPBUGS-17506, which is valid. The bug has been moved to the POST state.
Requesting review from QA contact: /cc @juzhao
/lgtm
/skip
/retest-required
/test e2e-aws-ovn-techpreview
/retest-required
/hold cancel @machine424 I've included the unit tests that you've mentioned earlier :)
/retest-required
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: jan--f, machine424, simonpasquier
The full list of commands accepted by this bot can be found here.
The pull request process is described here
/hold cancel
@simonpasquier: all tests passed!
Full PR test history. Your PR dashboard.
@simonpasquier: Jira Issue OCPBUGS-17506: All pull requests linked via external trackers have merged:
Jira Issue OCPBUGS-17506 has been moved to the MODIFIED state.
[ART PR BUILD NOTIFIER]
Distgit: cluster-monitoring-operator This PR has been included in build cluster-monitoring-operator-container-v4.17.0-202408071315.p0.g7e14b2c.assembly.stream.el9. All builds following this will include this PR.