Skip to content
This repository has been archived by the owner on Sep 19, 2022. It is now read-only.

PyTorch Operator: Move manifests development upstream #318

Closed
yanniszark opened this issue Feb 10, 2021 · 0 comments · Fixed by #320
Closed

PyTorch Operator: Move manifests development upstream #318

yanniszark opened this issue Feb 10, 2021 · 0 comments · Fixed by #320
Assignees

Comments

@yanniszark
Copy link
Contributor

yanniszark commented Feb 10, 2021

Background

Umbrella-Issue: kubeflow/manifests#1740

As part of the work of wg-manifests for 1.3
(kubeflow/manifests#1735), we are moving manifests
development to upstream repos. This gives the application developers full
ownership of their manifests, tracked in a single place.

To give an example from the notebook-controller, the diagram shows the current
situation. Manifests are in two repos, blurring separation of responsibilities
and making things harder for application developers.

diagram_1

Instead, we will copy all manifests from the manifests repo back to each
upstream's repo. From there, they will be imported in the manifests repo. The
following diagram presents the desired state:

diagram_2

Current State

PyTorch Operator has manifests:

Desired State

The proposed folder in the upstream repo to receive the manifests is:
manifests.

The goal is to consolidate all manifests development in the upstream repo.
The manifests repo will include a copy of the manifests under apps/pytorch-job/upstream.
This copy will be synced periodically.

Success Criteria

The manifests in apps/pytorch-job/upstream should be a copy of the upstream manifests
folder. To do that, the application manifests in kubeflow/manifests should be
moved to the upstream application repo.

/assign @yanniszark
cc @kubeflow/wg-training-leads

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant