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

New Feature -- Eureka Integration to Hystrix Dashbaord #1197

Merged
merged 6 commits into from
Jul 13, 2016

Conversation

diegopacheco
Copy link

Hi Guys,

For sake of commodity I load all eureka apps into a select and them the use can just click on the app and it generate the URL for hystryx.stream or turbine.stream. The rest of the dashboard works as usual and the user still can type if he wants.

cheers,
Diego Pacheco

@mattrjacobs
Copy link
Contributor

Thanks for the contribution, @diegopacheco.

I tried this out, and it didn't work for me. I input the domain name of our Eureka VIP, and then clicked on the "Eureka Application" dropdown. The only element in the list was a greyed-out "Choose Here". In the browser devtools, I see "net::ERR_CONTENT_DECODING_FAILED" for GET http://localhost:8080/eureka?url=

Any ideas on what's going wrong on my local installation?

@diegopacheco
Copy link
Author

hi @mattrjacobs

For me is working. Which URL are you putting in eureka? Needs to be something that can be queried via REST. Take a look on my print screen. (I'm running on Tomcat) Could you share more details?

hystrix-dashboard-pr

Cheers,
Diego Pacheco

@mattrjacobs
Copy link
Contributor

I just retested this and it worked for me in both Firefox and Chrome. I have no idea why it didn't work before. Thanks again for the PR, @diegopacheco , and sincere apologies for the delay

@mattrjacobs mattrjacobs merged commit 8e6830c into Netflix:master Jul 13, 2016
@diegopacheco
Copy link
Author

Thanks @mattrjacobs

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