Skip to content

Commit 2f00627

Browse files
authored
Add role_name tag key to machine metrics (#19613)
* Add role_name tag key * Add changelog
1 parent 4ee4bcb commit 2f00627

File tree

3 files changed

+18
-17
lines changed

3 files changed

+18
-17
lines changed
Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
Add `role_name` tag key to machine metrics.

octopus_deploy/datadog_checks/octopus_deploy/check.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -474,7 +474,7 @@ def _collect_machine_metrics(self, space_id):
474474
f"health_status:{health_status}",
475475
f"operating_system:{os}",
476476
]
477-
machine_tags += roles
477+
machine_tags += [f"role_name:{role}" for role in roles]
478478
environment_ids = machine.get("EnvironmentIds")
479479
env_tags = []
480480
for env_id in environment_ids:

octopus_deploy/tests/test_unit.py

Lines changed: 16 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -2553,7 +2553,7 @@ def test_machines_metrics(
25532553
"machine_slug:test-machine",
25542554
"health_status:Healthy",
25552555
"operating_system:Ubuntu 24.04.1 LTS",
2556-
"test-tag",
2556+
"role_name:test-tag",
25572557
],
25582558
)
25592559
aggregator.assert_metric(
@@ -2567,7 +2567,7 @@ def test_machines_metrics(
25672567
"machine_slug:test-machine",
25682568
"health_status:Healthy",
25692569
"operating_system:Ubuntu 24.04.1 LTS",
2570-
"test-tag",
2570+
"role_name:test-tag",
25712571
],
25722572
)
25732573
aggregator.assert_metric(
@@ -2580,8 +2580,8 @@ def test_machines_metrics(
25802580
"machine_slug:test-machine1",
25812581
"health_status:Healthy with warnings",
25822582
"operating_system:Ubuntu 24.04.1 LTS",
2583-
"tag",
2584-
"test",
2583+
"role_name:tag",
2584+
"role_name:test",
25852585
],
25862586
)
25872587
aggregator.assert_metric(
@@ -2594,8 +2594,8 @@ def test_machines_metrics(
25942594
"machine_slug:test-machine1",
25952595
"health_status:Healthy with warnings",
25962596
"operating_system:Ubuntu 24.04.1 LTS",
2597-
"test",
2598-
"tag",
2597+
"role_name:tag",
2598+
"role_name:test",
25992599
],
26002600
)
26012601

@@ -2610,7 +2610,7 @@ def test_machines_metrics(
26102610
"machine_slug:test-machine3",
26112611
"health_status:Unhealthy",
26122612
"operating_system:Ubuntu 24.04.1 LTS",
2613-
"test",
2613+
"role_name:test",
26142614
],
26152615
)
26162616
aggregator.assert_metric(
@@ -2624,7 +2624,7 @@ def test_machines_metrics(
26242624
"machine_slug:test-machine3",
26252625
"health_status:Unhealthy",
26262626
"operating_system:Ubuntu 24.04.1 LTS",
2627-
"test",
2627+
"role_name:test",
26282628
],
26292629
)
26302630

@@ -2688,7 +2688,7 @@ def test_machines_pagination(
26882688
"machine_slug:test-machine",
26892689
"health_status:Healthy",
26902690
"operating_system:Ubuntu 24.04.1 LTS",
2691-
"test-tag",
2691+
"role_name:test-tag",
26922692
],
26932693
)
26942694
aggregator.assert_metric(
@@ -2702,7 +2702,7 @@ def test_machines_pagination(
27022702
"machine_slug:test-machine",
27032703
"health_status:Healthy",
27042704
"operating_system:Ubuntu 24.04.1 LTS",
2705-
"test-tag",
2705+
"role_name:test-tag",
27062706
],
27072707
)
27082708
aggregator.assert_metric(
@@ -2715,8 +2715,8 @@ def test_machines_pagination(
27152715
"machine_slug:test-machine1",
27162716
"health_status:Healthy with warnings",
27172717
"operating_system:Ubuntu 24.04.1 LTS",
2718-
"tag",
2719-
"test",
2718+
"role_name:tag",
2719+
"role_name:test",
27202720
],
27212721
)
27222722
aggregator.assert_metric(
@@ -2729,8 +2729,8 @@ def test_machines_pagination(
27292729
"machine_slug:test-machine1",
27302730
"health_status:Healthy with warnings",
27312731
"operating_system:Ubuntu 24.04.1 LTS",
2732-
"test",
2733-
"tag",
2732+
"role_name:tag",
2733+
"role_name:test",
27342734
],
27352735
)
27362736

@@ -2745,7 +2745,7 @@ def test_machines_pagination(
27452745
"machine_slug:test-machine3",
27462746
"health_status:Unhealthy",
27472747
"operating_system:Ubuntu 24.04.1 LTS",
2748-
"test",
2748+
"role_name:test",
27492749
],
27502750
)
27512751
aggregator.assert_metric(
@@ -2759,7 +2759,7 @@ def test_machines_pagination(
27592759
"machine_slug:test-machine3",
27602760
"health_status:Unhealthy",
27612761
"operating_system:Ubuntu 24.04.1 LTS",
2762-
"test",
2762+
"role_name:test",
27632763
],
27642764
)
27652765

0 commit comments

Comments
 (0)