Don't try to fetch backup version if no compete backups available (#3230)
Regression from 62daf10213 (diff-f088d3a5f56348ddaf6fbf485c2aeaa6d8b40a860de014f7084db6daf0753829R267)
---
#### Does this PR need a docs update or release note?
- [ ] ✅ Yes, it's included
- [ ] 🕐 Yes, but in a later PR
- [x] ⛔ No
#### Type of change
<!--- Please check the type of change your PR introduces: --->
- [ ] 🌻 Feature
- [x] 🐛 Bugfix
- [ ] 🗺️ Documentation
- [ ] 🤖 Supportability/Tests
- [ ] 💻 CI/Deployment
- [ ] 🧹 Tech Debt/Cleanup
#### Issue(s)
<!-- Can reference multiple issues. Use one of the following "magic words" - "closes, fixes" to auto-close the Github issue. -->
* #<issue>
#### Test Plan
<!-- How will this be tested prior to merging.-->
- [x] 💪 Manual
- [ ] ⚡ Unit test
- [ ] 💚 E2E
This commit is contained in:
parent
6395dcbe39
commit
534924f696
@ -241,8 +241,9 @@ func (op *BackupOperation) do(
|
|||||||
backupID model.StableID,
|
backupID model.StableID,
|
||||||
) (*details.Builder, error) {
|
) (*details.Builder, error) {
|
||||||
var (
|
var (
|
||||||
reasons = selectorToReasons(op.Selectors, false)
|
reasons = selectorToReasons(op.Selectors, false)
|
||||||
fallbackReasons = makeFallbackReasons(op.Selectors)
|
fallbackReasons = makeFallbackReasons(op.Selectors)
|
||||||
|
lastBackupVersion = version.NoBackup
|
||||||
)
|
)
|
||||||
|
|
||||||
logger.Ctx(ctx).With(
|
logger.Ctx(ctx).With(
|
||||||
@ -264,9 +265,11 @@ func (op *BackupOperation) do(
|
|||||||
return nil, clues.Wrap(err, "producing manifests and metadata")
|
return nil, clues.Wrap(err, "producing manifests and metadata")
|
||||||
}
|
}
|
||||||
|
|
||||||
_, lastBackupVersion, err := lastCompleteBackups(ctx, op.store, mans)
|
if canUseMetaData {
|
||||||
if err != nil {
|
_, lastBackupVersion, err = lastCompleteBackups(ctx, op.store, mans)
|
||||||
return nil, clues.Wrap(err, "retrieving prior backups")
|
if err != nil {
|
||||||
|
return nil, clues.Wrap(err, "retrieving prior backups")
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
cs, excludes, err := produceBackupDataCollections(
|
cs, excludes, err := produceBackupDataCollections(
|
||||||
|
|||||||
Loading…
x
Reference in New Issue
Block a user