These are chat archives for spring-cloud/spring-cloud

19th
Nov 2015
Tommy Ludwig
@shakuzen
Nov 19 2015 05:13
Quick question on @RefreshScope and the /refresh endpoint. I tried confirming in the documentation, but it isn't quite clear after reading it whether Spring Boot Actuator is required to be able to use the /refresh endpoint. Is it required?
Spencer Gibb
@spencergibb
Nov 19 2015 06:33
@shakuzen yes it is
Tommy Ludwig
@shakuzen
Nov 19 2015 07:00
@spencergibb Thanks for the confirmation.
hacbq
@hacbq
Nov 19 2015 08:26
So /admin/pages/foo doesn't work in the first case, but /bla/pages/foo does in the second case?
@dsyer yes :smile: . I must restart zuul then first case works
Dave Syer
@dsyer
Nov 19 2015 08:37
Ah, so it's only a problem with the refresh?
I guess it's a question of the order
the changes are applied
Did you open an issue in github already?
hacbq
@hacbq
Nov 19 2015 08:47
Yes, I only try with the refresh. I created it :
spring-cloud/spring-cloud-config#274
Thank you for reading :)
gorelikov
@gorelikov
Nov 19 2015 09:20
@dsyer Could you help me with my question? It was few records above. I have no thoughts how i can reproduce it and don't understand which line in HystrixHealthIndicator could throw the exception.
Dave Syer
@dsyer
Nov 19 2015 09:20
Me neither
If you can't reproduce it, it's going to be hard
gorelikov
@gorelikov
Nov 19 2015 09:22
Too bad, it causes some problems for me. But thanks for answer.
Dave Syer
@dsyer
Nov 19 2015 09:22
In what circumstances do you see it?
gorelikov
@gorelikov
Nov 19 2015 09:23
I saw it when one of circuit breaker was open. But don't know exactly which circuit breaker it was.
I have not found any logs for healthindicator
And i use both @HystrixCommand annotation and HystrixCommand extensions
Dave Syer
@dsyer
Nov 19 2015 09:30
There's a null check in 1.1 that wasn't in 1.0 as far as I can see (spring-cloud/spring-cloud-netflix@5d3a0da)
Maybe you fell foul of that?
gorelikov
@gorelikov
Nov 19 2015 09:32
I think so. Thank you! I will update my dependecies.
Dave Syer
@dsyer
Nov 19 2015 09:33
1.1 isn't released yet so it's not trivial
We can cherry pick that change onto 1.0.x if it helps.
Since you can't reproduce the problem it's going to be hard to test
gorelikov
@gorelikov
Nov 19 2015 09:39
I've already tried all variants but i can't reproduce it. If this commit will be cherry picked it will be great.
Dave Syer
@dsyer
Nov 19 2015 09:40
Can you open an issue in github, please?
gorelikov
@gorelikov
Nov 19 2015 09:40
yes, sure
gorelikov
@gorelikov
Nov 19 2015 09:49
Created - spring-cloud/spring-cloud-netflix#653. Thanks again.
Tommy Ludwig
@shakuzen
Nov 19 2015 16:10
Looks like Brixton.M3 is available now. But this page still shows M2 as the latest milestone.
Spencer Gibb
@spencergibb
Nov 19 2015 16:12
@shakuzen yeah, we haven’t announced it yet.
Tommy Ludwig
@shakuzen
Nov 19 2015 16:13
Oh, alright. No problem then.
Dave Syer
@dsyer
Nov 19 2015 16:30
I just updated that page (caches might still be stale)
Josh and I both did demos yesterday with start.spring.io using Brixton.M3
So we think it works
Tommy Ludwig
@shakuzen
Nov 19 2015 16:33
I can see it updated now. Great to hear. I'll be using it right away at work tomorrow morning.