Skip to content

Commit fc7ce84

Browse files
committed
Only include Up Proxies in proxy metrics
1 parent 4c549fe commit fc7ce84

1 file changed

Lines changed: 2 additions & 1 deletion

File tree

src/main/java/eu/openanalytics/containerproxy/stat/impl/Micrometer.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,7 @@
2626
import eu.openanalytics.containerproxy.event.ProxyStopEvent;
2727
import eu.openanalytics.containerproxy.event.UserLoginEvent;
2828
import eu.openanalytics.containerproxy.event.UserLogoutEvent;
29+
import eu.openanalytics.containerproxy.model.runtime.ProxyStatus;
2930
import eu.openanalytics.containerproxy.model.spec.ProxySpec;
3031
import eu.openanalytics.containerproxy.service.ProxyService;
3132
import eu.openanalytics.containerproxy.service.session.ISessionService;
@@ -148,7 +149,7 @@ public void onAuthFailedEvent(AuthFailedEvent event) {
148149
*/
149150
private void updateCachedProxyCount() {
150151
for (ProxySpec spec : proxyService.getProxySpecs(null, true)) {
151-
Integer count = proxyService.getProxies(p -> p.getSpec().getId().equals(spec.getId()), true).size();
152+
Integer count = proxyService.getProxies(p -> p.getSpec().getId().equals(spec.getId()) && p.getStatus() == ProxyStatus.Up, true).size();
152153
proxyCountCache.put(spec.getId(), count);
153154
logger.debug(String.format("Running proxies count for spec %s: %s ", spec.getId(), count));
154155
}

0 commit comments

Comments
 (0)