Require self-assessed technical and governance reviews for matriculation#2114
Merged
brandtkeller merged 2 commits intocncf:mainfrom May 6, 2026
Merged
Require self-assessed technical and governance reviews for matriculation#2114brandtkeller merged 2 commits intocncf:mainfrom
brandtkeller merged 2 commits intocncf:mainfrom
Conversation
Signed-off-by: Josh Gavant <joshgavant@gmail.com>
joshgav
force-pushed
the
review-template
branch
from
5f0875d to
dffcced
Compare
Contributor
|
+1 This is a welcome addition! Hope to see it go through! |
brandtkeller
requested changes
Apr 16, 2026
Member
brandtkeller
left a comment
brandtkeller
left a comment
There was a problem hiding this comment.
Passing thought on issue template. Still reviewing updates.
Contributor
|
This needs to be discussed by the TOC - thank you for the recommendation from the Subproject @joshgav |
github-actions
Bot
added
the
needs-triage
github-project-automation
Bot
added this to
TAG Workloads Foundation and TAG Security and Compliance
github-actions
Bot
added
the
needs-kind
github-actions
Bot
added
the
needs-group
Co-authored-by: Brandt Keller <43887158+brandtkeller@users.noreply.github.com> Signed-off-by: Karena Angell <kangell@redhat.com>
brandtkeller
approved these changes
May 6, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR proposes requiring preliminary self-assessed technical and governance reviews for incoming and matriculating projects. It's based on consensus reached in the project-reviews subproject on 2026-01-22. We determined then that projects should complete their own technical reviews ("GTR") and that technical presentations to TAGs should be secondary and optional, with a process for such presentations yet to be determined. CNCF reviewers will then take a project's technical self-assessment, verify it and commit it to the cncf/toc repo, as they do for governance reviews today.
This PR also proposes a congruent process for governance reviews: a project will self-assess a governance review like the technical review prior to requesting matriculation. Projects already do exactly this today inline in matriculation issues, so this just asks them to save the assessment somewhere alongside their technical review rather than only including answers in the issue.
Ultimately, the idea is that this will lead to technical, governance, and security reviews being saved and persisted by a project, e.g., in a
communityor.projectrepo, and referenced from matriculation issues, rather than being inline in the matriculation issue itself. This will make review docs easier to maintain and iterate upon for projects and easier for users to discover.