Skip to content

Clean up dead code in kubeflow/kubeflow? #5419

Closed
@jlewi

Description

Follow on to kubeflow/community#454?

The question posed in that issue is:

What's the kubeflow/kubeflow positioning? Should we deprecate it once notebook and control plane components move out of it?

With the exception of notebooks and one or two other packages (e.g. PodDefaults) that should probably move into WG owned repositories (#5418) it looks like most of the code in kubeflow/kubeflow is dead code. We should probably delete it and aim to whiddle down the repo to just a readme.

Looking at the top level directories

  • bootstrap - this can be deleted; all relevant code should now be in kubeflow/kfctl
  • build - This looks like some utilities for managing license files in code
    • We should probably keep it but might want to move it
    • kubeflow/testing is generally where we've been gathering shared engprod resources
  • components - This directory should be whiddled down to nothing
    • Sub directories should either be moved into WG owned repositories or be deleted if they are no longer being developed/maintained
  • docs/gke - This should probably move to a GCP directory (gcp-blueprints?) or be deleted if no longer used
  • hack - should probably move to kubeflow/testing for shared engprod
  • metric-collector - I don't believe this is being maintained so we can probably just delete it
    • If a WG wants to claim it they should move it to one of their repos
  • py/kubeflow - I don't think this is being used anymore so it can probably be deleted
    • It looks like its just engprod scripts so if its being used it should move to kubeflow/testing
  • releasing - dead code; just delete
  • scripts
    • This looks like outdated scripts that have been superseded by other solutions
    • WGs should probably claim anything if they still need it.
    • I would suggest just deleting the directory to shakeout who might still be depending on it
  • testing
    • Similar to scripts
    • Any generic testing code still in use should live in kubeflow/testing
  • tf-controller-examples
    • Delete; looks very stale
    • Any operator specific examples should move into training operator repos
  • tools/gcb - delete

Metadata

Assignees

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions