Skip to content

Commit

Permalink
Merge pull request #4446 from cnmcavoy/cmcavoy/awsmanagedcontrolplane…
Browse files Browse the repository at this point in the history
…-tag-unmanaged-network-resources

Fix AWSManagedControlPlane controller to use TagUnmanagedNetworkResources feature gate
  • Loading branch information
k8s-ci-robot authored Aug 17, 2023
2 parents 44e0edc + 7ec53c2 commit 13ae637
Showing 1 changed file with 9 additions and 8 deletions.
17 changes: 9 additions & 8 deletions main.go
Original file line number Diff line number Diff line change
Expand Up @@ -345,14 +345,15 @@ func setupEKSReconcilersAndWebhooks(ctx context.Context, mgr ctrl.Manager, awsSe

setupLog.Debug("enabling EKS control plane controller")
if err := (&ekscontrolplanecontrollers.AWSManagedControlPlaneReconciler{
Client: mgr.GetClient(),
EnableIAM: enableIAM,
AllowAdditionalRoles: allowAddRoles,
Endpoints: awsServiceEndpoints,
WatchFilterValue: watchFilterValue,
ExternalResourceGC: externalResourceGC,
AlternativeGCStrategy: alternativeGCStrategy,
WaitInfraPeriod: waitInfraPeriod,
Client: mgr.GetClient(),
EnableIAM: enableIAM,
AllowAdditionalRoles: allowAddRoles,
Endpoints: awsServiceEndpoints,
WatchFilterValue: watchFilterValue,
ExternalResourceGC: externalResourceGC,
AlternativeGCStrategy: alternativeGCStrategy,
WaitInfraPeriod: waitInfraPeriod,
TagUnmanagedNetworkResources: feature.Gates.Enabled(feature.TagUnmanagedNetworkResources),
}).SetupWithManager(ctx, mgr, controller.Options{MaxConcurrentReconciles: awsClusterConcurrency, RecoverPanic: pointer.Bool(true)}); err != nil {
setupLog.Error(err, "unable to create controller", "controller", "AWSManagedControlPlane")
os.Exit(1)
Expand Down

0 comments on commit 13ae637

Please sign in to comment.