Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions .github/workflows/release-dispatch.yml
Original file line number Diff line number Diff line change
Expand Up @@ -58,6 +58,7 @@ jobs:
steps:
- uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2
with:
ref: ${{ inputs.source-repo-ref || github.sha }}
fetch-depth: 0 # ddev needs full tag history

- name: Checkout integrations-core actions
Expand Down
9 changes: 8 additions & 1 deletion ibm_ace/tests/docker/docker-compose.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,8 @@ services:
- ./policy.descriptor:/home/aceuser/initial-config/policy/policy.descriptor
- ./setdbparms.txt:/home/aceuser/initial-config/setdbparms/setdbparms.txt
depends_on:
- ibm-ace-mq
ibm-ace-mq:
condition: service_healthy
ibm-ace-mq:
container_name: ibm-ace-mq
image: ibmcom/mq:${IBM_MQ_VERSION}
Expand All @@ -28,3 +29,9 @@ services:
- LICENSE=accept
ports:
- 11414:1414
healthcheck:
test: ["CMD", "chkmqready"]
interval: 10s
timeout: 5s
retries: 12
start_period: 30s
1 change: 1 addition & 0 deletions snmp/changelog.d/23791.fixed
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
Breaking change: Fixed metric type for `snmp.cppmSystemMemoryFree`, `snmp.cppmSystemMemoryTotal`, `snmp.cppmSystemDiskSpaceFree`, and `snmp.cppmSystemDiskSpaceTotal` in the aruba-clearpass profile. These were previously submitted as rates due to their underlying `Counter64` SNMP type, producing incorrect values. They are now submitted as gauges. Customers with existing dashboards or monitors on these metrics will see a discontinuity between historical rate datapoints and new gauge datapoints; please rebuild any affected widgets or monitors.
Original file line number Diff line number Diff line change
Expand Up @@ -23,12 +23,16 @@ metrics:
symbols:
- name: cppmSystemMemoryTotal
OID: 1.3.6.1.4.1.14823.1.6.1.1.1.1.1.12
metric_type: gauge
- name: cppmSystemMemoryFree
OID: 1.3.6.1.4.1.14823.1.6.1.1.1.1.1.13
metric_type: gauge
- name: cppmSystemDiskSpaceTotal
OID: 1.3.6.1.4.1.14823.1.6.1.1.1.1.1.14
metric_type: gauge
- name: cppmSystemDiskSpaceFree
OID: 1.3.6.1.4.1.14823.1.6.1.1.1.1.1.15
metric_type: gauge
metric_tags:
- symbol:
OID: 1.3.6.1.4.1.14823.1.6.1.1.1.1.1.4
Expand Down
22 changes: 22 additions & 0 deletions snmp/tests/test_e2e_core_profiles/test_profile_aruba_clearpass.py
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@
import pytest

from datadog_checks.dev.utils import get_metadata_metrics
from datadog_checks.snmp.utils import get_profile_definition

from .. import common
from ..test_e2e_core_metadata import assert_device_metadata
Expand All @@ -19,9 +20,30 @@

pytestmark = [pytest.mark.e2e, common.py3_plus_only, common.snmp_integration_only]

CPPM_CAPACITY_METRIC_TYPES = {
'cppmSystemMemoryFree': 'gauge',
'cppmSystemMemoryTotal': 'gauge',
'cppmSystemDiskSpaceFree': 'gauge',
'cppmSystemDiskSpaceTotal': 'gauge',
}


def assert_cppm_capacity_metrics_are_forced_to_gauge(profile):
profile_definition = get_profile_definition({'definition_file': '{}.yaml'.format(profile)})
metric_types = {}
for metric in profile_definition['metrics']:
for symbol in metric.get('symbols', []):
name = symbol['name']
if name in CPPM_CAPACITY_METRIC_TYPES:
metric_types[name] = symbol.get('metric_type')

assert metric_types == CPPM_CAPACITY_METRIC_TYPES


def test_e2e_profile_aruba_clearpass(dd_agent_check):
profile = 'aruba-clearpass'
assert_cppm_capacity_metrics_are_forced_to_gauge(profile)

config = create_e2e_core_test_config(profile)
aggregator = common.dd_agent_check_wrapper(dd_agent_check, config, rate=True)

Expand Down
Loading