Merge pull request #5 from guidograzioli/jmx_exporter

Add vars for prometheus_jmx_exporter setup
ansible-middleware · Jun 22, 2022 · b5567f4 · b5567f4
2 parents 7b3d12b + 028da0e
commit b5567f4
Show file tree

Hide file tree

Showing 6 changed files with 62 additions and 5 deletions.
diff --git a/roles/amq_broker/defaults/main.yml b/roles/amq_broker/defaults/main.yml
@@ -99,4 +99,9 @@ amq_broker_logger_jms_level: INFO
 amq_broker_logger_integration_bootstrap_level: INFO
 amq_broker_logger_jetty_level: WARN
 amq_broker_logger_curator_level: WARN
-amq_broker_logger_zookeeper_level: ERROR
+amq_broker_logger_zookeeper_level: ERROR
+
+# Misc
+amq_broker_jmx_exporter_port: 18080
+amq_broker_jmx_exporter_config_path: "{{ amq_broker_dest }}/{{ amq_broker_instance_name }}/etc/jmx_exporter.yml"
+amq_broker_jmx_exporter_enabled: False
diff --git a/roles/amq_broker/meta/argument_specs.yml b/roles/amq_broker/meta/argument_specs.yml
@@ -362,4 +362,16 @@ argument_specs:
             amq_broker_logger_zookeeper_level:
                 description: "Logging level for org.apache.zookeeper"
                 default: 'ERROR'
-                type: "str"
+                type: "str"
+            amq_broker_jmx_exporter_port:
+                default: 18080
+                type: "int"
+                description: "Port for prometheus JMX exporter to listen"
+            amq_broker_jmx_exporter_config_path:
+                default: "{{ amq_broker_dest }}/{{ amq_broker_instance_name }}/etc/jmx_exporter.yml"
+                type: "str"
+                description: "JMX exporter configuration path"
+            amq_broker_jmx_exporter_enabled:
+                description: "Enable install and configuration of prometheus-jmx-exporter"
+                default: False
+                type: "bool"
diff --git a/roles/amq_broker/tasks/firewalld.yml b/roles/amq_broker/tasks/firewalld.yml
@@ -26,3 +26,12 @@
     - "{{ amq_broker_port_amqp }}/tcp"
     - "{{ amq_broker_port_mqtt }}/tcp"
     - "{{ amq_broker_port_stomp }}/tcp"
+
+- name: "Configure firewall for JMX exporter port"
+  become: yes
+  firewalld:
+    port: "{{ amq_broker_jmx_exporter_port }}/tcp"
+    permanent: true
+    state: enabled
+    immediate: yes
+  when: amq_broker_jmx_exporter_enabled
diff --git a/roles/amq_broker/tasks/prereqs.yml b/roles/amq_broker/tasks/prereqs.yml
@@ -40,4 +40,10 @@
       - unzip
       - procps-ng
       - initscripts
-      - libaio
+      - libaio
+
+- name: Ensure required packages are installed
+  ansible.builtin.dnf:
+    name: prometheus-jmx-exporter
+    state: present
+  when: amq_broker_jmx_exporter_enabled
diff --git a/roles/amq_broker/templates/amq_broker.sysconfig.j2 b/roles/amq_broker/templates/amq_broker.sysconfig.j2
@@ -1,3 +1,7 @@
 # {{ ansible_managed }}
-JAVA_OPTS='{{ amq_broker_java_opts }}'
-JAVA_HOME={{ amq_broker_java_home  | default(amq_broker_rpm_java_home, true) }}
+{% if amq_broker_jmx_exporter_enabled  %}
+JAVA_ARGS='{{ amq_broker_java_opts }} -javaagent:/usr/share/java/prometheus-jmx-exporter/jmx_prometheus_javaagent.jar=0.0.0.0:{{ amq_broker_jmx_exporter_port }}:{{ amq_broker_jmx_exporter_config_path }}'
+{% else %}
+JAVA_ARGS='{{ amq_broker_java_opts }}'
+{% endif %}
+JAVA_HOME={{ amq_broker_java_home  | default(amq_broker_rpm_java_home, true) }}
diff --git a/roles/amq_broker/templates/jmx_exporter.yml.j2 b/roles/amq_broker/templates/jmx_exporter.yml.j2
@@ -0,0 +1,21 @@
+---
+lowercaseOutputName: true
+lowercaseOutputLabelNames: true
+rules:
+  - pattern: "^org.apache.activemq.artemis<broker=\"([^\"]*)\"><>([^:]*):\\s(.*)"
+    attrNameSnakeCase: true
+    name: artemis_$2
+    type: COUNTER
+  - pattern: "^org.apache.activemq.artemis<broker=\"([^\"]*)\",\\s*component=addresses,\\s*address=\"([^\"]*)\"><>([^:]*):\\s(.*)"
+    attrNameSnakeCase: true
+    name: artemis_$3
+    type: COUNTER
+    labels:
+        address: $2
+  - pattern: "^org.apache.activemq.artemis<broker=\"([^\"]*)\",\\s*component=addresses,\\s*address=\"([^\"]*)\",\\s*subcomponent=(queue|topic)s,\\s*routing-type=\"([^\"]*)\",\\s*(queue|topic)=\"([^\"]*)\"><>([^: ]*):\\s(.*)"
+    attrNameSnakeCase: true
+    name: artemis_$7
+    type: COUNTER
+    labels:
+        address: $2
+        "$5": $6