Closed simonpasquier closed 8 months ago
/lgtm Thought about doing the same for the client's one https://github.com/openshift/cluster-monitoring-operator/blob/dbcfed9c91a37adbc1e2eb5af5e09172d701cdff/pkg/client/client.go#L2037 but I was worried about getting more failure logs + making unnecessary requests. (especially for delete ones that are known to take time.) unlike tests, It' not a problem if the sync takes an extra 5 seconds (especially that the tasks run in //).
@simonpasquier: This pull request references MON-3706 which is a valid jira issue.
Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the task to target the "4.16.0" version, but no target version was set.
/skip
@machine424 100% agreed: for CMO tasks, waiting an extra 5s is often not a big deal.
/retest-required
Remaining retests: 0 against base HEAD dbcfed9c91a37adbc1e2eb5af5e09172d701cdff and 2 for PR HEAD de9cb0dea2bbdf4aebe13b4401dd0ba07f5ee52a in total
/retest-required
/retest-required
Remaining retests: 0 against base HEAD b7e3f50875f2bb1fed912b23fb80a101d3a786c0 and 1 for PR HEAD de9cb0dea2bbdf4aebe13b4401dd0ba07f5ee52a in total
/retest images
@simonpasquier: The /retest
command does not accept any targets.
The following commands are available to trigger required jobs:
/test e2e-agnostic-operator
/test e2e-aws-ovn
/test e2e-aws-ovn-techpreview
/test e2e-aws-ovn-upgrade
/test generate
/test go-fmt
/test golangci-lint
/test images
/test jsonnet-fmt
/test rules
/test shellcheck
/test unit
/test vendor
/test verify
The following commands are available to trigger optional jobs:
/test e2e-aws-ovn-single-node
/test versions
Use /test all
to run all jobs.
/test images
The latest run took 5561s
(https://gcsweb-ci.apps.ci.l2s4.p1.openshiftapps.com/gcs/test-platform-results/pr-logs/pull/openshift_cluster-monitoring-operator/2188/pull-ci-openshift-cluster-monitoring-operator-master-e2e-agnostic-operator/1757316599150284800/artifacts/e2e-agnostic-operator/test/build-log.txt). I've looked at another run (https://gcsweb-ci.apps.ci.l2s4.p1.openshiftapps.com/gcs/test-platform-results/pr-logs/pull/openshift_cluster-monitoring-operator/2188/pull-ci-openshift-cluster-monitoring-operator-master-e2e-agnostic-operator/1757316599150284800/artifacts/e2e-agnostic-operator/test/build-log.txt) and it took 6855s
.
/test e2e-agnostic-operator
@simonpasquier: The following test failed, say /retest
to rerun all failed tests or /retest-required
to rerun all mandatory failed tests:
Test name | Commit | Details | Required | Rerun command |
---|---|---|---|---|
ci/prow/versions | 5038186e1c1bf9c86054444310484cc351357f36 | link | false | /test versions |
Full PR test history. Your PR dashboard.
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: machine424, simonpasquier
The full list of commands accepted by this bot can be found here.
The pull request process is described here
/hold unhold whenever you want.
/hold cancel
[ART PR BUILD NOTIFIER]
This PR has been included in build cluster-monitoring-operator-container-v4.16.0-202402141610.p0.g6449c59.assembly.stream.el9 for distgit cluster-monitoring-operator. All builds following this will include this PR.
It should speed up the test run if we check the condition immediately instead of waiting for 5 seconds.