search

cncf / foundation

☁️♮🏛 This repo contains several documents related to the operation of the CNCF. File non-technical issues related to CNCF here.
https://cncf.io
Other
552 stars 542 forks source link

Add Kubeflow maintainers #662

Closed annajung closed 10 months ago

annajung commented 10 months ago

Part of https://github.com/cncf/toc/issues/1139

Add Kubeflow maintainers to the maintainers list

cc Interim Steering Committee: @james-jwu @theadactyl @jbottum Katib: @andreyvelich @gaocegege @johnugeorge @tenzen-y Manifests: @elikatsis @kimwnasptd @PatrickXYS @StefanoFioravanzo @yanniszark MPI Operator: @alculquicondor @rongou @terrytangyuan Notebooks: @elikatsis @kimwnasptd @thesuperzapper @StefanoFioravanzo @yanniszark Pipelines: @chensun @zijianjoy @james-jwu @IronPan Pipelines on Tekton: @animeshsingh @ckadner @fenglixa @pugangxa @scrapcodes @Tomcli @yhwang Training Operators: @gaocegege @Jeffwan @johnugeorge @zw0610 @terrytangyuan @tenzen-y @andreyvelich

preview: https://github.com/annajung/foundation/blob/main/project-maintainers.csv

terrytangyuan commented 10 months ago

Thank you for creating this PR. Can we also add maintainers for the mpi-operator?

IIUC, we donated the mpi-operator to the CNCF as well: https://github.com/cncf/toc/blob/main/proposals/incubation/kubeflow.md#maintainers

cc: @terrytangyuan @alculquicondor

Good catch. @annajung Could you add everyone listed in https://github.com/cncf/toc/blob/main/proposals/incubation/kubeflow.md#maintainers?

annajung commented 10 months ago

@terrytangyuan The list of maintainers comes from the OWNERS file managed within these Kubeflow repos, which is more up to date than the proposal. If any modifications are needed in the list of maintainers not in the OWNERS file, I suggest modifying the OWNERS file first and updating the PR when it's OWNERS file has been updated.

zijianjoy commented 10 months ago

LGTM and cc @amye for merging this PR.

amye commented 10 months ago

This PR touches a lot more than just Kubeflow project - please revise and only add Kubeflow. Thanks!

annajung commented 10 months ago

@amye sorry, misunderstood what you meant. It's been updated now to only add Kubeflow

alculquicondor commented 10 months ago

LGTM

terrytangyuan commented 10 months ago

@annajung Thank you! Sounds good.

amye commented 10 months ago

Much easier to have just the top level org listed instead of all orgs - you can keep your own orgs over in your repos!

annajung commented 10 months ago

@amye it's been updated with an aggregated list with top level instead of multiple component levels.