#176 Small refactoring
This commit is contained in:
parent
dbf6184898
commit
d68430cdf1
|
|
@ -219,19 +219,19 @@ func triggerBackup(ticker *time.Ticker, k8sClient k8s.Client, logger logr.Logger
|
||||||
func (bar *BackupAndRestore) EnsureBackupTrigger() error {
|
func (bar *BackupAndRestore) EnsureBackupTrigger() error {
|
||||||
trigger, found := triggers.get(bar.jenkins.Namespace, bar.jenkins.Name)
|
trigger, found := triggers.get(bar.jenkins.Namespace, bar.jenkins.Name)
|
||||||
|
|
||||||
isBackupConfigured := len(bar.jenkins.Spec.Backup.ContainerName) == 0 || bar.jenkins.Spec.Backup.Interval == 0
|
isBackupConfigured := len(bar.jenkins.Spec.Backup.ContainerName) > 0 && bar.jenkins.Spec.Backup.Interval > 0
|
||||||
if found && !isBackupConfigured {
|
if found && !isBackupConfigured {
|
||||||
bar.StopBackupTrigger()
|
bar.StopBackupTrigger()
|
||||||
return nil
|
return nil
|
||||||
}
|
}
|
||||||
|
|
||||||
// configured backup has no trigger
|
// configured backup has no trigger
|
||||||
if !found {
|
if !found && isBackupConfigured {
|
||||||
bar.startBackupTrigger()
|
bar.startBackupTrigger()
|
||||||
|
return nil
|
||||||
}
|
}
|
||||||
|
|
||||||
// configured interval has changes
|
if found && isBackupConfigured && bar.jenkins.Spec.Backup.Interval != trigger.interval {
|
||||||
if found && bar.jenkins.Spec.Backup.Interval != trigger.interval {
|
|
||||||
bar.StopBackupTrigger()
|
bar.StopBackupTrigger()
|
||||||
bar.startBackupTrigger()
|
bar.startBackupTrigger()
|
||||||
}
|
}
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue