Browse Source

Add ports configs for SNMP transport to Docker scripts

pull/8757/head
ViacheslavKlimov 3 years ago
parent
commit
1aaa017d92
  1. 4
      common/transport/snmp/src/main/java/org/thingsboard/server/transport/snmp/service/SnmpTransportService.java
  2. 2
      docker/docker-compose.yml
  3. 1
      docker/tb-snmp-transport.env

4
common/transport/snmp/src/main/java/org/thingsboard/server/transport/snmp/service/SnmpTransportService.java

@ -302,8 +302,8 @@ public class SnmpTransportService implements TbTransportService, CommandResponde
*
* TODO: add check for host uniqueness when saving device (for backward compatibility - only for the ones using from-device RPC requests)
*
* TODO: this won't work properly in a cluster mode, due to load-balancing of requests from devices:
* session might not be on this instance
* NOTE: SNMP TRAPs support won't work properly when there is more than one SNMP transport,
* due to load-balancing of requests from devices: session might not be on this instance
* */
@Override
public void processPdu(CommandResponderEvent event) {

2
docker/docker-compose.yml

@ -235,6 +235,8 @@ services:
tb-snmp-transport:
restart: always
image: "${DOCKER_REPO}/${SNMP_TRANSPORT_DOCKER_NAME}:${TB_VERSION}"
ports:
- "1620:1620/udp"
environment:
TB_SERVICE_ID: tb-snmp-transport
JAVA_OPTS: "${JAVA_OPTS}"

1
docker/tb-snmp-transport.env

@ -1,6 +1,7 @@
ZOOKEEPER_ENABLED=true
ZOOKEEPER_URL=zookeeper:2181
SNMP_BIND_PORT=1620
METRICS_ENABLED=true
METRICS_ENDPOINTS_EXPOSE=prometheus
WEB_APPLICATION_ENABLE=true

Loading…
Cancel
Save