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

Allow configuration for internal docker registry volume #46

Closed
6 tasks
kwiatekus opened this issue Mar 19, 2024 · 2 comments
Closed
6 tasks

Allow configuration for internal docker registry volume #46

kwiatekus opened this issue Mar 19, 2024 · 2 comments
Labels
wontfix This will not be worked on

Comments

@kwiatekus
Copy link
Contributor

kwiatekus commented Mar 19, 2024

Description

Expose internal docker registry storage size in the serverless CR:

  spec:
    dockerRegistry:
      enableInternal: true
      size: 20Gi
  • when edited in the DockerRegistry CR the operator should reconfigure the PVC
  • operator should execute a rollout restart of the docker registry workload in order to allow volume expansion
  • provide documentation about this configurability option
  • update outdated documentation
  • when actual PVC size is different from desired (default=20Gi) than DockerRegistry CR should result in warning condition
  • actual used size is shown in the status

Reasons

At the moment it it a fixed value of 20 Gi and for some users who use serverless extensively it is not enough.
Additionally, if we expose it for other usecases as well (kyma-project/kyma#18555) than the need to expand it would become even more urgent.

@kwiatekus
Copy link
Contributor Author

Depends on #19

@kwiatekus kwiatekus transferred this issue from kyma-project/serverless May 21, 2024
@kwiatekus kwiatekus removed their assignment May 27, 2024
@kwiatekus
Copy link
Contributor Author

Docker registry comes now with the option to configure pvc or allows setting alternative HA storage variants like s3 bucket, azure storage ,gcs or SAP BTP Object Store.
Managing the volume size of the default PV seems redundant, and also introduces lifecycle management problems related with resizing of already mounted volumes.
Closing as wontfix

@kwiatekus kwiatekus added the wontfix This will not be worked on label Sep 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wontfix This will not be worked on
Projects
None yet
Development

No branches or pull requests

2 participants