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

Enable Hystrix Dashboard and Turbine #17

Closed
mszarlinski opened this issue Nov 22, 2016 · 4 comments
Closed

Enable Hystrix Dashboard and Turbine #17

mszarlinski opened this issue Nov 22, 2016 · 4 comments
Assignees
@mszarlinski
Labels
enhancement

Comments

@mszarlinski
Copy link
Collaborator

We could configure Hystrix Dashboard and Turbine as described here https://github.com/spring-cloud-samples/hystrix-dashboard
@mszarlinski mszarlinski self-assigned this Nov 22, 2016
@mszarlinski
Copy link
Collaborator Author

I can handle it

@mszarlinski
Copy link
Collaborator Author

mszarlinski commented Dec 5, 2016 •
edited
Loading

Apparently, we could not use vanilla nor javanica Hystrix command together with Zuul: spring-cloud/spring-cloud-netflix#250
I am working on solution that benefits from "Zuul fallbacks" as described here:
http://ryanjbaxter.com/cloud/spring%20cloud/spring/2016/10/27/hystrix-fallbacks-with-zuul.html

I am only wondering if it wouldn't be too "magical".

@mszarlinski mszarlinski mentioned this issue Jan 14, 2017
Closed
arey added a commit to arey/spring-petclinic-microservices that referenced this issue Nov 18, 2018
@arey
spring-petclinic#17 Add a Hystrix Dashboard application
81eb122
@arey arey mentioned this issue Nov 18, 2018
Merged
mszarlinski pushed a commit that referenced this issue Nov 18, 2018
@arey @mszarlinski
Add Hystrix Dashboard (#115)
ece0dc1 
* Add circuit breaker to the getVisitsForPets method

* Use Collections.emptyMap()

* Fallback method with private visibility

* Unit test Hystrix fallback method

* #17 Add a Hystrix Dashboard application

* Fix hystrix-dashboard container name
@arey
Copy link
Member

arey commented Nov 19, 2018

@mszarlinski a part of this issue has been implemented by the PR #17
But what about Turbine?

@arey
Copy link
Member

arey commented Dec 17, 2018

Bad news, we still have to replace them: #117

@arey arey closed this as completed Mar 20, 2019
talha-01 pushed a commit to talha-01/microservices-ci-cd-pipeline-with-petclinic-app that referenced this issue Nov 13, 2020
@arey @mszarlinski
Add Hystrix Dashboard (spring-petclinic#115)
e957e33 
* Add circuit breaker to the getVisitsForPets method

* Use Collections.emptyMap()

* Fallback method with private visibility

* Unit test Hystrix fallback method

* spring-petclinic#17 Add a Hystrix Dashboard application

* Fix hystrix-dashboard container name
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mszarlinski mszarlinski

Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@arey @mszarlinski