-
Notifications
You must be signed in to change notification settings - Fork 9.4k
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
Update module aws/aws-sdk-go to v1.25.42 #10955
Conversation
b357224
to
9b89a8f
Compare
9b89a8f
to
3ffe45f
Compare
3ffe45f
to
baf38b2
Compare
…ssions argument validation and documentation Reference: https://github.com/aws/aws-sdk-go/pull/2962/files#diff-75343d273fb0174cc5d3bc82469018b5L6419 Reference: #10955 The upstream AWS Go SDK in v1.25.39 update removes the available SDK constant. In testing the behavior against the current SDK which still has this constant and when `BEST_EFFORT` is provided as a value to the DataSync API, the API returns a validation error: ``` --- FAIL: TestAccAWSDataSyncTask_DefaultSyncOptions_PosixPermissions (233.01s) testing.go:635: Step 0 error: errors during apply: Error: error creating DataSync Task: ValidationException: 1 validation error detected: Value 'BEST_EFFORT' at 'options.posixPermissions' failed to satisfy constraint: Member must satisfy enum value set: [PRESERVE, NONE] status code: 400, request id: 1b101b77-f03f-4be2-b018-e2edf4088536 ``` This API behavior seems to indicate that even if existing Terraform configurations might contain this value, they are invalid now according to the API and must be updated regardless. If for some reason this old value is still valid for previously existing tasks, we can hardcode the `"BEST_EFFORT"` string into the validation if necessary.
Submitted #10985 to handle the SDK constant removal in the DataSync service. |
…ssions argument validation and documentation (#10985) Reference: https://github.com/aws/aws-sdk-go/pull/2962/files#diff-75343d273fb0174cc5d3bc82469018b5L6419 Reference: #10955 The upstream AWS Go SDK in v1.25.39 update removes the available SDK constant. In testing the behavior against the current SDK which still has this constant and when `BEST_EFFORT` is provided as a value to the DataSync API, the API returns a validation error: ``` --- FAIL: TestAccAWSDataSyncTask_DefaultSyncOptions_PosixPermissions (233.01s) testing.go:635: Step 0 error: errors during apply: Error: error creating DataSync Task: ValidationException: 1 validation error detected: Value 'BEST_EFFORT' at 'options.posixPermissions' failed to satisfy constraint: Member must satisfy enum value set: [PRESERVE, NONE] status code: 400, request id: 1b101b77-f03f-4be2-b018-e2edf4088536 ``` This API behavior seems to indicate that even if existing Terraform configurations might contain this value, they are invalid now according to the API and must be updated regardless. If for some reason this old value is still valid for previously existing tasks, we can hardcode the `"BEST_EFFORT"` string into the validation if necessary.
Telling renovate to rebase this with #10985 merged now. |
baf38b2
to
694fc62
Compare
694fc62
to
c7a78c2
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Took awhile, but got through reviewing these. No red flags.
This has been released in version 2.40.0 of the Terraform AWS provider. Please see the Terraform documentation on provider versioning or reach out if you need any assistance upgrading. For further feature requests or bug reports with this functionality, please create a new GitHub issue following the template for triage. Thanks! |
I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. Thanks! |
This PR contains the following updates:
v1.25.38
->v1.25.42
Release Notes
aws/aws-sdk-go
v1.25.42
Compare Source
===
Service Client Updates
service/alexaforbusiness
: Updates service API and documentationservice/appconfig
: Adds new serviceservice/application-autoscaling
: Updates service API and documentationservice/application-insights
: Updates service API, documentation, and paginatorsservice/athena
: Updates service API and documentationservice/ce
: Updates service API and documentationservice/codebuild
: Updates service API and documentationservice/cognito-idp
: Updates service APIservice/comprehend
: Updates service API and documentationservice/dlm
: Updates service API and documentationservice/ec2
: Updates service API and documentationservice/elasticloadbalancingv2
: Updates service API and documentationservice/greengrass
: Updates service API and documentationservice/iot
: Updates service API and documentationservice/iotsecuretunneling
: Adds new serviceservice/kinesisanalyticsv2
: Updates service API and documentationservice/kms
: Updates service API and documentationservice/lambda
: Updates service API, documentation, waiters, and paginatorsservice/mediaconvert
: Updates service API and documentationservice/medialive
: Updates service API, documentation, waiters, and paginatorsservice/mediapackage-vod
: Updates service API and documentationservice/monitoring
: Updates service API, documentation, and paginatorsservice/ram
: Updates service API and documentationservice/rds
: Updates service API and documentationservice/redshift
: Updates service API, documentation, and paginatorsservice/runtime.lex
: Updates service API and documentationservice/sesv2
: Updates service API, documentation, and paginatorsservice/ssm
: Updates service API and documentationSDK Enhancements
aws/credentials/processcreds
: Increase the default max buffer size (#2957)v1.25.41
Compare Source
===
Service Client Updates
service/acm
: Updates service API and documentationservice/application-autoscaling
: Updates service APIservice/autoscaling-plans
: Updates service APIservice/codebuild
: Updates service API and documentationservice/ec2
: Updates service API and documentationservice/elasticmapreduce
: Updates service API and documentationservice/forecast
: Updates service API and documentationservice/mediapackage-vod
: Updates service API and documentationservice/rekognition
: Updates service API and documentationservice/sns
: Updates service documentationservice/ssm
: Updates service API and documentationservice/sts
: Updates service API, documentation, and examplesSDK Bugs
aws/ec2metadata
: Fix failing concurrency test for ec2metadata client (#2960)v1.25.40
Compare Source
===
Service Client Updates
service/amplify
: Updates service API and documentationservice/appsync
: Updates service API and documentationservice/config
: Updates service API and documentationservice/connect
: Updates service API and documentationservice/connectparticipant
: Adds new serviceservice/dynamodb
: Updates service API and documentationservice/ec2
: Updates service API and documentationservice/glue
: Updates service API and documentationservice/lex-models
: Updates service API and documentationservice/meteringmarketplace
: Updates service documentationservice/runtime.lex
: Updates service API and documentationservice/ssm
: Updates service API and documentationservice/transcribe
: Updates service APIv1.25.39
Compare Source
===
Service Client Updates
service/AWSMigrationHub
: Updates service API, documentation, and paginatorsservice/chime
: Updates service API, documentation, and paginatorsservice/cloudtrail
: Updates service API and documentationservice/codecommit
: Updates service API, documentation, and paginatorsservice/datasync
: Updates service API and documentationservice/discovery
: Updates service API and documentationservice/dlm
: Updates service API and documentationservice/ec2
: Updates service API, documentation, waiters, and paginatorsservice/ecs
: Updates service API and documentationservice/firehose
: Updates service API and documentationservice/fsx
: Updates service API and documentationservice/mediastore
: Updates service API and documentationservice/migrationhub-config
: Adds new serviceservice/quicksight
: Updates service API, documentation, and paginatorsservice/s3
: Updates service API and documentationservice/storagegateway
: Updates service API and documentationservice/transcribe
: Updates service API and documentationRenovate configuration
📅 Schedule: At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻️ Rebasing: Whenever PR becomes conflicted, or if you modify the PR title to begin with "
rebase!
".🔕 Ignore: Close this PR and you won't be reminded about this update again.
Newsflash: Renovate has joined WhiteSource, and is now free for all use. Learn more or view updated terms and privacy policies.