Skip to content

Commit 9006b2d

Browse files
committed
Fix for API change in cadvisor
Signed-off-by: Davanum Srinivas <[email protected]>
1 parent 3992c86 commit 9006b2d

File tree

3 files changed

+11
-3
lines changed

3 files changed

+11
-3
lines changed

pkg/kubelet/cadvisor/cadvisor_linux.go

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,6 +45,7 @@ import (
4545
"github.com/google/cadvisor/manager"
4646
"github.com/google/cadvisor/utils/sysfs"
4747
"k8s.io/klog"
48+
"k8s.io/utils/pointer"
4849
)
4950

5051
type cadvisorClient struct {
@@ -99,8 +100,14 @@ func New(imageFsInfoProvider ImageFsInfoProvider, rootPath string, cgroupRoots [
99100
includedMetrics[cadvisormetrics.DiskUsageMetrics] = struct{}{}
100101
}
101102

103+
duration := maxHousekeepingInterval
104+
housekeepingConfig := manager.HouskeepingConfig{
105+
Interval: &duration,
106+
AllowDynamic: pointer.BoolPtr(allowDynamicHousekeeping),
107+
}
108+
102109
// Create the cAdvisor container manager.
103-
m, err := manager.New(memory.New(statsCacheDuration, nil), sysFs, maxHousekeepingInterval, allowDynamicHousekeeping, includedMetrics, http.DefaultClient, cgroupRoots)
110+
m, err := manager.New(memory.New(statsCacheDuration, nil), sysFs, housekeepingConfig, includedMetrics, http.DefaultClient, cgroupRoots, "")
104111
if err != nil {
105112
return nil, err
106113
}

pkg/kubelet/server/server.go

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -39,6 +39,7 @@ import (
3939
"google.golang.org/grpc"
4040
"k8s.io/klog"
4141
"k8s.io/kubernetes/pkg/kubelet/metrics/collectors"
42+
"k8s.io/utils/clock"
4243

4344
"k8s.io/api/core/v1"
4445
metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
@@ -342,7 +343,7 @@ func (s *Server) InstallDefaultHandlers(enableCAdvisorJSONEndpoints bool) {
342343
cadvisormetrics.AppMetrics: struct{}{},
343344
cadvisormetrics.ProcessMetrics: struct{}{},
344345
}
345-
r.RawMustRegister(metrics.NewPrometheusCollector(prometheusHostAdapter{s.host}, containerPrometheusLabelsFunc(s.host), includedMetrics))
346+
r.RawMustRegister(metrics.NewPrometheusCollector(prometheusHostAdapter{s.host}, containerPrometheusLabelsFunc(s.host), includedMetrics, clock.RealClock{}))
346347
s.restfulCont.Handle(cadvisorMetricsPath,
347348
compbasemetrics.HandlerFor(r, compbasemetrics.HandlerOpts{ErrorHandling: compbasemetrics.ContinueOnError}),
348349
)

pkg/kubelet/stats/stats_provider_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -557,7 +557,7 @@ func generateCustomMetricSpec() []cadvisorapiv1.MetricSpec {
557557
case 1:
558558
e.Type = cadvisorapiv1.MetricCumulative
559559
case 2:
560-
e.Type = cadvisorapiv1.MetricDelta
560+
e.Type = cadvisorapiv1.MetricType("delta")
561561
}
562562
switch c.Intn(2) {
563563
case 0:

0 commit comments

Comments
 (0)