Skip to content

Conversation

@xuezhaojun
Copy link
Member

@xuezhaojun xuezhaojun commented Aug 13, 2025

Changes Made

This PR refactors the governance document structure to improve clarity and organization:

1. Renamed Section

  • Renamed "Default Subproject Governance" to "Subproject Governance" to align with "Core Project Governance"

2. Reorganized Project Management Sections

  • Moved "Adding to Core Project" and "Removing from Core Project" under Core Project Governance section
  • Moved "Adding Subprojects" and "Removing Subprojects" under Subproject Governance section

3. Separated Core vs Subproject Processes

  • Core Project: Requires 2/3 majority vote for removal due to critical nature
  • Subproject: Requires simple majority vote for removal, more streamlined process

4. Simplified Governance Structure

  • Removed steering committee references: Eliminated separate steering committee concept
  • Unified governance model: Core Project Maintainers now directly handle both technical and governance responsibilities
  • Maintainer Council: All Core Project Maintainers collectively form the governing body

5. Updated Table of Contents

  • Reflects the new hierarchical structure
  • Updated internal references throughout the document

Benefits

  • Better Organization: Project management operations are now grouped with their respective governance sections
  • Clearer Distinction: Separates core project processes from subproject processes
  • Simplified Governance: Eliminates confusion between steering committee and maintainer roles
  • Unified Leadership: Core Project Maintainers handle both technical and strategic decisions
  • Improved Navigation: More logical document structure and updated TOC
  • Consistent Naming: Parallel naming convention between Core Project Governance and Subproject Governance

This change makes the governance document more intuitive to navigate, eliminates governance complexity, and better reflects the unified leadership model where Core Project Maintainers serve as both technical and governance leaders.


Vote Tracker

@openshift-ci openshift-ci bot requested review from deads2k and mikeshng August 13, 2025 07:05
@xuezhaojun
Copy link
Member Author

/assign @qiujian16

Signed-off-by: xuezhaojun <[email protected]>
@xuezhaojun xuezhaojun changed the title Refactor governance structure: reorganize project add/remove sections Refactor governance. Aug 14, 2025
@xuezhaojun
Copy link
Member Author

xuezhaojun commented Aug 15, 2025

Hi @lbocinco, I’ve updated the PR. Could you review it again when you have time to confirm if it’s ready for a community vote to merge? Thanks!

- Community feedback is considered valuable input but is non-binding
- Major decisions should be communicated to the community for comment before implementation

### Adding to Core Project

Choose a reason for hiding this comment

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

Suggested change
### Adding to Core Project
### Adding Core Project components

Copy link
Member Author

Choose a reason for hiding this comment

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

The preposition "to" clearly indicates the destination of the addition, following standard English grammar patterns. It precisely means "adding [components] to the Core Project" rather than the ambiguous "adding Core Project components." This structure maintains consistency with common technical documentation conventions like "Deploying to..." or "Migrating to...".

Before submitting an application to the Steering Committee, the applying component must hold an internal consensus vote of all major contributors to join Open Cluster Management as part of the Core Project. The Steering Committee will then review the application, and decide whether or not to accept it. If accepted, the Committee will assign one person to assist the component in their integration and ensure compliance with core governance requirements.
Before submitting an application to the Core Project Maintainer Council, the applying component must hold an internal consensus vote of all major contributors to join Open Cluster Management as part of the Core Project. The Maintainer Council will then review the application through discussion and consensus, and decide whether or not to accept it. If accepted, the Council will assign one or more maintainers to assist the component in their integration and ensure compliance with core governance requirements.

### Removing from Core Project

Choose a reason for hiding this comment

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

Suggested change
### Removing from Core Project
### Removing Core Project components

Copy link
Member Author

Choose a reason for hiding this comment

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

The preposition "from" provides clear syntax that explicitly indicates the source of removal, following standard English grammar. It precisely means "removing [components] from the Core Project" rather than the ambiguous "removing Core Project components." This structure aligns with common technical documentation patterns like "Migrating from..." or "Importing from...".

Choose a reason for hiding this comment

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

Not my hill to die on, but the subject of the addition/removal is unclear in both section titles.

How about:

  • Adding components to the Core Project
  • Removing components from the Core Project

?

Promising but incomplete projects may be accepted as Experimental Subprojects. Such projects will be marked as Experimental on the website and in code repositories to inform users. Experimental projects are considered part of the OCM ecosystem with limited governance expectations. The Steering Committee will review Experimental projects at least twice per year to determine if they have matured to full status.

## Removing Projects
Promising but incomplete projects may be accepted as Experimental Subprojects. Such projects will be marked as Experimental on the website and in code repositories to inform users. Experimental projects are considered part of the OCM ecosystem with limited governance expectations. The Core Project Maintainer Council will review Experimental projects at least twice per year to determine if they have matured to full status.

Choose a reason for hiding this comment

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

Copy link
Member Author

Choose a reason for hiding this comment

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

Good question! I don't think we need to link the lab repo because when a project in the lab matures, we'll consider spinning it out independently. The lab serves as a transitional stage for subprojects, not a permanent location.

What do you think? @qiujian16 @mikeshng

Copy link
Member

Choose a reason for hiding this comment

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

seems to me when we talk about Experimental Subprojects, might be good to have a link on where such subproject will be located.

Copy link
Member

@mikeshng mikeshng left a comment

Choose a reason for hiding this comment

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

+1

@openshift-ci
Copy link

openshift-ci bot commented Aug 24, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: mikeshng, xuezhaojun

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@mikeshng
Copy link
Member

/hold

@openshift-merge-bot openshift-merge-bot bot merged commit 99412d6 into open-cluster-management-io:main Aug 24, 2025
2 checks passed
xuezhaojun added a commit to xuezhaojun/ocm-community that referenced this pull request Aug 25, 2025
@xuezhaojun
Copy link
Member Author

The previous PR was merged accidentally. This new PR will be used to track progress: #228

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants