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

Consolidating the implementation of the prometheus destination #209

Closed
roda82 opened this issue Nov 13, 2023 · 0 comments · Fixed by #210
Closed

Consolidating the implementation of the prometheus destination #209

roda82 opened this issue Nov 13, 2023 · 0 comments · Fixed by #210

Comments

@roda82
Copy link
Contributor

roda82 commented Nov 13, 2023

Currently there are two implementations of the Prometheus Destination on PowerAPI:

  • DirectPrometheusDB exposing only raw power estimations. It uses sensor and target as labels
  • PrometheusDB exposing different values such as mean, std, min and max. It uses sensor and target as labels

In this PR another Prometheus Destination is defined:

  • CloudJoulePrometheusDB exposing raw power estimations grouped by targets (i.e., rapl, global, pod). It also processes labels (tags) by considering K8s rules. This implementation uses pod_name, pod_namespace and socket as labels

In this issue is also indicated that there is a conflict between the usage of socket_db as source and prometheus because of their host (127.0.0.1 for socket_db and 0.0.0.0 for prometheus implementation).

A unique basic implementation of prometheus as destination is required exposing just raw estimation metrics. If users need to expose different values with different associated labels, they have to create their own version of the destination.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant