Fix usage of errgroup to sync team users (#829)

helderjs · web-flow · commit 5884b3f37a98 · 2023-01-10T12:15:06.000+01:00
diff --git a/pkg/controller/atlasproject/team_reconciler.go b/pkg/controller/atlasproject/team_reconciler.go
@@ -180,6 +180,7 @@ func ensureTeamUsersAreInSync(ctx context.Context, workflowCtx *workflow.Context
 		return workflow.Terminate(workflow.TeamUsersNotReady, err.Error())
 	}
 
+	g, taskContext = errgroup.WithContext(ctx)
 	toAdd := make([]string, 0, len(team.Spec.Usernames))
 	lock := sync.Mutex{}
 	for _, username := range team.Spec.Usernames {

Original file line number	Diff line number	Diff line change
`@@ -180,6 +180,7 @@ func ensureTeamUsersAreInSync(ctx context.Context, workflowCtx *workflow.Context`
`180`	`180`	`return workflow.Terminate(workflow.TeamUsersNotReady, err.Error())`
`181`	`181`	`}`
`182`	`182`
	`183`	`+ g, taskContext = errgroup.WithContext(ctx)`
`183`	`184`	`toAdd := make([]string, 0, len(team.Spec.Usernames))`
`184`	`185`	`lock := sync.Mutex{}`
`185`	`186`	`for _, username := range team.Spec.Usernames {`