From a4791af7bf79b82e6e66a90b52109efcbc772cbb Mon Sep 17 00:00:00 2001 From: ashmrtn Date: Fri, 23 Dec 2022 11:57:49 -0800 Subject: [PATCH] Remove some dead code in upload.go (#1920) ## Description Code was missed when switching to having BackupOp supply the OwnersCats for a backup. ## Does this PR need a docs update or release note? - [ ] :white_check_mark: Yes, it's included - [ ] :clock1: Yes, but in a later PR - [x] :no_entry: No ## Type of change - [ ] :sunflower: Feature - [ ] :bug: Bugfix - [ ] :world_map: Documentation - [ ] :robot: Test - [ ] :computer: CI/Deployment - [x] :hamster: Trivial/Minor ## Issue(s) * #1781 ## Test Plan - [ ] :muscle: Manual - [x] :zap: Unit test - [ ] :green_heart: E2E --- src/internal/kopia/upload.go | 8 -------- 1 file changed, 8 deletions(-) diff --git a/src/internal/kopia/upload.go b/src/internal/kopia/upload.go index a4cae93d8..9fc987eb0 100644 --- a/src/internal/kopia/upload.go +++ b/src/internal/kopia/upload.go @@ -575,10 +575,6 @@ func inflateCollectionTree( // Temporary variable just to track the things that have been marked as // changed while keeping a reference to their path. changedPaths := []path.Path{} - ownerCats := &OwnersCats{ - ResourceOwners: make(map[string]struct{}), - ServiceCats: make(map[string]ServiceCat), - } for _, s := range collections { switch s.State() { @@ -621,10 +617,6 @@ func inflateCollectionTree( ) } - serviceCat := serviceCatTag(s.FullPath()) - ownerCats.ServiceCats[serviceCat] = ServiceCat{} - ownerCats.ResourceOwners[s.FullPath().ResourceOwner()] = struct{}{} - // Make sure there's only a single collection adding items for any given // path in the new hierarchy. if node.collection != nil {