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

Retire storage enabled volumes for the OCI registry #1093

Open
4 tasks
michael-valdron opened this issue Apr 3, 2023 · 6 comments
Open
4 tasks

Retire storage enabled volumes for the OCI registry #1093

michael-valdron opened this issue Apr 3, 2023 · 6 comments
Labels
area/registry Devfile registry for stacks and infrastructure good first issue Good for newcomers

Comments

@michael-valdron
Copy link
Member

michael-valdron commented Apr 3, 2023

Which area/kind this issue is related to?

/area registry

Issue Description

Currently, the registry operator and the registry helm chart support the usage of a storage enabled volume for the OCI registry path to be mounted on. This allowed OCI artifacts to persist across deployments of the registry. These OCI artifacts at this point are created from the source content on the devfile-index (registry server) image, therefore in addition with security challenges with OpenShift it is best that support for this feature is retired.

The registry operator field spec.storage.enabled should be deprecated and the helm chart field persistence.enabled should also be deprecated. The field spec.storage.enabled should be disabled by default in #1092.

Acceptance Criteria

  • spec.storage.enabled should be disabled by default
  • spec.storage.enabled should be marked as deprecated
  • persistence.enabled should be marked as deprecated
  • Update documentation to reflect these changes
@openshift-ci openshift-ci bot added the area/registry Devfile registry for stacks and infrastructure label Apr 3, 2023
@github-actions
Copy link

github-actions bot commented Jul 3, 2023

This issue is stale because it has been open for 90 days with no activity. Remove stale label or comment or this will be closed in 60 days.

@github-actions github-actions bot added the lifecycle/stale Stale items. These items have not been updated for 90 days. label Jul 3, 2023
@michael-valdron
Copy link
Member Author

This is a future change that is planned to occur, removing the stale status.

@michael-valdron michael-valdron removed the lifecycle/stale Stale items. These items have not been updated for 90 days. label Jul 20, 2023
@thepetk
Copy link
Contributor

thepetk commented Aug 4, 2023

We might need to add the acceptance criteria for this issue. Putting it on "waiting" for now.

@michael-valdron
Copy link
Member Author

We might need to add the acceptance criteria for this issue. Putting it on "waiting" for now.

This issue description has been updated to include the acceptance criteria items.

@elsony
Copy link
Contributor

elsony commented Aug 11, 2023

Make sure the doc includes that the doc mentioned will be using emptyDir storage by default.

@Horiodino
Copy link

Hii , i would like to work on the issue .can refer where to make changes exactly?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/registry Devfile registry for stacks and infrastructure good first issue Good for newcomers
Projects
Status: Backlog
Development

No branches or pull requests

4 participants