Skip to content

Refresh IMDS credentials more aggressively. #3147

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

Merged
merged 2 commits into from
Apr 5, 2022
Merged

Conversation

millems
Copy link
Contributor

@millems millems commented Apr 4, 2022

Intended to address #3142. It hasn't been verified to work, but it should address any issues until we can figure out why the previous strategy did not work.

@millems millems requested a review from a team as a code owner April 4, 2022 20:11
@millems millems force-pushed the millem/hosm-aggression branch 2 times, most recently from 51ffc7b to 8b64283 Compare April 4, 2022 20:22
@millems millems force-pushed the millem/hosm-aggression branch from 8b64283 to 6c54cd1 Compare April 4, 2022 20:30
@millems millems changed the title Refresh IMDS credentials more aggressively when they are within 15 minutes of expiration. Refresh IMDS credentials more aggressively. Apr 4, 2022
@millems millems force-pushed the millem/hosm-aggression branch from 6c54cd1 to ff9bb7c Compare April 4, 2022 20:50
This improves the behavior introduced in #2989 if the customer is using a mock IMDS endpoint with aggressively short session durations.
@millems millems force-pushed the millem/hosm-aggression branch from ff9bb7c to f5891ed Compare April 5, 2022 00:41
@sonarqubecloud
Copy link

sonarqubecloud bot commented Apr 5, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

81.0% 81.0% Coverage
0.0% 0.0% Duplication

@millems millems merged commit f369c57 into master Apr 5, 2022
@millems millems deleted the millem/hosm-aggression branch April 5, 2022 17:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants