amend kopia error clues

This commit is contained in:
ryanfkeepers 2023-02-20 17:56:21 -07:00
parent e2530a0bd9
commit aa90a36e76
4 changed files with 9 additions and 13 deletions

View File

@ -101,8 +101,7 @@ func (c Contacts) GetContainerByID(
// EnumerateContainers iterates through all of the users current // EnumerateContainers iterates through all of the users current
// contacts folders, converting each to a graph.CacheFolder, and calling // contacts folders, converting each to a graph.CacheFolder, and calling
// fn(cf) on each one. If fn(cf) errors, the error is aggregated // fn(cf) on each one.
// into a multierror that gets returned to the caller.
// Folder hierarchy is represented in its current state, and does // Folder hierarchy is represented in its current state, and does
// not contain historical data. // not contain historical data.
func (c Contacts) EnumerateContainers( func (c Contacts) EnumerateContainers(

View File

@ -133,8 +133,7 @@ func (c Events) GetItem(
// EnumerateContainers iterates through all of the users current // EnumerateContainers iterates through all of the users current
// calendars, converting each to a graph.CacheFolder, and // calendars, converting each to a graph.CacheFolder, and
// calling fn(cf) on each one. If fn(cf) errors, the error is // calling fn(cf) on each one.
// aggregated into a multierror that gets returned to the caller.
// Folder hierarchy is represented in its current state, and does // Folder hierarchy is represented in its current state, and does
// not contain historical data. // not contain historical data.
func (c Events) EnumerateContainers( func (c Events) EnumerateContainers(

View File

@ -156,8 +156,7 @@ func (c Mail) GetItem(
// EnumerateContainers iterates through all of the users current // EnumerateContainers iterates through all of the users current
// mail folders, converting each to a graph.CacheFolder, and calling // mail folders, converting each to a graph.CacheFolder, and calling
// fn(cf) on each one. If fn(cf) errors, the error is aggregated // fn(cf) on each one.
// into a multierror that gets returned to the caller.
// Folder hierarchy is represented in its current state, and does // Folder hierarchy is represented in its current state, and does
// not contain historical data. // not contain historical data.
func (c Mail) EnumerateContainers( func (c Mail) EnumerateContainers(

View File

@ -461,16 +461,15 @@ func consumeBackupDataCollections(
return nil, nil, nil, err return nil, nil, nil, err
} }
return nil, nil, nil, errors.Wrapf( return nil, nil, nil, clues.Stack(err).With(
err, "kopia_errors", kopiaStats.ErrorCount,
"kopia snapshot failed with %v catastrophic errors and %v ignored errors", "kopia_ignored_errors", kopiaStats.IgnoredErrorCount)
kopiaStats.ErrorCount, kopiaStats.IgnoredErrorCount)
} }
if kopiaStats.ErrorCount > 0 || kopiaStats.IgnoredErrorCount > 0 { if kopiaStats.ErrorCount > 0 || kopiaStats.IgnoredErrorCount > 0 {
err = errors.Errorf( err = clues.New("building kopia snapshot").With(
"kopia snapshot failed with %v catastrophic errors and %v ignored errors", "kopia_errors", kopiaStats.ErrorCount,
kopiaStats.ErrorCount, kopiaStats.IgnoredErrorCount) "kopia_ignored_errors", kopiaStats.IgnoredErrorCount)
} }
return kopiaStats, deets, itemsSourcedFromBase, err return kopiaStats, deets, itemsSourcedFromBase, err