Skip to content

Governance#32

Draft
klanclos wants to merge 19 commits intomainfrom
governance
Draft

Governance#32
klanclos wants to merge 19 commits intomainfrom
governance

Conversation

@klanclos
Copy link
Copy Markdown
Contributor

This pull request adds a governance section to the official mKTL documentation. The contents are based on language from other resources, specifically:

https://opensource.guide/leadership-and-governance/

https://pypeit.readthedocs.io/en/stable/governance.html

The settings of the GitHub repository will be modified once the governance is accepted.

All maintainers implicitly agree to abide by the mKTL code of conduct.


Communication
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Pulling this over from Slack -

We're also missing a section for work intake and prioritization, meaning how work gets chosen in the first place

I think some areas I'd like to add to this --

Work items in mKTL may originate from multiple sources

  • Bug reports submitted via GitHub issues
  • Operational needs identified by WMKO
  • Feature requests from users or contributors

Maintainers are responsible for triaging incoming issues

  • Confirming the validity of a bug or request
  • Closing issues that are out of scope or not actionable

What does prioritization mean?
What does Selection for development mean?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I feel like if we go that route we are inviting a full Kanban board approach, and I don't know if that's strictly necessary. I'm happy to let things be brought up in a conversational context and we go from there; if that proves unworkable I'm happy to revisit the question.

much time a contributor might provide to the project.

All contributors implicitly agree to abide by the mKTL code of conduct.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Contributor status alone does not confer approval authority under this
policy. Binding approvals are provided only by maintainers.

repository. There will always be an odd number of designated maintainers
to ensure a majority vote is possible, should voting situations arise.
The minimum number of maintainers is one.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

There shall at all times be at least one designated maintainer from each
of the following participating institutions: W. M. Keck Observatory
(WMKO), University of California Observatories (UCO), and the California
Institute of Technology (Caltech). Additional maintainers may be added
as needed. The total number of maintainers need not be odd.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

If the number of maintainers is allowed to be even, is a 50% vote considered a deadlock, or is that an approval?

of the current maintainers. The initial set of maintainers is expected
to include representatives from W. M. Keck Observatory, University
of California Observatories, and the California Institute of Technology.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Any change to the list of maintainers, including addition, removal,
or change in institutional affiliation, must be approved by at least one
maintainer from each participating institution. The last maintainer seat
for a participating institution may not be removed, left vacant, or
reassigned without that institution's affirmative approval. This
requirement is intended to preserve institutional veto authority.

of California Observatories, and the California Institute of Technology.

All maintainers implicitly agree to abide by the mKTL code of conduct.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

The initial set of maintainers shall include representatives from WMKO,
UCO, and Caltech.

* major, for changes breaking backwards compatibility
* minor, for backwards-compatible additions or changes
* patch, for backwards-compatible bug fixes

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Project support

WMKO may be the principal operational customer for many mKTL deployments.
That operational role does not alter the governance or approval rights
defined in this document.

WMKO, UCO, and Caltech are the participating institutions for purposes of
project governance. Each participating institution is expected to support
the participation of at least one maintainer for the duration of its
operational or development use of mKTL, and to internally support
contributors as needed for ongoing development and operational fixes.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I'm fine with making the language stronger with respect to participation from UCO and Caltech, but I didn't want to commit resources on their behalf. If both organizations are on board with making that commitment we can revise the language.

@scizen9
Copy link
Copy Markdown
Collaborator

scizen9 commented Mar 23, 2026

Approval

For purposes of project governance, each participating institution holds
one required institutional approval seat. That approval seat may be
exercised by any current maintainer from that institution.

This policy intentionally grants WMKO, UCO, and Caltech veto authority.
Absent affirmative approval from at least one maintainer from each of
those three institutions, a change shall not be merged and a governance
action shall not be adopted.

Changes to mKTL governance require affirmative approval from at least one
maintainer from WMKO, one from UCO, and one from Caltech.

Approval of any other change to a protected branch, including code,
documentation, interface definitions, protocol changes, tests, packaging,
continuous integration configuration, and release artifacts, likewise
requires affirmative approval from at least one maintainer from WMKO,
one from UCO, and one from Caltech.

Contributor feedback and review are strongly encouraged, but contributor
comments do not satisfy the approval requirements unless the contributor
is also a maintainer.

Maintainers should avoid being the sole approving voice for their own
changes when another maintainer from the same institution is reasonably
available. Where no such maintainer exists, the institution may exercise
its approval through the submitting maintainer.

No maintainer may bypass the institutional approval requirements.

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.

4 participants