HoustonPutman commented on a change in pull request #359:
URL: https://github.com/apache/solr-operator/pull/359#discussion_r741153892
##########
File path: api/v1beta1/solrbackup_types.go
##########
@@ -64,6 +71,29 @@ func (spec *SolrBackupSpec) withDefaults(backupName string)
(changed bool) {
return changed
}
+// BackupRecurrence defines the recurrence of the incremental backup
+type BackupRecurrence struct {
+ // Perform a backup on the given schedule, in CRON format.
+ //
+ // Multiple CRON syntaxes are supported
+ // - Standard CRON (e.g. "CRON_TZ=Asia/Seoul 0 6 * * ?")
+ // - Predefined Schedules (e.g. "@yearly", "@weekly", "@daily", etc.)
+ // - Intervals (e.g. "@every 10h30m")
+ //
+ // For more information please check this reference:
+ //
https://pkg.go.dev/github.com/robfig/cron/v3?utm_source=godoc#hdr-CRON_Expression_Format
+ Schedule string `json:"schedule"`
+
+ // Define the number of backup points to save for this backup at any
given time.
+ // The oldest backups will be deleted if too many exist when a backup
is taken.
+ // If not provided, this defaults to 10.
+ //
+ // +kubebuilder:default:=10
+ // +kubebuilder:validation:Minimum:=1
+ // +optional
+ MaxSaved int `json:"maxSaved,omitempty"`
Review comment:
Very open. I literally just chose a number here. I am very happy to
differ to whatever you think it should be. Give the call and it is done.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org
For additional commands, e-mail: issues-h...@solr.apache.org
