Page MenuHomeFreeBSD

Clarify Commit Bit and Project Resources Policy
ClosedPublic

Authored by jrm on Thu, Aug 13, 5:50 PM.
Tags
None
Referenced Files
F167277763: D58830.diff
Thu, Aug 20, 2:43 PM
Unknown Object (File)
Mon, Aug 17, 9:12 PM
Unknown Object (File)
Sat, Aug 15, 3:04 PM
Unknown Object (File)
Fri, Aug 14, 10:25 PM
Unknown Object (File)
Thu, Aug 13, 8:47 PM
Unknown Object (File)
Thu, Aug 13, 8:47 PM
F166465663: image.png
Thu, Aug 13, 5:54 PM
Subscribers

Details

Summary

Elaborate on how the core team delegates management of
repository-specific resources, such as commit bits, to the responsible
teams.

Add guidelines for suspending or revoking a commit bit: except where
circumstances justify immediate action, teams are expected to contact
the committer and provide an opportunity to address the problem before
acting, and to inform both the committer and the core team when a commit
bit is suspended or revoked.

Diff Detail

Repository
R9 FreeBSD doc repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 75572
Build 72455: arc lint + arc unit

Event Timeline

jrm requested review of this revision.Thu, Aug 13, 5:50 PM
jrm created this revision.
This revision is now accepted and ready to land.Thu, Aug 13, 5:55 PM

I have some suggestions

website/content/en/internal/resources.adoc
16

The core team reserves the right to review and modify decisions to grant, revoke or suspend a commit bit, but prefers to defer to its delegated partners where possible.

20

or to conduct that damages developer relations.

21

I'd change 'contact' here to 'engage the committer to resolve the the issue if possible before suspending or revoking access.'

24

The core team generally avoids second-guessing the delegated teams when established due process has been followed, and the decisions do not appear arbitrary.

@imp, these are changes that attempt to address your feedback. There are two places where I think I captured your sentiment, but proposed some tweaks.

  1. Saying "conduct that damages developer relations" might be too broad.

Maybe we can be more specific by deferring to the authoritative document:

A commit bit may be suspended or revoked in response to a policy violation, link:{../code-of-conduct}[a conduct violation], or actions that put the integrity of a repository at risk.

  1. The core team generally avoids second-guessing the delegated teams when established due process has been followed, and the decisions do not appear arbitrary.

I agree with this message, but I'm concerned with using "due process" when it's not clearly established (yet). I intentionally described these as "guidelines" to imply that the various teams have agency. How about something like this that, I think, says something similar, but backs off "due process" until that's clearly established.

However, the core team generally avoids second-guessing teams that have followed these guidelines.

This revision now requires review to proceed.Thu, Aug 13, 7:41 PM
  • Link to commit bit expiration policy
  • Tweak some wording
This revision is now accepted and ready to land.Tue, Aug 18, 10:47 PM