-
Notifications
You must be signed in to change notification settings - Fork 244
Adjust the platform/manifests charter according to the last 5 years and the state of 2025 #837
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
base: master
Are you sure you want to change the base?
Changes from 4 commits
659ad32
cdb3c7d
b30d85f
a5401ba
63a6bb7
cb9c26f
42f7f12
08734c2
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,70 +1,32 @@ | ||
| # WG Manifests Charter | ||
| # WG Platform/Manifests Charter | ||
juliusvonkohout marked this conversation as resolved.
Outdated
Show resolved
Hide resolved
|
||
|
|
||
| This charter adheres to the conventions, roles and organization management | ||
| outlined in [wg-governance]. | ||
| This platform/manifests charter describes the working mode / reality / status quo of the last 5 years as of March 2025. | ||
| It tries to be as lean as possible and balance community and commercial interests. | ||
juliusvonkohout marked this conversation as resolved.
Outdated
Show resolved
Hide resolved
|
||
|
|
||
| ## Scope | ||
|
|
||
| - Provide a catalog (centralized repository) of Kubeflow application manifests. | ||
| - Provide a catalog of third-party apps for common services. | ||
|
|
||
| ### In scope | ||
|
|
||
| #### Code, Binaries and Services | ||
juliusvonkohout marked this conversation as resolved.
Show resolved
Hide resolved
|
||
|
|
||
| - Maintain tooling to automate copying manifests from upstream app repos. | ||
juliusvonkohout marked this conversation as resolved.
Show resolved
Hide resolved
|
||
| - Maintain a catalog that will allow users to install Kubeflow apps and | ||
| common services easily on Kubernetes, either on the cloud or on-prem, without | ||
| depending on external cloud services or closed source solutions. Those | ||
juliusvonkohout marked this conversation as resolved.
Show resolved
Hide resolved
|
||
| manifests are deployed using `kubectl` and `kustomize` and include: | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. We also need to ensure that Also, are we allowing other deployment tools beyond |
||
| 1. A common set of manifests for the current official Kubeflow applications: | ||
| - Training Operators | ||
| - Kubeflow Pipelines (KFP) | ||
| - Notebooks | ||
| - KFServing | ||
| - Katib | ||
| - Central Dashboard | ||
| - Profile Controller | ||
| - PodDefaults Controller | ||
| 1. Manifests for a set of specific common services: | ||
| - Istio | ||
| - KNative | ||
| - Dex | ||
| - Cert-Manager | ||
|
|
||
| #### Cross-cutting and Externally Facing Processes | ||
|
|
||
| ##### With Application Owners | ||
|
|
||
| - Aid applications owners in creating kustomize manifests for their application, | ||
| inside the app repo, if those don't exist already. | ||
| - Communicate with application owners to agree upon the version they want to be | ||
| included in the next Kubeflow release. | ||
|
|
||
| ##### With Distribution Owners | ||
|
|
||
| - Coordinate with distribution owners, to make sure they are in-sync about the | ||
| release schedule and have time to test and bring their distributions | ||
| up-to-date. | ||
|
|
||
| ### Out of scope | ||
juliusvonkohout marked this conversation as resolved.
Show resolved
Hide resolved
|
||
|
|
||
| This WG is NOT going to: | ||
| - Maintain deployment-specific tools like `kfctl`. | ||
| - Maintain distribution-specific manifests. | ||
| - Decide which applications to include in Kubeflow. | ||
juliusvonkohout marked this conversation as resolved.
Show resolved
Hide resolved
|
||
| - Decide which variant of an application to include (e.g., KFP Standalone vs | ||
| KFP with Istio). | ||
| - Create and maintain one or more Kubeflow distributions. | ||
juliusvonkohout marked this conversation as resolved.
Show resolved
Hide resolved
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. (Opening a new thread because the old one was marked as resolved) It is critical that we still explicitly exclude the creation of a distribution by the manifest working group, as this would create a massive conflict of interest. |
||
| - Support configurations with environment-specific requirements, like special | ||
| hardware, different versions of third-party apps (e.g., Istio, KNative, etc.) | ||
| or custom OIDC providers. | ||
| - Support and promote a specific deployment tool (e.g., `kfctl`). Opinionated | ||
| deployment tools can extend the base kustomizations to create manifests that | ||
| support their methods. | ||
| - For example, people invested in `kfctl` can create overlays that enable | ||
| the use of `kfctl`'s parameter substitution, which expects a specific | ||
| folder structure (`params.env`). | ||
| - Enable users / distributions to install, extend and maintain Kubeflow as a multi-tenant platform for multiple users | ||
juliusvonkohout marked this conversation as resolved.
Outdated
Show resolved
Hide resolved
|
||
| - This includes dependencies, security efforts and exemplary integration with popular tools and frameworks. | ||
| - Synchronize the manifests (Helm, Kustomize) between working groups | ||
juliusvonkohout marked this conversation as resolved.
Outdated
Show resolved
Hide resolved
|
||
| - We try to be compatible with the popular Kubernetes clusters (Kind, Rancher, AKS, EKS, GKE, ...) | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Why is this necessary for the manifest working group? We intentionally excluded this goal from the original manifest wg charter to prevent unnecessary focus on vendor-specific issues.
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. In practice the users are going to have different Kubernetes layers (Kind, Rancher, AKS, EKS, GKE, ...) but this only covers Kubernetes, not AWS managed databases or so. We definitely try to be compatible with the most popular ones although we cannot guarantee it. Right now it works on Kind, Rancher, AKS, EKS, GKE for me and this is also what most users expect. So it is a "soft goal" we try for our users, but we do not guarantee it. In the end this is done by volunteers, that is what we want to work on. This is where we see the value in contributing to Kubeflow. If someone else wants to focus on something else he is free to do that what is sustainable and valuable for him. No one is forced to work on that.
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I thought the Should we say that instead? |
||
| - **We do not support a specific deployment tool (e.g., ArgoCD, Flux)** | ||
| - The default installation shall not contain deep integration with external cloud services or closed source solutions, instead we aim for Kubernetes-native solutions and light authentication and authorization integration with external IDPs | ||
| - We provide hints and experimental examples how a user / distribution could integrate non-default external authentication (e.g. companies Identity Provider) and popular non-default services on his own | ||
|
||
| - There is the evolving and not exhaustive list of dependencies for a proper multi-tenant platform installation: Istio, KNative, Dex, Oauth2-proxy, Cert-Manager, ... | ||
| - There is the evolving and not exhaustive list of applications: KFP, Trainer, Dashboard, Workspaces / Noteboks, Kserve, Spark, ... | ||
|
|
||
| ## Communication Tasks | ||
|
|
||
| ### With Application Owners | ||
|
|
||
| - Aid the application owner in creating manifests (Helm, Kustomize) for his application | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I'm not sure that requiring the manifests WG to support the upstream manifests is sustainable. But obviously, it is something that the individuals who are participating might also choose to do if they are so inclined.
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. You always have to keep in mind that we are volunteers. All of this is best-effort. We try it. Sometimes other working groups need help to understand for example securitycontexts of a pod, since they are rather focused on the source code. Or we help them to fix the kustomize 5 warnings.
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I get that many contributors are volunteers, but either way, the WG charters are governance documents. It's important for the health of the WG (and project) that we set reasonable expectations for the working group members. I am not sure it's sustainable to include the expectation of upstream manifest maintenance, this is why the original charter focused only on "aggregating manifests".
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Over the last five years it was very sustainable and we made great progress. |
||
| - Aid the application owner regarding security best practices | ||
| - Communicate with the application owner regarding releases and versioning | ||
|
|
||
| ### With Users / Distribution Owners | ||
| - Distributions are strongly opinionated derivatives of Kubeflow platform/manifests, for example replacing all databases with closed source managed databases from AWS, GKE, Azure, ... | ||
| - A distribution can be created by an arbitrary amount of users / companies in private or in public by deriving from Kubeflow platform/manifests, see the definition above | ||
|
||
| - Coordinate with "distribution owners" / users to take part in the testing of Kubeflow releases. | ||
|
|
||
| ## Roles and Organization Management | ||
|
|
||
|
|
@@ -76,36 +38,36 @@ The positions of the Chairs and TLs are granted to the organizations and compani | |
| Kubeflow's [governance model](https://github.com/kubeflow/community/blob/master/wgs/wg-governance.md) | ||
| includes a plethora of different leadership roles. | ||
| This section aims to provide a clear description of what these roles mean for | ||
| this repo, as well as set expectations from people with these roles and requirements | ||
| this repository, as well as set expectations from people with these roles and requirements | ||
| for people to be promoted in a role. | ||
|
|
||
| A Working Group lead is considered someone that has either the role of | ||
| **Subproject Owner**, **Tech Lead** or **Chair**. These roles were defined by trying | ||
| to provide different responsibility levels for repo owners. For the Manifests WG | ||
| we'd like to start by treating *approvers* in the root [OWNERS](https://github.com/kubeflow/manifests/blob/master/OWNERS), | ||
| to provide different responsibility levels for repository owners. For the Manifests WG | ||
| we would like to start by treating *approvers* in the root [OWNERS](https://github.com/kubeflow/manifests/blob/master/OWNERS), | ||
| as Subproject Owners, Tech Leads and Chairs. This is done to ensure we have a | ||
| simple enough model to start that people can understand and get used to. So for | ||
| the Manifests WG we only have Manifests WG Leads, which are the root approvers. | ||
|
|
||
| The following sections will aim to define the requirements for someone to become | ||
| a reviewer and an approver in the root OWNERS file (Manifests WG Lead). | ||
|
|
||
| ### Manifests WG Lead Requirements | ||
| ### Platform/Manifests WG Lead Requirements | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Lets keep the name of the WG the same, as discussed above. |
||
|
|
||
| The requirements for someone to be a Lead come from the processes and work required | ||
| to be done in this repo. The main goal with having multiple Leads is to ensure | ||
| to be done in this repository. The main goal with having multiple Leads is to ensure | ||
| that in case there's an absence of one of the Leads the rest will be able to ensure | ||
| the established processes and the health of the repo will be preserved. | ||
| the established processes and the health of the repository will be preserved. | ||
|
|
||
| With the above the main pillars of work and responsibilities that we've seen for | ||
| this repo throughout the years are the following: | ||
| 1. Being involved with the release team, since the [release process](https://github.com/kubeflow/community/tree/master/releases) is tightly intertwined with the manifests repo | ||
| 2. Testing methodologies (GitHub Actions, E2E testing with AWS resources etc) | ||
| 3. Processes regarding the [contrib/addon](https://github.com/kubeflow/manifests/blob/master/contrib) components | ||
| 4. [Common manifests](https://github.com/kubeflow/manifests/tree/master/common) maintained by Manifests WG (Istio, Knative, Cert Manager etc) | ||
| this repository throughout the years are the following: | ||
| 1. Being involved with the release team, since the [release process](https://github.com/kubeflow/community/tree/master/releases) is tightly intertwined with the manifests/platform repository | ||
| 2. Testing methodologies (GitHub Actions) | ||
| 3. Processes regarding the [experimental](https://github.com/kubeflow/manifests/blob/master/experimental) components | ||
| 4. [Platform manifests](https://github.com/kubeflow/manifests/tree/master/common) maintained irectly by Manifests/Platform WG (Istio, Knative, Cert Manager etc.) | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. There is a typo, and also lets keep the name as |
||
| 5. Community and health of the project | ||
|
|
||
| Root approvers, or Manifests WG Leads, are expected to have expertise and be able | ||
| Root approvers, or Manifests/Platform WG Leads, are expected to have expertise and be able | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Lets keep the name as |
||
| to drive all the above areas. Root reviewers on the other hand are expected to | ||
| have knowledge in all the above and have as a goal to grow into the approvers | ||
| role by helping with reviews throughout the project. | ||
|
|
@@ -120,7 +82,7 @@ role by helping with reviews throughout the project. | |
|
|
||
| The goal of the requirements is to quantify the main pillars that we documented | ||
| above. The high level reasoning is that approvers should have lead efforts and | ||
| have expertise in the different processes and artefacts maintained in this repo | ||
| have expertise in the different processes and artefacts maintained in this repository | ||
| as well as be invested in the community of the WG. | ||
|
|
||
| * Need to be a root reviewer | ||
|
|
||
Uh oh!
There was an error while loading. Please reload this page.