Skip to content
  1. Jul 08, 2021
  2. Jul 07, 2021
  3. Jul 06, 2021
    • Martin Weinelt's avatar
      df446adc
    • Martin Weinelt's avatar
      prometheus-node-exporter-lua: lower case bssid label value in wifi exporter · 445d8927
      Martin Weinelt authored
      
      
      To allow cross matching bssids between different exporters we need to
      use the same case, as label matching is case senstive.
      
      Signed-off-by: default avatarMartin Weinelt <hexa@darmstadt.ccc.de>
      445d8927
    • Martin Weinelt's avatar
      prometheus-node-exporter-lua: fix corner case in hostapd_stations · a1b145ab
      Martin Weinelt authored
      
      
      There was a corner case, when a vif had no stations, that
      evaluate_metrics for a station that was nil and had no collected metrics
      would have been called.
      
      Comment the code, to make it easier to understand and follow, and
      simplify some variable names along the way.
      
      Signed-off-by: default avatarMartin Weinelt <hexa@darmstadt.ccc.de>
      a1b145ab
    • Martin Weinelt's avatar
      prometheus-node-exporter-lua: normalize metric names in hostapd_stations · a9859616
      Martin Weinelt authored
      Normalizes metrics according to the Prometheus upstream metric
      guidelines available at https://prometheus.io/docs/practices/naming/
      
      .
      
      Drops the `hostapd_station_wpa` metric, because it is misleading, as it
      is not a differentiator between WPA versions, like one could be led to
      assume.
      
      Exposes more flags in a more consistent manner. Their metric was
      previously only exposed if the flag was present, but not if it wasn't.
      The same applies issue was fixed with regards to vht caps.
      
      After this commit the following breaking changes are in place:
      
      - All flags have been moved below `hostapd_station_flags_$flagname`:
        - `hostapd_station_ht` is now `hostapd_station_flag_ht`
        - `hostapd_station_mfp` is now `hostapd_station_flag_mfp`
        - `hostapd_station_vht` is now `hostapd_station_flag_vht`
        - `hostapd_station_wmm` is now `hostapd_station_flag_wmm`
      
      - New flags have been exposed:
        - `hostapd_station_flag_he` for high-efficency connections
        - `hostapd_station_flag_short_preamble` for short preamble connections
        - `hostapd_station_flag_auth` for authentication state
        - `hostapd_station_flag_assoc` for association state
      
      - Some metrics have had their unit normalized to the SI base unit or
        embedded into the metrics name:
        - `hostapd_station_inactive_msec` is now
          `hostapd_station_inactive_seconds`, the value is still float64 and
          as such has enough precision anyway, but becomes easier to reason
          about
        - `hostapd_station_connected_time` has been renamed to
          `hostapd_station_connected_seconds_total` so the unit, as well as
          the nature of the counter is reflected
        - `hostapd_station_signal` now includes its unit and is therefore
          named `hostapd_station_signal_dbm`
      
      - The packet counter metrics have been normalized to what the node
        exporter uses, so it is more in line with the defaults in the
        Prometheus ecosystem:
        - `hostapd_station_rx_packets` is now
          `hostapd_station_receive_packets_total`
        - `hostapd_station_rx_bytes` is now
          `hostapd_station_receive_bytes_total`
        - `hostapd_station_tx_packets` is now
          `hostapd_station_transmit_packets_total`
        - `hostapd_station_tx_bytes` is now
          `hostapd_station_transmit_bytes_total`
      
      Signed-off-by: default avatarMartin Weinelt <hexa@darmstadt.ccc.de>
      a9859616