Skip to content

Commit d76d2a1

Browse files
authored
Merge pull request #1055 from haarchri/fix/gluesecurityconf
cleanup glue securityconfig &cr.name to annotation
2 parents bfa1d10 + b6de9f0 commit d76d2a1

1 file changed

Lines changed: 1 addition & 2 deletions

File tree

  • pkg/controller/glue/securityconfiguration

pkg/controller/glue/securityconfiguration/setup.go

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,6 @@ func SetupSecurityConfiguration(mgr ctrl.Manager, l logging.Logger, limiter work
5555
For(&svcapitypes.SecurityConfiguration{}).
5656
Complete(managed.NewReconciler(mgr,
5757
resource.ManagedKind(svcapitypes.SecurityConfigurationGroupVersionKind),
58-
managed.WithInitializers(managed.NewDefaultProviderConfig(mgr.GetClient())),
5958
managed.WithExternalConnecter(&connector{kube: mgr.GetClient(), opts: opts}),
6059
managed.WithPollInterval(poll),
6160
managed.WithLogger(l.WithValues("controller", name)),
@@ -89,7 +88,7 @@ func postCreate(_ context.Context, cr *svcapitypes.SecurityConfiguration, obj *s
8988
}
9089

9190
func preCreate(_ context.Context, cr *svcapitypes.SecurityConfiguration, obj *svcsdk.CreateSecurityConfigurationInput) error {
92-
obj.Name = &cr.Name
91+
obj.Name = awsclients.String(meta.GetExternalName(cr))
9392

9493
if cr.Spec.ForProvider.CustomEncryptionConfiguration != nil {
9594
obj.EncryptionConfiguration = &svcsdk.EncryptionConfiguration{

0 commit comments

Comments
 (0)