Aaron Crickenberger, Samsung SDS
KubeCon Europe 2018
URL: https://www.youtube.com/watch?v=BsIC7gPkH5M
[Notes on the Talk]
- implementing pr workflow
- facilitate triage workflow
- configure and run jobs
- provide insights into project health
- Blame: that's just the buildcop's job
- Gain: Automate the Game of gettting
green
PRs
Ends up in frustrating situation where you check-in build and it fails and only buildcop
can solve
Prow Objectives:
- checks conventions (coding standards)
- make sure it builds!
- pass unit and integration tests
- pass e2e tests
- perf tests (using simulation cluster with kubemark)
Code Review (PR Flow Improvements)
- solve problem for knowing how to ask for a Review
- critiques on CODEOWNERS file
- assigns to all people who match that regex for that file So improve OWNERS file to look and choose selectively
- reviewers
- approvers
- labels
The project CANT live without them.
- MERGE Commit must pass Test
- 25% flake expected. (Test is 75% Consistent)
- Testst N PRs at once (batch them, maintain velocity)
/retest
command saves time by only running failed jobs