Skip to content

Conversation

@vladimirdolzhenko
Copy link
Contributor

@vladimirdolzhenko vladimirdolzhenko commented Nov 5, 2018

Discovery EC2 uses hard coded url for EC2 metadata server http://169.254.169.254/. It makes some difficulties on testing like #35141

With this PR discovery-ec2 uses Amazon SDK to resolve metadata server, improves unit and integration testing and aligns with approach used in repositoty-s3.

Closes #35141

@vladimirdolzhenko vladimirdolzhenko added >test Issues or PRs that are addressing/adding tests :Distributed Coordination/Discovery-Plugins Anything related to our integration plugins with EC2, GCP and Azure labels Nov 5, 2018
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-distributed

Copy link
Member

@tlrx tlrx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like it, thanks!

@vladimirdolzhenko vladimirdolzhenko merged commit 9d28a10 into elastic:master Nov 5, 2018
@vladimirdolzhenko vladimirdolzhenko deleted the fix/35141 branch November 5, 2018 14:02
@vladimirdolzhenko
Copy link
Contributor Author

thanks @tlrx for review

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Distributed Coordination/Discovery-Plugins Anything related to our integration plugins with EC2, GCP and Azure >test Issues or PRs that are addressing/adding tests v6.6.0 v7.0.0-beta1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Ec2NetworkTests don't assert much

4 participants