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

Instance monitor init() giving response code other then 200 without response message #117

Open
JagmohanSharma opened this issue Jan 19, 2017 · 6 comments

Comments

@JagmohanSharma
Copy link

JagmohanSharma commented Jan 19, 2017

@ryanjbaxter and @spencergibb Reference to another misplaced issue
spring-cloud/spring-cloud-netflix#1631

which is closed to have discussion on the right place as in turbine-core.

@JagmohanSharma
Copy link
Author

@JagmohanSharma
Copy link
Author

Can somebody help in this issue? it bit urgent to get it resolved for us. Or tell me if this contains wrong information or at wrong place to be asked?

@JagmohanSharma
Copy link
Author

Please let me know if it not allowed to post issues here?

@spencergibb
Copy link

Turbine isn't maintained much anymore.

@JagmohanSharma
Copy link
Author

JagmohanSharma commented Jan 27, 2017

I got one more issue here , as while reading stream through /turbine.stream endpoint for any cluster , if user close this through browser , it throws ClientAbortException with message as "java.io.IOException: Broken pipe". Now as in TurbineStreamingConnection as we have catch block to handle this and we are checking as

                                   catch (Exception e) {
                                      if ("Broken pipe".equals(e.getMessage())) { 
                                    // use debug instead of error as this is expected to happen every time a client            
                                   disconnects
                               logger.debug("Broken pipe (most likely client disconnected) when writing to                   
                                     response  stream", e);
                                    } else {
                                   logger.error("Got exception when writing to response stream", e);
                               }
                                 stopMonitoring = true;
                             }

Here shouldn't we have checked for contains instead of equals for exception message "broken pipe"
as (e.getMessage()).contains("Broken pipe") so it will be true for above mentioned message as "java.io.IOException: Broken pipe". This should be addressed in turbine as in our application we have email notification for all error log message and in this situation this is causing unrelevant email notification by wrong condition check.

Let me know if I have mis interpreted this.

Thanks.

@JagmohanSharma
Copy link
Author

JagmohanSharma commented Jan 27, 2017

if this project is not being maintained then how would issues with this be resolved? As many would be using turbine application to have an aggregated view of hystrix circuit breaker dashboard.

Or is there any way to get the same functionality without using Turbine anymore?

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

No branches or pull requests

2 participants