feat: read maxAttempts value from retry-config

[trivikr]

Issue #, if available:
Follow-up to #1284

Description of changes:

read maxAttempts value from retry-config

• maxAttempts is now updated from number to Provider<string>
  • Reason: The values in environment variables and config are stored as strings, and retry-config-provider utility functions are shared between maxAttempts (string) and retryMode (string)
• maxAttempts is removed from RetryStrategy interface
• retryMiddleware is always added in getRetryPlugin, as maxAttempts value is not available.
  • retryMiddleware decides to not retry in case maxAttempts is less than or equal to 1

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

[codecov-commenter]

Codecov Report

Merging #1286 into master will increase coverage by 0.13%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1286      +/-   ##
==========================================
+ Coverage   73.04%   73.17%   +0.13%     
==========================================
  Files         287      290       +3     
  Lines       12730    12792      +62     
  Branches     2922     2936      +14     
==========================================
+ Hits         9299     9361      +62     
  Misses       3431     3431              

Impacted Files	Coverage Δ
clients/client-s3/S3Client.ts	94.11% <0.00%> (ø)
clients/client-s3/runtimeConfig.ts	100.00% <0.00%> (ø)
packages/region-provider/src/fromEnv.ts	100.00% <0.00%> (ø)
protocol_tests/aws-ec2/runtimeConfig.ts	100.00% <0.00%> (ø)
protocol_tests/aws-json/runtimeConfig.ts	100.00% <0.00%> (ø)
packages/property-provider/src/memoize.ts	100.00% <0.00%> (ø)
protocol_tests/aws-query/runtimeConfig.ts	100.00% <0.00%> (ø)
protocol_tests/aws-ec2/EC2ProtocolClient.ts	92.30% <0.00%> (ø)
protocol_tests/aws-restxml/runtimeConfig.ts	100.00% <0.00%> (ø)
protocol_tests/aws-restjson/runtimeConfig.ts	100.00% <0.00%> (ø)
... and 11 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9e2614e...e11cef7. Read the comment docs.

[alexforsyth]

LGTM. Minor comments to be addressed

[github-actions]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs and link to relevant comments in this thread.