Github user tbouron commented on a diff in the pull request:
https://github.com/apache/brooklyn-server/pull/829#discussion_r141829990
--- Diff:
policy/src/main/java/org/apache/brooklyn/policy/ha/ServiceRestarter.java ---
@@ -133,6 +135,10 @@ protected synchronized void
onDetectedFailure(SensorEvent<Object> event) {
LOG.warn("ServiceRestarter suspended, so not acting on failure
detected at "+entity+" ("+event.getValue()+")");
return;
}
+ if (isEntityStopping()) {
+ highlightViolation("Failure detected but entity stopping");
+ LOG.info("Entity stopping, so ServiceRestarter not acting on
failure detected at "+entity+" ("+event.getValue()+")");
--- End diff --
You are missing a `return;`
---
