mirror of
				https://github.com/KevinMidboe/mktxp-no-cli.git
				synced 2025-10-29 17:50:23 +00:00 
			
		
		
		
	Add mac_address label to all wlan/capsman metrics
				
					
				
			`dhcp_name` is not sufficient to identify wireless clients. DHCP Host
Name Option (Code 12) is not required to be unique. As a result, it's
possible to have multiple metrics with exactly same labels. Adding
`mac_address` label to all wlan/capsman metrics allows to identify
the clients.
```
mktxp_wlan_clients_signal_strength{dhcp_name="xyz",routerboard_address="a.b.c.d",routerboard_name="foobar"} -63.0
mktxp_wlan_clients_signal_strength{dhcp_name="xyz",routerboard_address="a.b.c.d",routerboard_name="foobar"} -64.0
mktxp_wlan_clients_signal_strength{dhcp_name="xyz",routerboard_address="a.b.c.d",routerboard_name="foobar"} -65.0
```
vs.
```
mktxp_wlan_clients_signal_strength{dhcp_name="xyz",mac_address="11:22:33:44:55:AA",routerboard_address="a.b.c.d",routerboard_name="foobar"} -63.0
mktxp_wlan_clients_signal_strength{dhcp_name="xyz",mac_address="11:22:33:44:55:BB",routerboard_address="a.b.c.d",routerboard_name="foobar"} -64.0
mktxp_wlan_clients_signal_strength{dhcp_name="xyz",mac_address="11:22:33:44:55:CC",routerboard_address="a.b.c.d",routerboard_name="foobar"} -65.0
```
			
			
This commit is contained in:
		| @@ -55,13 +55,13 @@ class CapsmanCollector(BaseCollector): | ||||
|                 for registration_record in registration_records: | ||||
|                     BaseOutputProcessor.augment_record(router_entry, registration_record) | ||||
|                      | ||||
|                 tx_byte_metrics = BaseCollector.counter_collector('capsman_clients_tx_bytes', 'Number of sent packet bytes', registration_records, 'tx_bytes', ['dhcp_name']) | ||||
|                 tx_byte_metrics = BaseCollector.counter_collector('capsman_clients_tx_bytes', 'Number of sent packet bytes', registration_records, 'tx_bytes', ['dhcp_name', 'mac_address']) | ||||
|                 yield tx_byte_metrics | ||||
|  | ||||
|                 rx_byte_metrics = BaseCollector.counter_collector('capsman_clients_rx_bytes', 'Number of received packet bytes', registration_records, 'rx_bytes', ['dhcp_name']) | ||||
|                 rx_byte_metrics = BaseCollector.counter_collector('capsman_clients_rx_bytes', 'Number of received packet bytes', registration_records, 'rx_bytes', ['dhcp_name', 'mac_address']) | ||||
|                 yield rx_byte_metrics | ||||
|  | ||||
|                 signal_strength_metrics = BaseCollector.gauge_collector('capsman_clients_signal_strength', 'Client devices signal strength', registration_records, 'rx_signal', ['dhcp_name']) | ||||
|                 signal_strength_metrics = BaseCollector.gauge_collector('capsman_clients_signal_strength', 'Client devices signal strength', registration_records, 'rx_signal', ['dhcp_name', 'mac_address']) | ||||
|                 yield signal_strength_metrics | ||||
|  | ||||
|                 registration_metrics = BaseCollector.info_collector('capsman_clients_devices', 'Registered client devices info',  | ||||
|   | ||||
| @@ -54,19 +54,19 @@ class WLANCollector(BaseCollector): | ||||
|                 for registration_record in registration_records: | ||||
|                     BaseOutputProcessor.augment_record(router_entry, registration_record) | ||||
|  | ||||
|                 tx_byte_metrics = BaseCollector.counter_collector('wlan_clients_tx_bytes', 'Number of sent packet bytes', registration_records, 'tx_bytes', ['dhcp_name']) | ||||
|                 tx_byte_metrics = BaseCollector.counter_collector('wlan_clients_tx_bytes', 'Number of sent packet bytes', registration_records, 'tx_bytes', ['dhcp_name', 'mac_address']) | ||||
|                 yield tx_byte_metrics | ||||
|  | ||||
|                 rx_byte_metrics = BaseCollector.counter_collector('wlan_clients_rx_bytes', 'Number of received packet bytes', registration_records, 'rx_bytes', ['dhcp_name']) | ||||
|                 rx_byte_metrics = BaseCollector.counter_collector('wlan_clients_rx_bytes', 'Number of received packet bytes', registration_records, 'rx_bytes', ['dhcp_name', 'mac_address']) | ||||
|                 yield rx_byte_metrics | ||||
|  | ||||
|                 signal_strength_metrics = BaseCollector.gauge_collector('wlan_clients_signal_strength', 'Average strength of the client signal recevied by AP', registration_records, 'signal_strength', ['dhcp_name']) | ||||
|                 signal_strength_metrics = BaseCollector.gauge_collector('wlan_clients_signal_strength', 'Average strength of the client signal recevied by AP', registration_records, 'signal_strength', ['dhcp_name', 'mac_address']) | ||||
|                 yield signal_strength_metrics | ||||
|  | ||||
|                 signal_to_noise_metrics = BaseCollector.gauge_collector('wlan_clients_signal_to_noise', 'Client devices signal to noise ratio', registration_records, 'signal_to_noise', ['dhcp_name']) | ||||
|                 signal_to_noise_metrics = BaseCollector.gauge_collector('wlan_clients_signal_to_noise', 'Client devices signal to noise ratio', registration_records, 'signal_to_noise', ['dhcp_name', 'mac_address']) | ||||
|                 yield signal_to_noise_metrics | ||||
|  | ||||
|                 tx_ccq_metrics = BaseCollector.gauge_collector('wlan_clients_tx_ccq', 'Client Connection Quality (CCQ) for transmit', registration_records, 'tx_ccq', ['dhcp_name']) | ||||
|                 tx_ccq_metrics = BaseCollector.gauge_collector('wlan_clients_tx_ccq', 'Client Connection Quality (CCQ) for transmit', registration_records, 'tx_ccq', ['dhcp_name', 'mac_address']) | ||||
|                 yield tx_ccq_metrics | ||||
|  | ||||
|                 registration_metrics = BaseCollector.info_collector('wlan_clients_devices', 'Client devices info',  | ||||
|   | ||||
		Reference in New Issue
	
	Block a user