Enhancement Description

• One-line enhancement description (can be used as a release note):

New label for trusted PR identification

• Kubernetes Enhancement Proposal:

This document describes a major change to the way the trigger plugin determines if test jobs should be started on a pull request (PR).

We propose introducing a new label named ok-to-test that will be applied on non-member PRs once they have been /ok-to-test by a legitimate reviewer.

We suggest introducing a new label named ok-to-test that would be required on any non-member PR before automatic test jobs can be started by the trigger plugin.

This label will be added by members of the trusted organization for the repository using the /ok-to-test command, detected with a single GenericCommentEvent handler on corresponding events (issue_comment, pull_request_review, and pull_request_review_comment).

• Discussion Link:

• Primary contact (assignee): @matthyx

• Responsible SIGs:

sig-testing

• Enhancement target (which target equals to which milestone):
  • Alpha release target (x.y):
  • Beta release target (x.y):
  • Stable release target (x.y):
• Alpha
  • KEP (k/enhancements) update PR(s):
  • Code (k/k) update PR(s):
  • Docs (k/website) update PR(s):

References

• ok-to-test not detected in review approval test-infra#3827
• "/ok-to-test" is not respected when it's inline review comment. test-infra#7801
• Trust a PR if it doesn't have "needs-ok-to-test" test-infra#5246

Implementation History

• 2018-06-25: creation of the KEP
• 2018-07-09: KEP content LGTM during sig-testing presentation
• 2018-07-24: KEP updated to keep needs-ok-to-test for better UX
• 2018-09-03: KEP rewritten with template
• 2018-10-04: KEP merged into master
• 2018-10-08: start of implementation
• 2018-10-10: ok-to-test label added
• 2019-01-29: remove comment parsing code for PR trust

Please keep this description up to date. This will help the Enhancement Team to track the evolution of the enhancement efficiently.