Skip to content

docs: clarify new TAG role in sandbox reviews#493

Open
mfahlandt wants to merge 4 commits into
mainfrom
clarify-tag-role-sandbox-reviews
Open

docs: clarify new TAG role in sandbox reviews#493
mfahlandt wants to merge 4 commits into
mainfrom
clarify-tag-role-sandbox-reviews

Conversation

@mfahlandt

@mfahlandt mfahlandt commented Jun 2, 2026

Copy link
Copy Markdown
Member

Summary

Updated the README to reflect changes in the TAG review process for sandbox applications.

Changes

  • Removed the label and formal TAG review workflow
  • Added optional GTR via TOC Project Reviews Subproject
  • Clarified that TAG Leads and subproject members can voluntarily support by doing a GTR Until Day 0
  • Added note that projects may still reach out to TAGs directly for technical input
  • Updated examples (
  • Added links to Sandbox Review Guide

Details

The TAG review process is no longer mandatory for sandbox applications. Instead the TOC Project Reviews Subproject members can provide a GTR Day 0for additional technical context.

- Remove TAG-Assigned label and formal TAG review process
- Add optional General Technical Review via TOC Project Reviews Subproject
- TAG Leads and subproject members can support reviews until Day 0
- Clarify that projects can still reach out to TAGs directly for input
- Update examples and references to sandbox-review-guide
Aligns with TOC process changes removing mandatory TAG reviews.

Signed-off-by: Mario Fahlandt <mfahlandt@pixel-haufen.de>
@mfahlandt mfahlandt force-pushed the clarify-tag-role-sandbox-reviews branch from 2847939 to b116858 Compare June 2, 2026 15:20
@mfahlandt mfahlandt requested a review from Copilot June 5, 2026 15:16
@mfahlandt mfahlandt requested a review from a team June 5, 2026 15:17

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the repository README to reflect the current CNCF Sandbox application review process, clarifying that TAG involvement is optional (via General Technical Reviews) and no longer a required, label-driven workflow.

Changes:

  • Removes the formal TAG-assigned review label/workflow from the documented sandbox process.
  • Adds guidance and links for optional General Technical Reviews (GTR) by TAG Leads and TOC Project Reviews Subproject members up to Day 0.
  • Updates the example flow and status definitions to align with the new process (Need-Info/Returning without TAG-Assigned).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread README.md Outdated
@mfahlandt

Copy link
Copy Markdown
Member Author

i missclicked for the review 🫠

@mfahlandt mfahlandt moved this to New in CNCF TOC Board Jun 9, 2026
@chira001 chira001 self-assigned this Jun 9, 2026

@salaboy salaboy left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Karena Angell <kangell@redhat.com>
@angellk angellk self-requested a review June 9, 2026 17:05

@angellk angellk left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Per TOC process over the last few years, typically any TOC comments and review are due on Sandbox issues in the 2 weeks prior to Sandbox Review.

The project maintainers require time to respond and have any follow-up questions or feedback.

To get to Sandbox zero, the TOC began enlisting TAG leads to help with the reviews. At that time, the TOC had requested 3 weeks prior to any Sandbox review.

The TAG Restructure has thrown a wrench in the usual processes, however, going forward, 2 weeks minimum to leave comments/reviews should still be adhered to.

Comment thread README.md Outdated
Comment thread README.md Outdated
mfahlandt and others added 2 commits June 10, 2026 08:42
Co-authored-by: Karena Angell <karena.angell@gmail.com>
Signed-off-by: Mario <mfahlandt@pixel-haufen.de>
Co-authored-by: Karena Angell <karena.angell@gmail.com>
Signed-off-by: Mario <mfahlandt@pixel-haufen.de>
@mfahlandt

Copy link
Copy Markdown
Member Author

addressed comments @angellk

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: New

Development

Successfully merging this pull request may close these issues.

6 participants