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

Create Model Serving Team: Consolidate model serving repo permissions into one team #79

Merged
merged 1 commit into from
Jun 21, 2023

Conversation

heyselbi
Copy link
Contributor

Description

Three changes:

  1. Restructured model serving repos into one model serving team
  2. Added Mariah Holder (see issue: REQUEST: New membership for mholder6 #74 )
  3. Added Selbi's name to opendatahub-community, for ability to edit the trackers for GitHub Projects/Issues

New Open Data Hub Member Requirements

Pull Request Requirements

  • New members are added in alphabetical order
  • Only one new member change per commit (if you add two members separate it in two commits
  • For individual user changes: Commit message format Add <USERNAME> to <opendatahub-io, mlops-sig, ...>.
  • For new team requests: Commit message format Create <TEAMNAME>. If the new team consists solely of existing members, you may
  • New GitHub team requests are from an existing opendatahub-io member who will function as the maintainer of the team. Each additional member will follow the same requirements for adding new members

config/opendatahub-io/org.yaml Outdated Show resolved Hide resolved
config/opendatahub-io/org.yaml Show resolved Hide resolved
@heyselbi
Copy link
Contributor Author

@LaVLaS @jkoehler-redhat could I get the approval for these changes? Looks like approval has to come from either of you per OpenShift CI.

@LaVLaS LaVLaS self-requested a review June 21, 2023 15:32
@LaVLaS LaVLaS merged commit 8d1184d into opendatahub-io:main Jun 21, 2023
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.

3 participants