-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
Better eureka-based discovery integration #159
Conversation
2. Added EurekaApplicationDiscoveryListener that properly propagates service-configured health and management URLs to Application
@joshiste , please merge. |
The polling in the UI is replaced with server side events. Status changes are now reported immediately from the admin server to the UI.
You better would have done a rebase.... And forced push |
I started digging into the changes. Unfortunatly the upgrade to a current Brixton build snapshot is not a no-brainer and will take me some days. |
That's why I've upgraded to Brixton.M3, not to the latest M5 which introduced major non backward compatible API changes. |
On hold, until Spring Cloud Brixton is released. |
private EmbeddedWebApplicationContext eurekaServer; | ||
|
||
@Before | ||
public void setup() { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
closed by 272633e |
Should address #154