Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add kubernetes.container.status.last.reason metric #30306

Merged
merged 3 commits into from
Feb 11, 2022
Merged

Add kubernetes.container.status.last.reason metric #30306

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
3f6426a
add kubernetes.container.status.last.reason metric
tetianakravchenko Feb 9, 2022
28f2f7e
rename status.last.reason -> status.last_terminated_reason
tetianakravchenko Feb 10, 2022
b0f0bb6
fix changelog
tetianakravchenko Feb 10, 2022
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 CHANGELOG.next.asciidoc
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -180,6 +180,7 @@ https://github.com/elastic/beats/compare/v7.0.0-alpha2...main[Check the HEAD dif
- Add `xpack.enabled` support for Enterprise Search module. {pull}29871[29871]
- Add gcp firestore metricset. {pull}29918[29918]
- Remove strict parsing on RabbitMQ module {pull}30090[30090]
- Add `kubernetes.container.status.last.reason` metric {issue}27840|[27840]
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
- Add `kubernetes.container.status.last.reason` metric {issue}27840|[27840]
- Add `kubernetes.container.status.last.reason` metric {issue}30306|[30306]

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the issue number is 27840, but anyway - changed it to pull 28f2f7e


*Packetbeat*

Expand Down
12 changes: 11 additions & 1 deletion metricbeat/docs/fields.asciidoc
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -44256,7 +44256,17 @@ type: integer
*`kubernetes.container.status.reason`*::
+
--
Waiting (ContainerCreating, CrashLoopBackoff, ErrImagePull, ImagePullBackoff) or termination (Completed, ContainerCannotRun, Error, OOMKilled) reason.
The reason the container is currently in waiting (ContainerCreating, CrashLoopBackoff, ErrImagePull, ImagePullBackoff) or terminated (Completed, ContainerCannotRun, Error, OOMKilled) state.


type: keyword

--

*`kubernetes.container.status.last.reason`*::
+
--
The last reason the container was in terminated state (Completed, ContainerCannotRun, Error or OOMKilled).


type: keyword
Expand Down
2 changes: 1 addition & 1 deletion metricbeat/module/kubernetes/fields.go
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

8 changes: 6 additions & 2 deletions metricbeat/module/kubernetes/state_container/_meta/fields.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -26,8 +26,12 @@
- name: reason
type: keyword
description: >
Waiting (ContainerCreating, CrashLoopBackoff, ErrImagePull, ImagePullBackoff)
or termination (Completed, ContainerCannotRun, Error, OOMKilled) reason.
The reason the container is currently in waiting (ContainerCreating, CrashLoopBackoff, ErrImagePull,
ImagePullBackoff) or terminated (Completed, ContainerCannotRun, Error, OOMKilled) state.
- name: last.reason
type: keyword
description: >
The last reason the container was in terminated state (Completed, ContainerCannotRun, Error or OOMKilled).
- name: cpu
type: group
fields:
Expand Down
Loading