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

reduce number of getPasswordData calls #471

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

Conversation

obstiger
Copy link

If the AMI was not setup to generate passwords, AWS does not generate passwords. In such cases, ec2 plugin keeps making EC2 api attempting to get the password. This exhausts the allowed EC2 api calls from aws side and AWS starts throttling the api calls, especially if multiple nodes are spun up at the same time.

This PR attempts to address this issue by reducing the frequency of GetPasswordData calls using exponential back off with full jitter.

Here is an example of the log messages printed out:
image

@res0nance
Copy link
Contributor

This seems to come with the problem that it would increase the time it takes for the node to connect by a large amount

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.

2 participants