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.

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

[Kubernetes] Metricset naming for namespace: state_namespace or namespace? #36737

Open
constanca-m opened this issue Oct 4, 2023 · 1 comment
Labels
Metricbeat Metricbeat Stalled Team:Cloudnative-Monitoring Label for the Cloud Native Monitoring team

Comments

@constanca-m
Copy link
Contributor

constanca-m commented Oct 4, 2023

The introduction of a new metricset for the namespace resource is tracked in this issue.

When trying to add the metricset with the name namespace, following the same logic we have for pods, deployments etc, we saw that kubernetes.namespace was already being used as a string. Because of, trying to introduce the object kubernetes.namespace was no longer possible:

{\"type\":\"document_parsing_exception\",\"reason\":\"[1:2039] object mapping for [kubernetes.namespace] tried to parse field [namespace] as object, but found a concrete value\"}, dropping event!","service.name":"metricbeat","ecs.version":"1.6.0"}

The workaround found was to use this new metricset with the name state_namespace. However, this is not ideal.

This issue is a reminder that maybe we should embrace the breaking change for kubernetes.namespace:

  1. We should free the kubernetes.namespace as a string field.
  2. Now that the field is available, we should rename the object kubernetes.state_namespace to kubernetes.namespace.
@constanca-m constanca-m added the Metricbeat Metricbeat label Oct 4, 2023
@botelastic botelastic bot added the needs_team Indicates that the issue/PR needs a Team:* label label Oct 4, 2023
@constanca-m constanca-m added the Team:Cloudnative-Monitoring Label for the Cloud Native Monitoring team label Oct 4, 2023
@botelastic botelastic bot removed the needs_team Indicates that the issue/PR needs a Team:* label label Oct 4, 2023
@botelastic
Copy link

botelastic bot commented Oct 3, 2024

Hi!
We just realized that we haven't looked into this issue in a while. We're sorry!

We're labeling this issue as Stale to make it hit our filters and make sure we get back to it as soon as possible. In the meantime, it'd be extremely helpful if you could take a look at it as well and confirm its relevance. A simple comment with a nice emoji will be enough :+1.
Thank you for your contribution!

@botelastic botelastic bot added the Stalled label Oct 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Metricbeat Metricbeat Stalled Team:Cloudnative-Monitoring Label for the Cloud Native Monitoring team
Projects
None yet
Development

No branches or pull requests

1 participant