feat(account): Name or email of policy, group, user or service user should be visible in deploy info logs

Use `'` to make this stand out

Author: arthurpitman

pkg/account/deployer/deployer.go

@@ -209,7 +209,7 @@ func (d *AccountDeployer) deployPolicies(ctx context.Context, policies map[strin
 		policy := policy
 		deployPolicyJob := func(wg *sync.WaitGroup, errCh chan error) {
 			defer wg.Done()
-			d.logger.Info("Deploying policy %s", policy.Name)
+			d.logger.Info("Deploying policy '%s'", policy.Name)
 			pUuid, err := d.upsertPolicy(d.logCtx(ctx), policy)
 			if err != nil {
 				errCh <- fmt.Errorf("unable to deploy policy for account %s: %w", d.accClient.getAccountInfo().AccountUUID, err)
@@ -225,7 +225,7 @@ func (d *AccountDeployer) deployGroups(ctx context.Context, groups map[string]ac
 		group := group
 		deployGroupJob := func(wg *sync.WaitGroup, errCh chan error) {
 			defer wg.Done()
-			d.logger.Info("Deploying group %s", group.Name)
+			d.logger.Info("Deploying group '%s'", group.Name)
 			gUuid, err := d.upsertGroup(d.logCtx(ctx), group)
 			if err != nil {
 				errCh <- fmt.Errorf("unable to deploy group for account %s: %w", d.accClient.getAccountInfo().AccountUUID, err)
@@ -243,7 +243,7 @@ func (d *AccountDeployer) deployUsers(ctx context.Context, users map[string]acco
 		user := user
 		deployUserJob := func(wg *sync.WaitGroup, errCh chan error) {
 			defer wg.Done()
-			d.logger.Info("Deploying user %s", user.Email)
+			d.logger.Info("Deploying user '%s'", user.Email)
 			if _, err := d.upsertUser(d.logCtx(ctx), user); err != nil {
 				errCh <- fmt.Errorf("unable to deploy user for account %s: %w", d.accClient.getAccountInfo().AccountUUID, err)
 			}
@@ -258,7 +258,7 @@ func (d *AccountDeployer) deployServiceUsers(ctx context.Context, serviceUsers [
 		serviceUser := serviceUser
 		deployServiceUserJob := func(wg *sync.WaitGroup, errCh chan error) {
 			defer wg.Done()
-			d.logger.Info("Deploying service user %s", serviceUser.Name)
+			d.logger.Info("Deploying service user '%s'", serviceUser.Name)
 			if _, err := d.upsertServiceUser(d.logCtx(ctx), serviceUser); err != nil {
 				errCh <- fmt.Errorf("unable to deploy service user for account %s: %w", d.accClient.getAccountInfo().AccountUUID, err)
 			}
@@ -271,7 +271,7 @@ func (d *AccountDeployer) deployServiceUsers(ctx context.Context, serviceUsers [
 func (d *AccountDeployer) deployGroupBindings(ctx context.Context, groups map[account.GroupId]account.Group, dispatcher *Dispatcher) {
 	for _, group := range groups {
 		group := group
-		d.logger.Info("Updating policy bindings and permissions for group %s", group.Name)
+		d.logger.Info("Updating policy bindings and permissions for group '%s'", group.Name)
 
 		updateBindingsJob := func(wg *sync.WaitGroup, errCh chan error) {
 			defer wg.Done()
@@ -295,7 +295,7 @@ func (d *AccountDeployer) deployUserBindings(ctx context.Context, users map[acco
 		deployUserBindingsJob :=
 			func(wg *sync.WaitGroup, errCh chan error) {
 				defer wg.Done()
-				d.logger.Info("Updating group bindings for user %s", user.Email)
+				d.logger.Info("Updating group bindings for user '%s'", user.Email)
 				if err := d.updateUserGroupBindings(d.logCtx(ctx), user); err != nil {
 					errCh <- fmt.Errorf("unable to deploy user binding for account %s: %w", d.accClient.getAccountInfo().AccountUUID, err)
 				}
@@ -312,7 +312,7 @@ func (d *AccountDeployer) deployServiceUserBindings(ctx context.Context, service
 		deployUserBindingsJob :=
 			func(wg *sync.WaitGroup, errCh chan error) {
 				defer wg.Done()
-				d.logger.Info("Updating group bindings for service user %s", serviceUser.Name)
+				d.logger.Info("Updating group bindings for service user '%s'", serviceUser.Name)
 				if err := d.updateServiceUserGroupBindings(d.logCtx(ctx), serviceUser); err != nil {
 					errCh <- fmt.Errorf("unable to deploy user binding for account %s: %w", d.accClient.getAccountInfo().AccountUUID, err)
 				}