Reenable point-in-time longevity test (#4090)
Now that kopia won't try to mutate state in read-only mode it's safe to reenable the test ensuring backups that were deleted are still available when opening the repo at a specific point in time This reverts commit 8d3fdeeb8dbd8dc68284c1d33b0b431d309b4f5b. --- #### 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 - [ ] 🌻 Feature - [ ] 🐛 Bugfix - [ ] 🗺️ Documentation - [x] 🤖 Supportability/Tests - [x] 💻 CI/Deployment - [ ] 🧹 Tech Debt/Cleanup #### Issue(s) * closes #4031 merge after: * #4089 #### Test Plan - [x] 💪 Manual - [ ] ⚡ Unit test - [ ] 💚 E2E
This commit is contained in:
parent
719d6b98cd
commit
3a5fcdce99
@ -67,9 +67,6 @@ func deleteBackups(
|
||||
// pitrListBackups connects to the repository at the given point in time and
|
||||
// lists the backups for service. It then checks the list of backups contains
|
||||
// the backups in backupIDs.
|
||||
//
|
||||
//nolint:unused
|
||||
//lint:ignore U1000 Waiting for upstream fix tracked by 4031
|
||||
func pitrListBackups(
|
||||
ctx context.Context,
|
||||
service path.ServiceType,
|
||||
@ -159,10 +156,16 @@ func main() {
|
||||
fatal(ctx, "invalid number of days provided", nil)
|
||||
}
|
||||
|
||||
_, err = deleteBackups(ctx, service, days)
|
||||
beforeDel := time.Now()
|
||||
|
||||
backups, err := deleteBackups(ctx, service, days)
|
||||
if err != nil {
|
||||
fatal(ctx, "deleting backups", clues.Stack(err))
|
||||
}
|
||||
|
||||
if err := pitrListBackups(ctx, service, beforeDel, backups); err != nil {
|
||||
fatal(ctx, "listing backups from point in time", clues.Stack(err))
|
||||
}
|
||||
}
|
||||
|
||||
func fatal(ctx context.Context, msg string, err error) {
|
||||
|
||||
Loading…
x
Reference in New Issue
Block a user