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

Missing node_status data when using kube-state-metrics > v0.5.0 #6499

Closed
bwunderlich824 opened this issue Mar 5, 2018 · 4 comments · Fixed by #7699
Closed

Missing node_status data when using kube-state-metrics > v0.5.0 #6499

bwunderlich824 opened this issue Mar 5, 2018 · 4 comments · Fixed by #7699
Labels
bug containers Related to containers use case Metricbeat Metricbeat

Comments

@bwunderlich824
Copy link

bwunderlich824 commented Mar 5, 2018

In version v1.0.0-rc.1 of kube-state-metrics included a number of changes to various node_status metrics including the removal of kube_node_status_ready and the addition new metric kube_node_status_condition.

Full details can of changes can be found here.
(https://github.com/kubernetes/kube-state-metrics/releases/tag/v1.0.0-rc.1)

Metricbeats needs to be updated to deal with the changes in data from newer versions of kube-state-metrics.

@ruflin ruflin added the Metricbeat Metricbeat label Mar 11, 2018
@exekias
Copy link
Contributor

exekias commented Mar 11, 2018

Thanks for reporting @bwunderlich824, this seems about right, It looks like we will have to add some conditions to check for kube_node_status_condition here: https://github.com/elastic/beats/blob/master/metricbeat/module/kubernetes/state_node/data.go

@exekias
Copy link
Contributor

exekias commented Mar 11, 2018

Also it would be nice to update https://github.com/elastic/beats/blob/master/metricbeat/module/kubernetes/_meta/Dockerfile.kube-state to a newer version to ensure everything is still compatible

@ITBlogger
Copy link

What's the status on this? We can't properly build kubernetes alerts because of this

@exekias
Copy link
Contributor

exekias commented Jul 24, 2018

Hi @ITBlogger I didn't have a look to this yet, I'm raising it to bug as current supported version got old already

@exekias exekias added bug and removed enhancement labels Jul 24, 2018
@tsg tsg closed this as completed in #7699 Jul 24, 2018
@exekias exekias added the containers Related to containers use case label Jul 25, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug containers Related to containers use case Metricbeat Metricbeat
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants