From b36ccf95f2f40d76b54397327035ebb8fe0fd8a2 Mon Sep 17 00:00:00 2001 From: Lado Golijashvili Date: Mon, 27 Nov 2023 17:50:32 +0400 Subject: [PATCH] skip actions enablement on org lvl, if enabledRepo is not set in CRD --- .../controller/organization/organization.go | 21 +++++++++++-------- 1 file changed, 12 insertions(+), 9 deletions(-) diff --git a/internal/controller/organization/organization.go b/internal/controller/organization/organization.go index 4fb23f0..0b77889 100644 --- a/internal/controller/organization/organization.go +++ b/internal/controller/organization/organization.go @@ -219,11 +219,12 @@ func (c *external) Update(ctx context.Context, mg resource.Managed) (managed.Ext if err != nil { return managed.ExternalUpdate{}, err } - - for _, missingRepo := range missingReposIds { - _, err := gh.Actions.AddEnabledReposInOrg(ctx, name, missingRepo) - if err != nil { - return managed.ExternalUpdate{}, err + if cr.Spec.ForProvider.Actions.EnabledRepos != nil { + for _, missingRepo := range missingReposIds { + _, err := gh.Actions.AddEnabledReposInOrg(ctx, name, missingRepo) + if err != nil { + return managed.ExternalUpdate{}, err + } } } @@ -233,10 +234,12 @@ func (c *external) Update(ctx context.Context, mg resource.Managed) (managed.Ext } // Disable actions for missing repositories - for _, toDeleteRepo := range toDeleteReposIds { - _, err := gh.Actions.RemoveEnabledRepoInOrg(ctx, name, toDeleteRepo) - if err != nil { - return managed.ExternalUpdate{}, err + if cr.Spec.ForProvider.Actions.EnabledRepos != nil { + for _, toDeleteRepo := range toDeleteReposIds { + _, err := gh.Actions.RemoveEnabledRepoInOrg(ctx, name, toDeleteRepo) + if err != nil { + return managed.ExternalUpdate{}, err + } } }