Skip to content
This repository was archived by the owner on Jul 31, 2025. It is now read-only.

aws/ec2metadata: Reduces timeout and number of max retries for EC2Metadata client#3066

Merged
skotambkar merged 1 commit intoaws:masterfrom
skotambkar:rev/IMDS
Jan 7, 2020
Merged

aws/ec2metadata: Reduces timeout and number of max retries for EC2Metadata client#3066
skotambkar merged 1 commit intoaws:masterfrom
skotambkar:rev/IMDS

Conversation

@skotambkar
Copy link
Copy Markdown
Contributor

The PR address the issues related to EC2Metadata client having long timeouts in case of failure in obtaining EC2Metadata token, while making a request to IMDS.

The PR reduces the timeout to 1 sec, and number of max retries to 2 for the EC2Metadata client. This would help reduce the long timeouts faced by the customers.

Fixes #2972

@skotambkar skotambkar marked this pull request as ready for review January 6, 2020 22:40
@skotambkar skotambkar self-assigned this Jan 6, 2020
@skotambkar skotambkar added pr/work-in-progress This PR is a draft and needs further work. needs-review This issue or pull request needs review from a core team member. and removed pr/work-in-progress This PR is a draft and needs further work. labels Jan 6, 2020
@skotambkar skotambkar requested a review from jasdel January 7, 2020 16:54
@skotambkar skotambkar changed the title aws/ec2metadata`: Reduces timeout and number of max retries for EC2Metadata client aws/ec2metadata: Reduces timeout and number of max retries for EC2Metadata client Jan 7, 2020
@skotambkar skotambkar merged commit 0c96ad5 into aws:master Jan 7, 2020
@skotambkar skotambkar deleted the rev/IMDS branch January 7, 2020 18:11
aws-sdk-go-automation pushed a commit that referenced this pull request Jan 7, 2020
===

### Service Client Updates
* `service/AWSMigrationHub`: Updates service API, documentation, and paginators
* `service/codebuild`: Updates service API and documentation
  * Add encryption key override to StartBuild API in AWS CodeBuild.
* `service/xray`: Updates service documentation
  * Documentation updates for xray

### SDK Enhancements
* `aws`: Add configuration option enable the SDK to unmarshal API response header maps to normalized lower case map keys. ([#3033](#3033))
  * Setting `aws.Config.LowerCaseHeaderMaps` to `true` will result in S3's X-Amz-Meta prefixed header to be unmarshaled to lower case Metadata member's map keys.

### SDK Bugs
* `aws/ec2metadata` : Reduces request timeout for EC2Metadata client along with maximum number of retries ([#3066](#3066))
  * Reduces latency while fetching response from EC2Metadata client running in a container to around 3 seconds
  * Fixes [#2972](#2972)
aws-sdk-go-automation added a commit that referenced this pull request Jan 7, 2020
Release v1.27.2 (2020-01-07)
===

### Service Client Updates
* `service/AWSMigrationHub`: Updates service API, documentation, and paginators
* `service/codebuild`: Updates service API and documentation
  * Add encryption key override to StartBuild API in AWS CodeBuild.
* `service/xray`: Updates service documentation
  * Documentation updates for xray

### SDK Enhancements
* `aws`: Add configuration option enable the SDK to unmarshal API response header maps to normalized lower case map keys. ([#3033](#3033))
  * Setting `aws.Config.LowerCaseHeaderMaps` to `true` will result in S3's X-Amz-Meta prefixed header to be unmarshaled to lower case Metadata member's map keys.

### SDK Bugs
* `aws/ec2metadata` : Reduces request timeout for EC2Metadata client along with maximum number of retries ([#3066](#3066))
  * Reduces latency while fetching response from EC2Metadata client running in a container to around 3 seconds
  * Fixes [#2972](#2972)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

needs-review This issue or pull request needs review from a core team member.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Get credentials from role has become slow

2 participants