Skip to content

Commit 9b0502a

Browse files
committed
ssm client wrapper removed
1 parent d3b8408 commit 9b0502a

File tree

4 files changed

+8
-42
lines changed

4 files changed

+8
-42
lines changed

go.sum

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -94,13 +94,10 @@ github.com/aws/aws-sdk-go-v2/service/secretsmanager v1.28.6 h1:TIOEjw0i2yyhmhRry
9494
github.com/aws/aws-sdk-go-v2/service/secretsmanager v1.28.6/go.mod h1:3Ba++UwWd154xtP4FRX5pUK3Gt4up5sDHCve6kVfE+g=
9595
github.com/aws/aws-sdk-go-v2/service/servicequotas v1.21.4 h1:SSDkZRAO8Ok5SoQ4BJ0onDeb0ga8JBOCkUmNEpRChcw=
9696
github.com/aws/aws-sdk-go-v2/service/servicequotas v1.21.4/go.mod h1:plXue/Zg49kU3uU6WwfCWgRR5SRINNiJf03Y/UhYOhU=
97-
<<<<<<< HEAD
9897
github.com/aws/aws-sdk-go-v2/service/sqs v1.38.8 h1:80dpSqWMwx2dAm30Ib7J6ucz1ZHfiv5OCRwN/EnCOXQ=
9998
github.com/aws/aws-sdk-go-v2/service/sqs v1.38.8/go.mod h1:IzNt/udsXlETCdvBOL0nmyMe2t9cGmXmZgsdoZGYYhI=
100-
=======
10199
github.com/aws/aws-sdk-go-v2/service/ssm v1.59.1 h1:Z4cmgV3hKuUIkhJsdn47hf/ABYHUtILfMrV+L8+kRwE=
102100
github.com/aws/aws-sdk-go-v2/service/ssm v1.59.1/go.mod h1:PUWUl5MDiYNQkUHN9Pyd9kgtA/YhbxnSnHP+yQqzrM8=
103-
>>>>>>> bb4fbed33 (Migrate SSM code to AWS SDK V2 -)
104101
github.com/aws/aws-sdk-go-v2/service/sso v1.20.5 h1:vN8hEbpRnL7+Hopy9dzmRle1xmDc7o8tmY0klsr175w=
105102
github.com/aws/aws-sdk-go-v2/service/sso v1.20.5/go.mod h1:qGzynb/msuZIE8I75DVRCUXw3o3ZyBmUvMwQ2t/BrGM=
106103
github.com/aws/aws-sdk-go-v2/service/ssooidc v1.23.4 h1:Jux+gDDyi1Lruk+KHF91tK2KCuY61kzoCpvtvJJBtOE=

pkg/cloud/scope/clients.go

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -37,8 +37,7 @@ import (
3737
"github.com/aws/aws-sdk-go/service/resourcegroupstaggingapi/resourcegroupstaggingapiiface"
3838
"github.com/aws/aws-sdk-go/service/secretsmanager"
3939
"github.com/aws/aws-sdk-go/service/secretsmanager/secretsmanageriface"
40-
"github.com/aws/aws-sdk-go/service/sqs"
41-
"github.com/aws/aws-sdk-go/service/ssm"
40+
4241
"github.com/aws/aws-sdk-go/service/sts"
4342
"github.com/aws/aws-sdk-go/service/sts/stsiface"
4443
"k8s.io/apimachinery/pkg/runtime"

pkg/cloud/services/ec2/service.go

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -40,11 +40,9 @@ type Service struct {
4040
// NewService returns a new service given the ec2 api client.
4141
func NewService(clusterScope scope.EC2Scope) *Service {
4242
return &Service{
43-
scope: clusterScope,
44-
EC2Client: scope.NewEC2Client(clusterScope, clusterScope, clusterScope, clusterScope.InfraCluster()),
45-
SSMClient: &ssm.SSMClientV2{
46-
Client: scope.NewSSMClient(clusterScope, clusterScope, clusterScope, clusterScope.InfraCluster()),
47-
},
43+
scope: clusterScope,
44+
EC2Client: scope.NewEC2Client(clusterScope, clusterScope, clusterScope, clusterScope.InfraCluster()),
45+
SSMClient: scope.NewSSMClient(clusterScope, clusterScope, clusterScope, clusterScope.InfraCluster()),
4846
netService: network.NewService(clusterScope.(scope.NetworkScope)),
4947
}
5048
}

pkg/cloud/services/ssm/service.go

Lines changed: 4 additions & 32 deletions
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,6 @@ package ssm
1818

1919
import (
2020
"context"
21-
"errors"
2221

2322
"github.com/aws/aws-sdk-go-v2/service/ssm"
2423

@@ -42,40 +41,13 @@ type SSMAPI interface {
4241
// Add more methods as needed
4342
}
4443

45-
// SSMClientV2 is a concrete implementation of the SSMAPI interface using AWS SDK v2.
46-
// It wraps PutParameter for potential custom logic while using the AWS SDK directly for other methods.
47-
type SSMClientV2 struct {
48-
Client *ssm.Client
49-
}
50-
51-
// PutParameter adds or overwrites a parameter in AWS SSM Parameter Store.
52-
// This method is wrapped to allow for custom error handling or retry logic if needed.
53-
func (c *SSMClientV2) PutParameter(ctx context.Context, input *ssm.PutParameterInput, optFns ...func(*ssm.Options)) (*ssm.PutParameterOutput, error) {
54-
if c.Client == nil {
55-
return nil, errors.New("SSM client is not initialized")
56-
}
57-
return c.Client.PutParameter(ctx, input, optFns...)
58-
}
59-
60-
// DeleteParameter deletes a parameter from AWS SSM Parameter Store.
61-
func (c *SSMClientV2) DeleteParameter(ctx context.Context, input *ssm.DeleteParameterInput, optFns ...func(*ssm.Options)) (*ssm.DeleteParameterOutput, error) {
62-
return c.Client.DeleteParameter(ctx, input, optFns...)
63-
}
64-
65-
// GetParameter retrieves a parameter from AWS SSM Parameter Store.
66-
func (c *SSMClientV2) GetParameter(ctx context.Context, input *ssm.GetParameterInput, optFns ...func(*ssm.Options)) (*ssm.GetParameterOutput, error) {
67-
return c.Client.GetParameter(ctx, input, optFns...)
68-
}
69-
70-
// Ensure SSMClientV2 satisfies the SSMAPI interface.
71-
var _ SSMAPI = &SSMClientV2{}
44+
// Ensure ssm.Client satisfies the SSMAPI interface.
45+
var _ SSMAPI = &ssm.Client{}
7246

7347
// NewService creates a new Service for managing secrets in AWS SSM.
7448
func NewService(secretsScope cloud.ClusterScoper) *Service {
7549
return &Service{
76-
scope: secretsScope,
77-
SSMClient: &SSMClientV2{
78-
Client: scope.NewSSMClient(secretsScope, secretsScope, secretsScope, secretsScope.InfraCluster()),
79-
},
50+
scope: secretsScope,
51+
SSMClient: scope.NewSSMClient(secretsScope, secretsScope, secretsScope, secretsScope.InfraCluster()),
8052
}
8153
}

0 commit comments

Comments
 (0)