kube-state-metrics icon indicating copy to clipboard operation
kube-state-metrics copied to clipboard

Published 20 hours ago • verified publisher icon kubernetes

Remove Verticalpodautoscaler Collector (VPA)

Open mrueg opened this issue 2 years ago • 2 comments

As we have now an implementation to collect metrics from Custom Resources, we want to remove the VPA Collector in KSM v3.0

mrueg avatar Jul 19 '22 13:07 mrueg

I fully understand the argument to remove the VPA Collector, but it's not possible right now.

The values from the vpa-recommender looks like this:

...
status:
  conditions:
  - lastTransitionTime: "2022-07-04T13:40:27Z"
    status: "True"
    type: RecommendationProvided
  recommendation:
    containerRecommendations:
    - containerName: main
      lowerBound:
        cpu: 25m
        memory: 262144k
      target:
        cpu: 25m
        memory: 262144k
      uncappedTarget:
        cpu: 25m
        memory: 262144k
      upperBound:
        cpu: 25m
        memory: 262144k

and the "25m" cant be converted to a value... will just result in this:

E0826 12:39:39.333864  925520 registry_factory.go:556] "kube_autoscaling_k8s_io_v1_verticalpodautoscaler_cpu_rec" err="[status,recommendation,containerRecommendations,0,target]: [cpu]: []: strconv.ParseFloat: parsing \"25m\": invalid syntax"

As I see it, either: a) update the VPA Collector to apiVersion autoscaling.k8s.io/v1 (according to #1718 ) b) implement a parser/converter for values that can convert the measurement unit to a plain value.

ninlil avatar Aug 26 '22 13:08 ninlil

/assign

rexagod avatar Sep 15 '22 05:09 rexagod

The Kubernetes project currently lacks enough contributors to adequately respond to all issues and PRs.

This bot triages issues and PRs according to the following rules:

  • After 90d of inactivity, lifecycle/stale is applied
  • After 30d of inactivity since lifecycle/stale was applied, lifecycle/rotten is applied
  • After 30d of inactivity since lifecycle/rotten was applied, the issue is closed

You can:

  • Mark this issue or PR as fresh with /remove-lifecycle stale
  • Mark this issue or PR as rotten with /lifecycle rotten
  • Close this issue or PR with /close
  • Offer to help out with Issue Triage

Please send feedback to sig-contributor-experience at kubernetes/community.

/lifecycle stale

k8s-triage-robot avatar Dec 18 '22 06:12 k8s-triage-robot

/remove-lifecycle stale

mrueg avatar Dec 18 '22 08:12 mrueg