[
https://issues.apache.org/jira/browse/AMQ-9447?focusedWorklogId=914144&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-914144
]
ASF GitHub Bot logged work on AMQ-9447:
---------------------------------------
Author: ASF GitHub Bot
Created on: 11/Apr/24 12:01
Start Date: 11/Apr/24 12:01
Worklog Time Spent: 10m
Work Description: jbonofre commented on code in PR #1194:
URL: https://github.com/apache/activemq/pull/1194#discussion_r1560916109
##########
activemq-broker/src/main/java/org/apache/activemq/broker/jmx/HealthView.java:
##########
@@ -167,6 +169,25 @@ public List<HealthStatus> healthList() throws Exception {
}
}
+ /**
+ * Check The Transport Connector limits
+ */
+
+ if (brokerService != null &&
!brokerService.getTransportConnectors().isEmpty()) {
+ for(TransportConnector tc:
brokerService.getTransportConnectors()) {
+ if(tc.getServer() instanceof TcpTransportServer) {
+ int connectionUsage = (int)
(((TcpTransportServer)tc.getServer()).getCurrentTransportCount().get() * 100 )
/ ((TcpTransportServer)tc.getServer()).getMaximumConnections();
+
if(((TcpTransportServer)tc.getServer()).getCurrentTransportCount().get() >=
((TcpTransportServer)tc.getServer()).getMaximumConnections()) {
+ String message = "Exceeded the maximum
number of allowed client connections: " +
((TcpTransportServer)tc.getServer()).getMaximumConnections();
Review Comment:
Agree, just the limit is enough.
We can relay on the existing metrics for that.
Issue Time Tracking
-------------------
Worklog Id: (was: 914144)
Time Spent: 2h (was: 1h 50m)
> ActiveMQ Jolokia health service returns "good" even when persistence or
> transports are failing
> ----------------------------------------------------------------------------------------------
>
> Key: AMQ-9447
> URL: https://issues.apache.org/jira/browse/AMQ-9447
> Project: ActiveMQ Classic
> Issue Type: Improvement
> Components: Broker, JMX
> Affects Versions: 5.18.3
> Reporter: Ephemeris Lappis
> Assignee: Jean-Baptiste Onofré
> Priority: Major
> Fix For: 6.2.0, 5.18.5
>
> Time Spent: 2h
> Remaining Estimate: 0h
>
> The Jolokia health service only considers the BrokerService status, it
> doesn't check all layers (like persistent store or transports).
> The CurrentStatus should not be "Good" if some underlying service is failing.
> For example, with a JDBC persistence adapter, if the DB is not reachable, the
> broker should not be considered as OK.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
