test-infra icon indicating copy to clipboard operation
test-infra copied to clipboard

Published 20 hours ago • verified publisher icon kubernetes

Tide for Gerrit: comment on batch merges

Open chaodaiG opened this issue 2 years ago • 2 comments

In case of flaky tests, Tide triggered prowjobs for highest priority PR might fail even when batch prowjobs passed. And in this case Crier would report this failure on the PR before Tide merges the PR, this might cause confusing to users so comment on the PR explaining that the merge was based on batch testing.

/cc @cjwagner @listx

chaodaiG avatar Oct 14 '22 17:10 chaodaiG

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: chaodaiG

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment Approvers can cancel approval by writing /approve cancel in a comment

k8s-ci-robot avatar Oct 14 '22 17:10 k8s-ci-robot

/lgtm /hold

listx avatar Oct 21 '22 17:10 listx

/unhold

chaodaiG avatar Oct 21 '22 17:10 chaodaiG