-
Notifications
You must be signed in to change notification settings - Fork 9.6k
[bitnami/harbor] Add support for registry storage upload purging setting #33587
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: main
Are you sure you want to change the base?
Conversation
Signed-off-by: Kevin Delbegue <[email protected]>
Signed-off-by: Bitnami Bot <[email protected]>
Signed-off-by: Carlos Rodríguez Hernández <[email protected]>
Signed-off-by: Bitnami Bot <[email protected]>
This Pull Request has been automatically marked as "stale" because it has not had recent activity (for 15 days). It will be closed if no further activity occurs. Thank you for your contribution. |
@carrodher any news on this review ? |
This Pull Request has been automatically marked as "stale" because it has not had recent activity (for 15 days). It will be closed if no further activity occurs. Thank you for your contribution. |
Still not stale, @migruiz4 or @carrodher any news ? |
This Pull Request has been automatically marked as "stale" because it has not had recent activity (for 15 days). It will be closed if no further activity occurs. Thank you for your contribution. |
Not stale... Maybe @juan131 ? |
Description of the change
Add the option to configure of the uploadpurging setting in the registry
storage.maintenance
section.Benefits
With this change, the registry could clean the
_uploads
folders if enabled, which could save a lot of space on the storage backend.Possible drawbacks
Currently, the value is hardcoded to
enabled: false
, in order to prevent massive deletes on the storage backend, the defaultenabled: false
has been kept in the values.yaml file.As opposed to the default values of distribution/distribution which is
enabled: true
.Applicable issues
Additional information
Checklist
Chart.yaml
according to semver. This is not necessary when the changes only affect README.md files.README.md
using readme-generator-for-helm