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.

Sign up for GitHub

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

Jump to bottom

Fix invalid metric name panic reading powercap dir #1800

Closed
wants to merge 8 commits into from
Closed

Fix invalid metric name panic reading powercap dir #1800

Changes from 2 commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
d0e7535
Fix invalid metric name panic reading powercap dir
chicknsoup Jul 27, 2020
d3cc495
Fixed imports
chicknsoup Jul 27, 2020
d99586b
Replace strings.Replace w/ strutil.SanitizeLabelName
chicknsoup Jul 27, 2020
3d51b99
Put rz.Name as value of name label
chicknsoup Jul 27, 2020
4c1ae90
Update rapl_linux.go
chicknsoup Jul 27, 2020
a679e54
Update e2e-output.txt
chicknsoup Jul 29, 2020
7a400fd
Update e2e-output.txt
chicknsoup Jul 29, 2020
48256b0
Update e2e-output.txt
chicknsoup Jul 29, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion collector/rapl_linux.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@ package collector

import (
"strconv"
"strings"

"github.com/go-kit/kit/log"
"github.com/prometheus/client_golang/prometheus"
Expand Down Expand Up @@ -61,7 +62,7 @@ func (c *raplCollector) Update(ch chan<- prometheus.Metric) error {
index := strconv.Itoa(rz.Index)

descriptor := prometheus.NewDesc(
prometheus.BuildFQName(namespace, "rapl", rz.Name+"_joules_total"),
prometheus.BuildFQName(namespace, "rapl", strings.ReplaceAll(rz.Name, "-", "_")+"_joules_total"),
chicknsoup marked this conversation as resolved.
Show resolved Hide resolved
"Current RAPL "+rz.Name+" value in joules",
[]string{"index"}, nil,
)
Expand Down