Skip to content

Commit 81278be

Browse files
ChrsMarkdashpole
andauthored
[k8s] Fix updowncounters to follow the non-pluralization rule (#2822)
Signed-off-by: ChrsMark <[email protected]> Co-authored-by: David Ashpole <[email protected]>
1 parent 743188c commit 81278be

File tree

5 files changed

+676
-188
lines changed

5 files changed

+676
-188
lines changed
Lines changed: 48 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,48 @@
1+
# Use this changelog template to create an entry for release notes.
2+
#
3+
# If your change doesn't affect end users you should instead start
4+
# your pull request title with [chore] or use the "Skip Changelog" label.
5+
6+
# One of 'breaking', 'deprecation', 'new_component', 'enhancement', 'bug_fix'
7+
change_type: breaking
8+
9+
# The name of the area of concern in the attributes-registry, (e.g. http, cloud, db)
10+
component: k8s
11+
12+
# A brief description of the change. Surround your text with quotes ("") if it needs to start with a backtick (`).
13+
note: "Fix plural-named updowncounter k8s metrics"
14+
15+
# Mandatory: One or more tracking issues related to the change. You can use the PR number here if no issue exists.
16+
# The values here must be integers.
17+
issues: [2301]
18+
19+
# (Optional) One or more lines of additional information to render under the primary note.
20+
# These lines will be padded with 2 spaces and then inserted directly into the document.
21+
# Use pipe (|) for multiline entries.
22+
subtext: |
23+
Renamed metrics:
24+
- k8s.node.allocatable.pods -> k8s.node.pod.allocatable
25+
- k8s.daemonset.current_scheduled_nodes -> k8s.daemonset.node.current_scheduled
26+
- k8s.daemonset.desired_scheduled_nodes -> k8s.daemonset.node.desired_scheduled
27+
- k8s.daemonset.misscheduled_nodes -> k8s.daemonset.node.misscheduled
28+
- k8s.daemonset.ready_nodes -> k8s.daemonset.node.ready
29+
- k8s.job.active_pods -> k8s.job.pod.active
30+
- k8s.job.failed_pods -> k8s.job.pod.failed
31+
- k8s.job.successful_pods -> k8s.job.pod.successful
32+
- k8s.job.desired_successful_pods -> k8s.job.pod.desired_successful
33+
- k8s.job.max_parallel_pods -> k8s.job.pod.max_parallel
34+
- k8s.cronjob.active_jobs -> k8s.cronjob.job.active
35+
- k8s.deployment.desired_pods -> k8s.deployment.pod.desired
36+
- k8s.deployment.available_pods -> k8s.deployment.pod.available
37+
- k8s.replicaset.desired_pods -> k8s.replicaset.pod.desired
38+
- k8s.replicaset.available_pods -> k8s.replicaset.pod.available
39+
- k8s.replicationcontroller.desired_pods -> k8s.replicationcontroller.pod.desired
40+
- k8s.replicationcontroller.available_pods -> k8s.replicationcontroller.pod.available
41+
- k8s.statefulset.desired_pods -> k8s.statefulset.pod.desired
42+
- k8s.statefulset.ready_pods -> k8s.statefulset.pod.ready
43+
- k8s.statefulset.current_pods -> k8s.statefulset.pod.current
44+
- k8s.statefulset.updated_pods -> k8s.statefulset.pod.updated
45+
- k8s.hpa.desired_pods -> k8s.hpa.pod.desired
46+
- k8s.hpa.current_pods -> k8s.hpa.pod.current
47+
- k8s.hpa.max_pods -> k8s.hpa.pod.max
48+
- k8s.hpa.min_pods -> k8s.hpa.pod.min

docs/non-normative/k8s-migration.md

Lines changed: 34 additions & 34 deletions
Original file line numberDiff line numberDiff line change
@@ -111,7 +111,7 @@ The changes between collector implementation and semantic conventions:
111111
| `k8s.node.allocatable_cpu` (type: `gauge`) | `k8s.node.allocatable.cpu` (type: `updowncounter`) |
112112
| `k8s.node.allocatable_memory` (type: `gauge`) | `k8s.node.allocatable.memory` (type: `updowncounter`) |
113113
| `k8s.node.allocatable_ephemeral_storage` (type: `gauge`) | `k8s.node.allocatable.ephemeral_storage` (type: `updowncounter`) |
114-
| `k8s.node.allocatable_pods` (type: `gauge`) | `k8s.node.allocatable.pods` (type: `updowncounter`) |
114+
| `k8s.node.allocatable_pods` (type: `gauge`) | `k8s.node.pod.allocatable` (type: `updowncounter`) |
115115

116116
<!-- prettier-ignore-end -->
117117

@@ -128,8 +128,8 @@ The changes in their metric names and types are the following:
128128

129129
| Old (Collector) ![changed](https://img.shields.io/badge/changed-orange?style=flat) | New |
130130
|------------------------------------------------------------------------------------|--------------------------------------------------------------|
131-
| `k8s.deployment.desired` (type: `gauge`) | `k8s.deployment.desired_pods` (type: `updowncounter`) |
132-
| `k8s.deployment.available` (type: `gauge`) | `k8s.deployment.available_pods` (type: `updowncounter`) |
131+
| `k8s.deployment.desired` (type: `gauge`) | `k8s.deployment.pod.desired` (type: `updowncounter`) |
132+
| `k8s.deployment.available` (type: `gauge`) | `k8s.deployment.pod.available` (type: `updowncounter`) |
133133

134134
<!-- prettier-ignore-end -->
135135

@@ -146,8 +146,8 @@ The changes in their metric names and types are the following:
146146

147147
| Old (Collector) ![changed](https://img.shields.io/badge/changed-orange?style=flat) | New |
148148
|------------------------------------------------------------------------------------|---------------------------------------------------------|
149-
| `k8s.replicaset.desired` (type: `gauge`) | `k8s.replicaset.desired_pods` (type: `updowncounter`) |
150-
| `k8s.replicaset.available` (type: `gauge`) | `k8s.replicaset.available_pods` (type: `updowncounter`) |
149+
| `k8s.replicaset.desired` (type: `gauge`) | `k8s.replicaset.pod.desired` (type: `updowncounter`) |
150+
| `k8s.replicaset.available` (type: `gauge`) | `k8s.replicaset.pod.available` (type: `updowncounter`) |
151151

152152
<!-- prettier-ignore-end -->
153153

@@ -164,8 +164,8 @@ The changes in their metric names and types are the following:
164164

165165
| Old (Collector) ![changed](https://img.shields.io/badge/changed-orange?style=flat) | New |
166166
|------------------------------------------------------------------------------------|--------------------------------------------------------------------|
167-
| `k8s.replication_controller.desired` (type: `gauge`) | `k8s.replicationcontroller.desired_pods` (type: `updowncounter`) |
168-
| `k8s.replication_controller.available` (type: `gauge`) | `k8s.replicationcontroller.available_pods` (type: `updowncounter`) |
167+
| `k8s.replication_controller.desired` (type: `gauge`) | `k8s.replicationcontroller.pod.desired` (type: `updowncounter`) |
168+
| `k8s.replication_controller.available` (type: `gauge`) | `k8s.replicationcontroller.pod.available` (type: `updowncounter`) |
169169

170170
<!-- prettier-ignore-end -->
171171

@@ -180,12 +180,12 @@ The changes in their metric types are the following:
180180

181181
<!-- prettier-ignore-start -->
182182

183-
| Old (Collector) ![changed](https://img.shields.io/badge/changed-orange?style=flat) | New |
184-
|------------------------------------------------------------------------------------|----------------------------------------------------------|
185-
| `k8s.statefulset.desired_pods` (type: `gauge`) | `k8s.statefulset.desired_pods` (type: `updowncounter`) |
186-
| `k8s.statefulset.ready_pods` (type: `gauge`) | `k8s.statefulset.ready_pods` (type: `updowncounter`) |
187-
| `k8s.statefulset.current_pods` (type: `gauge`) | `k8s.statefulset.current_pods` (type: `updowncounter`) |
188-
| `k8s.statefulset.updated_pods` (type: `gauge`) | `k8s.statefulset.updated_pods` (type: `updowncounter`) |
183+
| Old (Collector) ![changed](https://img.shields.io/badge/changed-orange?style=flat) | New |
184+
|------------------------------------------------------------------------------------|---------------------------------------------------------------|
185+
| `k8s.statefulset.desired_pods` (type: `gauge`) | `k8s.statefulset.pod.desired` (type: `updowncounter`) |
186+
| `k8s.statefulset.ready_pods` (type: `gauge`) | `k8s.statefulset.pod.ready` (type: `updowncounter`) |
187+
| `k8s.statefulset.current_pods` (type: `gauge`) | `k8s.statefulset.pod.current` (type: `updowncounter`) |
188+
| `k8s.statefulset.updated_pods` (type: `gauge`) | `k8s.statefulset.pod.updated` (type: `updowncounter`) |
189189

190190
<!-- prettier-ignore-end -->
191191

@@ -200,12 +200,12 @@ The changes in their metric names and types are the following:
200200

201201
<!-- prettier-ignore-start -->
202202

203-
| Old (Collector) ![changed](https://img.shields.io/badge/changed-orange?style=flat) | New |
204-
|------------------------------------------------------------------------------------|------------------------------------------------|
205-
| `k8s.hpa.desired_replicas` (type: `gauge`) | `k8s.hpa.desired_pods` (type: `updowncounter`) |
206-
| `k8s.hpa.current_replicas` (type: `gauge`) | `k8s.hpa.current_pods` (type: `updowncounter`) |
207-
| `k8s.hpa.max_replicas` (type: `gauge`) | `k8s.hpa.max_pods` (type: `updowncounter`) |
208-
| `k8s.hpa.min_replicas` (type: `gauge`) | `k8s.hpa.min_pods` (type: `updowncounter`) |
203+
| Old (Collector) ![changed](https://img.shields.io/badge/changed-orange?style=flat) | New |
204+
|------------------------------------------------------------------------------------|-----------------------------------------------------|
205+
| `k8s.hpa.desired_replicas` (type: `gauge`) | `k8s.hpa.pod.desired` (type: `updowncounter`) |
206+
| `k8s.hpa.current_replicas` (type: `gauge`) | `k8s.hpa.pod.current` (type: `updowncounter`) |
207+
| `k8s.hpa.max_replicas` (type: `gauge`) | `k8s.hpa.pod.max` (type: `updowncounter`) |
208+
| `k8s.hpa.min_replicas` (type: `gauge`) | `k8s.hpa.pod.min` (type: `updowncounter`) |
209209

210210
<!-- prettier-ignore-end -->
211211

@@ -220,12 +220,12 @@ The changes in their metric types are the following:
220220

221221
<!-- prettier-ignore-start -->
222222

223-
| Old (Collector) ![changed](https://img.shields.io/badge/changed-orange?style=flat) | New |
224-
|------------------------------------------------------------------------------------|-----------------------------------------------------------------|
225-
| `k8s.daemonset.current_scheduled_nodes` (type: `gauge`) | `k8s.daemonset.current_scheduled_nodes` (type: `updowncounter`) |
226-
| `k8s.daemonset.desired_scheduled_nodes` (type: `gauge`) | `k8s.daemonset.desired_scheduled_nodes` (type: `updowncounter`) |
227-
| `k8s.daemonset.misscheduled_nodes` (type: `gauge`) | `k8s.daemonset.misscheduled_nodes` (type: `updowncounter`) |
228-
| `k8s.daemonset.ready_nodes` (type: `gauge`) | `k8s.daemonset.ready_nodes` (type: `updowncounter`) |
223+
| Old (Collector) ![changed](https://img.shields.io/badge/changed-orange?style=flat) | New |
224+
|------------------------------------------------------------------------------------|----------------------------------------------------------------------|
225+
| `k8s.daemonset.current_scheduled_nodes` (type: `gauge`) | `k8s.daemonset.node.current_scheduled` (type: `updowncounter`) |
226+
| `k8s.daemonset.desired_scheduled_nodes` (type: `gauge`) | `k8s.daemonset.node.desired_scheduled` (type: `updowncounter`) |
227+
| `k8s.daemonset.misscheduled_nodes` (type: `gauge`) | `k8s.daemonset.node.misscheduled` (type: `updowncounter`) |
228+
| `k8s.daemonset.ready_nodes` (type: `gauge`) | `k8s.daemonset.node.ready` (type: `updowncounter`) |
229229

230230
<!-- prettier-ignore-end -->
231231

@@ -240,12 +240,12 @@ The changes in their metric types are the following:
240240

241241
<!-- prettier-ignore-start -->
242242

243-
| Old (Collector) ![changed](https://img.shields.io/badge/changed-orange?style=flat) | New |
244-
|----------------------------------------------------------|----------------------------------------|
245-
| `k8s.job.active_pods` (type: `gauge`) | `k8s.job.active_pods` (type: `updowncounter`) |
246-
| `k8s.job.failed_pods` (type: `gauge`) | `k8s.job.failed_pods` (type: `updowncounter`) |
247-
| `k8s.job.desired_successful_pods` (type: `gauge`) | `k8s.job.desired_successful_pods` (type: `updowncounter`) |
248-
| `k8s.job.max_parallel_pods` (type: `gauge`) | `k8s.job.max_parallel_pods` (type: `updowncounter`) |
243+
| Old (Collector) ![changed](https://img.shields.io/badge/changed-orange?style=flat) | New |
244+
|----------------------------------------------------------|-----------------------------------------------------------------|
245+
| `k8s.job.active_pods` (type: `gauge`) | `k8s.job.pod.active` (type: `updowncounter`) |
246+
| `k8s.job.failed_pods` (type: `gauge`) | `k8s.job.pod.failed` (type: `updowncounter`) |
247+
| `k8s.job.desired_successful_pods` (type: `gauge`) | `k8s.job.pod.desired_successful` (type: `updowncounter`) |
248+
| `k8s.job.max_parallel_pods` (type: `gauge`) | `k8s.job.pod.max_parallel` (type: `updowncounter`) |
249249

250250
### K8s Cronjob metrics
251251

@@ -258,9 +258,9 @@ The changes in their metric types are the following:
258258

259259
<!-- prettier-ignore-start -->
260260

261-
| Old (Collector) ![changed](https://img.shields.io/badge/changed-orange?style=flat) | New |
262-
|--------------------------------------------------|--------------------------------|
263-
| `k8s.cronjob.active_jobs` (type: `gauge`) | `k8s.cronjob.active_jobs` (type: `updowncounter`) |
261+
| Old (Collector) ![changed](https://img.shields.io/badge/changed-orange?style=flat) | New |
262+
|--------------------------------------------------|--------------------------------------------------------|
263+
| `k8s.cronjob.active_jobs` (type: `gauge`) | `k8s.cronjob.job.active` (type: `updowncounter`) |
264264

265265
<!-- prettier-ignore-end -->
266266

0 commit comments

Comments
 (0)