fix: Duplicated restores
This commit is contained in:
parent
e40032497b
commit
20246998da
|
|
@ -180,12 +180,7 @@ func (bar *BackupAndRestore) Restore(jenkinsClient jenkinsclient.Jenkins) error
|
|||
if err != nil {
|
||||
return err
|
||||
}
|
||||
//TODO fix me because we're doing two saves unatomically
|
||||
jenkins.Spec.Restore.RecoveryOnce = 0
|
||||
err = bar.Client.Update(context.TODO(), jenkins)
|
||||
if err != nil {
|
||||
return err
|
||||
}
|
||||
|
||||
key := types.NamespacedName{
|
||||
Namespace: jenkins.Namespace,
|
||||
Name: jenkins.Name,
|
||||
|
|
@ -194,6 +189,12 @@ func (bar *BackupAndRestore) Restore(jenkinsClient jenkinsclient.Jenkins) error
|
|||
if err != nil {
|
||||
return err
|
||||
}
|
||||
|
||||
jenkins.Spec.Restore.RecoveryOnce = 0
|
||||
err = bar.Client.Update(context.TODO(), jenkins)
|
||||
if err != nil {
|
||||
return err
|
||||
}
|
||||
bar.Configuration.Jenkins = jenkins
|
||||
|
||||
jenkins.Status.RestoredBackup = backupNumber
|
||||
|
|
|
|||
Loading…
Reference in New Issue