Skip to content

Commit

Permalink
skip actions enablement on org lvl, if enabledRepo is not set in CRD
Browse files Browse the repository at this point in the history
  • Loading branch information
lado936 committed Nov 27, 2023
1 parent ae6e4a5 commit b36ccf9
Showing 1 changed file with 12 additions and 9 deletions.
21 changes: 12 additions & 9 deletions internal/controller/organization/organization.go
Original file line number Diff line number Diff line change
Expand Up @@ -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
}
}
}

Expand All @@ -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
}
}
}

Expand Down

0 comments on commit b36ccf9

Please sign in to comment.