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

Removed BDFL #18

Closed
wants to merge 1 commit into from
Closed

Removed BDFL #18

wants to merge 1 commit into from

Conversation

bitwiseman
Copy link
Contributor

@oleg-nenashev @rtyler
After some review and discussion, I've removed the BDFL from the JEP.

Copy link
Member

@oleg-nenashev oleg-nenashev left a comment

Choose a reason for hiding this comment

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

Mostly LGTM, the main concern is "Jenkins Governance Board" and "Jenkins Governance Meeting". The first term is used, but from what I see @bitwiseman rather means the latter term. Please correct me if I am wrong.

Any core contributor that believes they have
suitable knowledge and standing in the community to make the final decision on that JEP
may offer to serve as the "Reviewer" for that JEP.
If their self-nomination is accepted by the other core contributors,
Copy link
Member

Choose a reason for hiding this comment

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

how it happens? Voting at the governance meeting? Or just a feedback in the JEP PR discussion/ML?


If the Reviewer for a JEP is a contributor other than the BDFL,
this will be recorded in the <<header-reviewer, "Reviewer" header>> in the JEP.
Unlike, Sponsors, there is only one Reviewer for each JEP.
Copy link
Member

Choose a reason for hiding this comment

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

I do not see strong need to have only one reviewer. I would just say "There may be multiple reviewers assigned. In order to get the JEP accepted, there should be at least one positive vote and no negative ones. So any reviewer has veto power, and they can abstain as well"

@@ -316,15 +298,15 @@ the idea is applicable to the entire community and not just the sponsor. Just
because an idea sounds good to the sponsor does not mean it will work for most
people in most areas where Jenkins is used.

A <<JEP Reviewer>> should be selected before the JEP is submitted,
but may also be added or changed any time before a document is reviewed.
Copy link
Member

Choose a reason for hiding this comment

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

I see no specific need in this requirement, but OK.
I would just say that "JEP Reviewers should be selected via a discussion in the mailing list or in the Governance meeting, discussion in the pull request is NOT enough"

@@ -546,6 +523,21 @@ If one or more contributors are not responding
and the sponsor chooses to move forard without their feedback,
they should document that choice in the "<<Reasoning>>" section of the JEP.

==== Resolving Disputes
Copy link
Member

Choose a reason for hiding this comment

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

👍

the JEP process is run by
link:https://en.wikipedia.org/wiki/Consensus_decision-making[consensus].
It is the responsibility of the every contributor to respect other contributors,
listen to their perspectives, and attempt to find solution that work for everyone.
Copy link
Member

Choose a reason for hiding this comment

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

"a solution that works" or "solutions that work"

listen to their perspectives, and attempt to find solution that work for everyone.
If consensus cannot be achieved or if there is strong disagreement with the decision of a reviewer,
a contributor may request that the Jenkins Governance Board intervene.
The board will take up the matter at their next regular meeting and render a decision at the meeting following.
Copy link
Member

Choose a reason for hiding this comment

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

Just to clarify, do you mean "Jenkins Governance Board" or "Jenkins Governance Meeting"? These are different entities. Regular && public meetings apply only to the latter one.

The board will take up the matter at their next regular meeting and render a decision at the meeting following.

This should be a last resort and should happen extremely rarely.

Copy link
Member

Choose a reason for hiding this comment

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

"Governance meeting may make a decision to return JEP to draft and to assign another reviewer".
I am not sure whether Governance Meeting should be a place for "accepting"/"rejecting" JEPs though formally it should be eligible to do that as an ultimate power entity in Jenkins org.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I considered adding some statement that said JEPs could be rejected or returned to draft via this path, but not accepted. Only reviewers can accept, but as I said below, please go write it up as you believe it should be.

People expressed concern over the limits of the BDFL-based model
They felt having 1-person bottlenecks or too much power in the JEP process could be problematic.
We chose to remove the BDFL role, adding a "Reviewer" role to handle JEP review and resolution,
and directing any disputes to the Jenkins Governance Board.
Copy link
Member

Choose a reason for hiding this comment

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

s/Board/Meeting?

@bitwiseman
Copy link
Contributor Author

@oleg-nenashev Great, thanks! Instead of asking what I meant or if something should be changed, it would be super helpful if you could submit the changes you'd believe need to happen to this PR as a PR of your own that targets this PR. see #13.

For example, Board vs Meeting. I don't know. What do you think? Which ever answer it is, the PR would be more effective. You can either change it or add clarification as to why it is this one.

@oleg-nenashev
Copy link
Member

For example, Board vs Meeting. I don't know. What do you think? Which ever answer it is, the PR would be more effective. You can either change it or add clarification as to why it is this one.

I would rather prefer "Governance Meeting", because it's more public. Also there is a reelection issue in the board, which I would prefer to avoid in the JEP process.

OTOH Board resolves conflicts between contributors in Code-of-Conduct and hence resolving JEP conflicts there would be natural.

I will create a pull request

@bitwiseman
Copy link
Contributor Author

Due to objections by @kohsuke, I'm closing this. See #19.

@bitwiseman bitwiseman closed this Oct 25, 2017
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.

None yet

2 participants