Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(mnq): migrate to github.com/aws/aws-sdk-go-v2 #2843

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

jremy42
Copy link
Contributor

@jremy42 jremy42 commented Dec 6, 2024

fix #2836

@github-actions github-actions bot added the mnq Messaging and queuing issues, bugs and feature requests label Dec 6, 2024
@jremy42 jremy42 marked this pull request as ready for review December 10, 2024 10:55
@jremy42 jremy42 force-pushed the feat/Migrate_sqs_from_v1_to_v2 branch from 64d4939 to 059d060 Compare December 10, 2024 13:25
@jremy42 jremy42 changed the title Feat/migrate sqs from v1 to v2 Feat/migrate mnq from v1 to v2 Dec 10, 2024
@jremy42 jremy42 force-pushed the feat/Migrate_sqs_from_v1_to_v2 branch from b3ad3dc to 0d3e400 Compare December 10, 2024 13:59
@jremy42 jremy42 force-pushed the feat/Migrate_sqs_from_v1_to_v2 branch from 0d3e400 to 9828642 Compare December 10, 2024 14:09
@remyleone remyleone changed the title Feat/migrate mnq from v1 to v2 chore(mnq): migrate to github.com/aws/aws-sdk-go-v2 Dec 10, 2024
@jremy42 jremy42 force-pushed the feat/Migrate_sqs_from_v1_to_v2 branch from 78c9f1d to 84e1d20 Compare December 10, 2024 14:49
@@ -247,12 +244,12 @@ func ResourceMNQSQSQueueUpdate(ctx context.Context, d *schema.ResourceData, m in
return diag.FromErr(err)
}

queue, err := transport.RetryWhenAWSErrCodeEquals(ctx, []string{sqs.ErrCodeQueueDoesNotExist}, &transport.RetryWhenConfig[*sqs.GetQueueUrlOutput]{
queue, err := transport.RetryWhenAWSErrCodeEquals(ctx, []string{"AWS.SimpleQueueService.NonExistentQueue"}, &transport.RetryWhenConfig[*sqs.GetQueueUrlOutput]{
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This string is a constant, could it be defined at package level?

if tfawserr.ErrCodeEquals(err, "AccessDeniedException") {
var apiErr *smithy.GenericAPIError
if errors.As(err, &apiErr) && apiErr.Code == "AccessDeniedException" {
// L'erreur AccessDenied signifie que la ressource est supprimée ou inaccessible
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please use English

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
mnq Messaging and queuing issues, bugs and feature requests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Migrate sqs from v1 to v2
2 participants