On Wed, Mar 13, 2024 at 11:48 AM Patrick Donnelly <pdonnell(a)redhat.com> wrote:
At the CLT today we discussed the proliferation of
write/admin access
on the ceph repository. One of the consequences of this has been that
Ceph's code guidelines have not been followed in merges [1].
Additionally, having too many folks -- many of whom have retired from
active development -- with write access to the repository presents
security concerns.
This experiment is suspended because it's been more disruptive than
intended and it breaks the code review workflow in GitHub. GitHub only
considers reviews from users with "write" access valid for satisfying
the "Require approvals" check for the "main" branch. Since a
component
lead cannot review their own PR, a fellow team member's review is
insufficient for merging.
--
Patrick Donnelly, Ph.D.
He / Him / His
Red Hat Partner Engineer
IBM, Inc.
GPG: 19F28A586F808C2402351B93C3301A3E258DD79D