-
Notifications
You must be signed in to change notification settings - Fork 37
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
Expose DBPurge Age and Schedule #436
Expose DBPurge Age and Schedule #436
Conversation
@ASBishop FYI: I have a WIP patch that will be rebased on top of this to simplify the cronJob handling in the operator for image-cache, but I wanted to share this change w/ you as it's supposed to align Glance and the API/interface with what we've done in cinder/manila. |
This patch updates the current glance CR to expose both dbPurgeAge and dbPurgeSchedule as customizable parameters. They were previously hardcoded as constants in the code, and it makes sense removing that part as long as the human operator has the ability to customize them. This also aligns the glance-operator with the work already done in both Manila and Cinder. Signed-off-by: Francesco Pantano <[email protected]>
85e98da
to
8b44f08
Compare
/test functional |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: abays, fmount The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
7c87bfa
into
openstack-k8s-operators:main
This patch updates the current glance CR to expose both
dbPurgeAge
anddbPurgeSchedule
as customizable parameters. They were previously hardcoded as constants in the code, and it makes sense removing that part as long as the human operator has the ability to customize them. This also aligns theglance-operator
with the work already done in bothManila
andCinder
.