Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

charter #8114

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

charter #8114

wants to merge 1 commit into from

Conversation

macfarla
Copy link
Contributor

@macfarla macfarla commented Jan 13, 2025

PR description

This PR proposes the addition of a Besu charter. Please add your feedback, questions etc in the form of comments on this PR.

Charter text originates from https://github.com/besu-eth/governance/blob/main/Besu%20Technical%20Charter.md however that charter is proposed for removal in this PR besu-eth/governance#7 since it was added without consultation with Besu maintainers.

Note this is a re-draft of #7128

This has come up again for discussion as part of move from Hyperledger to LFDT, and the relocation of this repo from hyperledger github org.

For context, as a LFDT project, Besu is already subject to the LFDT charter https://www.lfdecentralizedtrust.org/about/charter

Fixed Issue(s)

Thanks for sending a pull request! Have you done the following?

  • Checked out our contribution guidelines?
  • Considered documentation and added the doc-change-required label to this PR if updates are required.
  • Considered the changelog and included an update if required.
  • For database changes (e.g. KeyValueSegmentIdentifier) considered compatibility and performed forwards and backwards compatibility tests

Locally, you can run these tests to catch failures early:

  • unit tests: ./gradlew build
  • acceptance tests: ./gradlew acceptanceTest
  • integration tests: ./gradlew integrationTest
  • reference tests: ./gradlew ethereum:referenceTests:referenceTests

Signed-off-by: Sally MacFarlane <[email protected]>
Copy link
Contributor

@siladu siladu left a comment

Choose a reason for hiding this comment

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

I am not aware of any discussion about the need to have a Besu-specific charter in the first place. Until all maintainers are included in that discussion, and until I understand the benefit, I reject the idea of adding one, regardless of the content.


2. **Technical Steering Committee**

1. The Technical Steering Committee (the “TSC”) will be responsible for all technical oversight of the open source Project.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

If this is intended to be a Besu-specific TSC (as opposed to a LFDT-level body), I think this needs to be clarified


2. Maintainers are Contributors who have earned the ability to modify (“commit”) source code, documentation or other technical artifacts in a project’s repository; and

3. A Contributor may become a Maintainer by a majority approval of the existing Maintainers. A Maintainer may be removed by a majority approval of the other existing Maintainers.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

this process is described in more detail already in the MAINTAINERS.md


1. While the Project aims to operate as a consensus-based community, if any TSC decision requires a vote to move the Project forward, the voting members of the TSC will vote on a one vote per voting member basis.

2. Quorum for TSC meetings requires at least fifty percent of all voting members of the TSC to be present. The TSC may continue to meet if quorum is not met but will be prevented from making any decisions at the meeting.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Besu decision making is generally conducted async


**Technical** **Charter (the “Charter”)**
**for**
**Besu a Series of LF Projects, LLC**
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Is this just boilerplate or is this charter intended to cover multiple repositories?


6. establishing community norms, workflows, issuing releases, and security issue reporting policies;

7. approving and implementing policies and processes for contributing (to be published in the CONTRIBUTING file) and coordinating with the series manager of the Project (as provided for in the Series Agreement, the “Series Manager”) to resolve matters or concerns that may arise as set forth in Section 7 of this Charter;
Copy link
Contributor Author

Choose a reason for hiding this comment

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

who is the Series Manager, where is the Series Agreement?

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.

2 participants