-
Notifications
You must be signed in to change notification settings - Fork 611
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
[cinder-csi-plugin] Migrate in-tree cloud-provider metrics for cinder plugin #1392
Comments
#1236 merged recently /assign |
First , we need expose the http endpoint of the CSI containers #1398 is doing that by adding an http endpoint and by doing that , we can achieve
then after above is done, |
@jichenjc Thanks. Got it. I'm wondering if we are adding only openstack api call metrics, cant we use uniform way of retrieving metrics or same endpoint for metrics . Right now we have different ways of retrieving openstack metrics for diff plugins, so may be it would less confusing if we can expose metrics with same endpoint or extension of the same. not sure if this is possible? |
@ramineni I agree we need use same endpoint for all openstack calls, but seems we can't assume CSI metrics come from Cloud provider container endpoint? the /metrics are consistent across all containers |
@jichenjc I meant some extension like https://localhost:10258/metrics/csi etc? |
Issues go stale after 90d of inactivity. If this issue is safe to close now please do so with Send feedback to sig-contributor-experience at kubernetes/community. |
/remove-lifecycle stale |
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:
You can:
Please send feedback to sig-contributor-experience at kubernetes/community. /lifecycle stale |
/remove-lifecycle stale |
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:
You can:
Please send feedback to sig-contributor-experience at kubernetes/community. /lifecycle stale |
/remove-lifecycle stale |
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:
You can:
Please send feedback to sig-contributor-experience at kubernetes/community. /lifecycle stale |
/remove-lifecycle stale |
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:
You can:
Please send feedback to sig-contributor-experience at kubernetes/community. /lifecycle stale |
The Kubernetes project currently lacks enough active contributors to adequately respond to all issues and PRs. This bot triages issues and PRs according to the following rules:
You can:
Please send feedback to sig-contributor-experience at kubernetes/community. /lifecycle rotten |
/remove-lifecycle stale |
The Kubernetes project currently lacks enough active contributors to adequately respond to all issues and PRs. This bot triages issues and PRs according to the following rules:
You can:
Please send feedback to sig-contributor-experience at kubernetes/community. /close |
@k8s-triage-robot: Closing this issue. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
Is this a BUG REPORT or FEATURE REQUEST?:
What happened:
Current for in-tree cinder volume usage, we can get the cloud-provider related metrics from the legacy-cloud-provider package[1]. To fully support CSI migration, we should evaluate if these metrics are still essential and are being consumed by any components. If not, we might need to deprecate them. If yes, we should come up with a plan on how to include them in the CSI driver.
[1] https://github.com/kubernetes/kubernetes/blob/e161051c8366c6c90502f1a4ef06da2f01db2264/staging/src/k8s.io/legacy-cloud-providers/openstack/openstack_metrics.go#L35
What you expected to happen:
How to reproduce it:
Anything else we need to know?:
Environment:
The text was updated successfully, but these errors were encountered: