Skip to content

Expand and document PSRT coordination process#1769

Open
sethmlarson wants to merge 9 commits intopython:mainfrom
sethmlarson:psrt-process
Open

Expand and document PSRT coordination process#1769
sethmlarson wants to merge 9 commits intopython:mainfrom
sethmlarson:psrt-process

Conversation

@sethmlarson
Copy link
Contributor

@sethmlarson sethmlarson commented Mar 19, 2026

This PR defining the coordination process more concretely. Here are the changes:

  • Changes the process to automatically assign a Coordinator after a period of idleness. The exact selection process isn't defined, at this stage I am looking for reviews on whether this change to the process is acceptable to PSRT members.
  • The coordinator role per-ticket is explicit within GHSA using the "Credit: Coordinator" feature.
  • Separates the Triage, Remediation, Advisory sub-processes more clearly. This intended to be helpful for non-PSRT members that are involved in the remediation process to know what to expect.
  • Documents how the PSRT bot operates so coordinators know what to expect and how to take advantage of the automation.

cc @python/psrt

@read-the-docs-community
Copy link

read-the-docs-community bot commented Mar 19, 2026

Documentation build overview

📚 CPython devguide | 🛠️ Build #31990451 | 📁 Comparing ecf801b against latest (d5194a0)


🔍 Preview build

Show files changed (6 files in total): 📝 4 modified | ➕ 2 added | ➖ 0 deleted
File Status
index.html 📝 modified
_static/psrt-coordinator-report.html ➕ added
_static/psrt-coordinator-triage.html ➕ added
developer-workflow/index.html 📝 modified
core-team/team-log/index.html 📝 modified
developer-workflow/psrt/index.html 📝 modified

@sethmlarson sethmlarson marked this pull request as ready for review March 19, 2026 21:23
sethmlarson and others added 3 commits March 20, 2026 18:28
Co-authored-by: Stan Ulbrych <89152624+StanFromIreland@users.noreply.github.com>
Co-authored-by: Hugo van Kemenade <1324225+hugovk@users.noreply.github.com>
@hugovk
Copy link
Member

hugovk commented Mar 21, 2026

The diagram lines are hard to read in dark mode, can we adjust them?

Details image

@sethmlarson
Copy link
Contributor Author

@hugovk Done in 8332acd

@sethmlarson sethmlarson requested a review from hugovk March 25, 2026 22:18
Copy link
Member

@hugovk hugovk left a comment

Choose a reason for hiding this comment

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

Please could you add a note in the PR how you generated Mermaid -> SVG?


Also a number of the boxes are clipped on the right, I don't know if it's possible to increase the margin. Not the biggest problem in the world :)

Image Image Image Image

psrt_member_volunteers[PSRT member volunteers as Coordinator]
psrt_member_assigned[After 3 days idle, Coordinator is auto-assigned]
coordinator_assigned[GHSA ticket updated to set Credit for Coordinator]
coordinator_determines[Cooridnator determines whether report...]
Copy link
Member

@hugovk hugovk Mar 26, 2026

Choose a reason for hiding this comment

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

Also needs fixing in the SVG.

Suggested change
coordinator_determines[Cooridnator determines whether report...]
coordinator_determines[Coordinator determines whether report...]

Co-authored-by: Hugo van Kemenade <1324225+hugovk@users.noreply.github.com>
@sethmlarson
Copy link
Contributor Author

@hugovk I'm coming to the unfortunate conclusion that maybe Mermaid is more work than it's worth. I remember this language and the tooling around it being so much nicer in the past :/ Oh well.

I think I'm going to instead use an ASCII diagram unless there's a better method for diagramming you know of?

@sethmlarson
Copy link
Contributor Author

Here is how the graph would look as an ASCII diagram, light and dark modes included:

Screenshot from 2026-03-26 12-25-07 Screenshot from 2026-03-26 12-25-20

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants