From e2daf633aabd9d75f9bd6ba7e5af9c0864fe419b Mon Sep 17 00:00:00 2001 From: Scott Leggett Date: Mon, 15 Apr 2024 17:03:22 +0800 Subject: [PATCH] chore: remove unused parameter --- internal/sync/rolesmapping.go | 11 ++++++++--- internal/sync/sync.go | 2 +- 2 files changed, 9 insertions(+), 4 deletions(-) diff --git a/internal/sync/rolesmapping.go b/internal/sync/rolesmapping.go index b431479..aa25be5 100644 --- a/internal/sync/rolesmapping.go +++ b/internal/sync/rolesmapping.go @@ -116,9 +116,14 @@ func filterRolesMapping(rolesmapping map[string]opensearch.RoleMapping, // syncRolesmapping reconciles Opensearch rolesmapping with Lagoon keycloak // groups. -func syncRolesMapping(ctx context.Context, log *zap.Logger, groups []keycloak.Group, - projectNames map[int]string, roles map[string]opensearch.Role, - o OpensearchService, dryRun bool) { +func syncRolesMapping( + ctx context.Context, + log *zap.Logger, + groups []keycloak.Group, + roles map[string]opensearch.Role, + o OpensearchService, + dryRun bool, +) { // get rolesmapping from Opensearch existing, err := o.RolesMapping(ctx) if err != nil { diff --git a/internal/sync/sync.go b/internal/sync/sync.go index 5f1f1c4..f0ec20e 100644 --- a/internal/sync/sync.go +++ b/internal/sync/sync.go @@ -106,7 +106,7 @@ func Sync(ctx context.Context, log *zap.Logger, l LagoonDBService, case "roles": syncRoles(ctx, log, groups, projectNames, roles, o, dryRun) case "rolesmapping": - syncRolesMapping(ctx, log, groups, projectNames, roles, o, dryRun) + syncRolesMapping(ctx, log, groups, roles, o, dryRun) case "indexpatterns": syncIndexPatterns(ctx, log, groupsSansGlobal, projectNames, o, d, dryRun, legacyIndexPatternDelimiter)