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.

Sign up for GitHub

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

Jump to bottom

meta10? #47

Closed
siculars opened this issue Sep 9, 2017 · 4 comments
Closed

meta10? #47

siculars opened this issue Sep 9, 2017 · 4 comments

Comments

@siculars
Copy link

siculars commented Sep 9, 2017 •
edited
Loading

ndbench/ndbench-core/src/main/java/com/netflix/ndbench/core/discovery/AWSLocalClusterDiscovery.java

Line 32 in 1bf0af6

String urlLocal = "http://169.254.169.254/latest/meta10-data/local-hostname";

Are you sure that's correct?

ubuntu@ip-172-31-28-231:~$ curl http://169.254.169.254/latest/meta10-data/local-hostname
> <?xml version="1.0" encoding="iso-8859-1"?>
> <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN"
>          "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
> <html xmlns="http://www.w3.org/1999/xhtml" xml:lang="en" lang="en">
>  <head>
>   <title>404 - Not Found</title>
>  </head>
>  <body>
>   <h1>404 - Not Found</h1>
>  </body>
> </html>


ubuntu@ip-172-31-28-231:~$ curl http://169.254.169.254/latest/meta-data/local-hostname

ip-172-31-28-231.us-west-2.compute.internal
@ipapapa
Copy link
Contributor

ipapapa commented Sep 29, 2017

I guess @diegopacheco would be the appropriate person to answer this.

@diegopacheco
Copy link
Contributor

@ipapapa

There are mode documentation here. This is the AWS Metadata API http://docs.aws.amazon.com/AWSEC2/latest/UserGuide/ec2-instance-metadata.html

Looks like there is typo there - we need to remove the 10 from the URL.

Nice catch @siculars

Cheers,
Diego Pacheco

@ipapapa
Copy link
Contributor

ipapapa commented Sep 29, 2017

Do you guys want to do a PR?

@ipapapa ipapapa mentioned this issue Sep 30, 2017
Merged
@ipapapa
Copy link
Contributor

ipapapa commented Sep 30, 2017

Fixed

@ipapapa ipapapa closed this as completed Sep 30, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@siculars @diegopacheco @ipapapa