Merge pull request #27995 from ncdc/bump-cadvisor

Automatic merge from submit-queue

bump(google/cadvisor): v0.23.6

Fix a bug where cadvisor couldn't gather container filesystem stats on RHEL 7.2 devicemapper.

Fixes #27955 

cc @kubernetes/sig-node @kubernetes/rh-cluster-infra
This commit is contained in:
k8s-merge-robot 2016-06-23 22:14:25 -07:00 committed by GitHub
commit 7552f43c69
2 changed files with 87 additions and 87 deletions

160
Godeps/Godeps.json generated
View File

@ -927,203 +927,203 @@
}, },
{ {
"ImportPath": "github.com/google/cadvisor/api", "ImportPath": "github.com/google/cadvisor/api",
"Comment": "v0.23.5", "Comment": "v0.23.6",
"Rev": "3ecedda96383d3342a5c8e5b8f39c7c9db65982f" "Rev": "4dbefc9b671b81257973a33211fb12370c1a526e"
}, },
{ {
"ImportPath": "github.com/google/cadvisor/cache/memory", "ImportPath": "github.com/google/cadvisor/cache/memory",
"Comment": "v0.23.5", "Comment": "v0.23.6",
"Rev": "3ecedda96383d3342a5c8e5b8f39c7c9db65982f" "Rev": "4dbefc9b671b81257973a33211fb12370c1a526e"
}, },
{ {
"ImportPath": "github.com/google/cadvisor/collector", "ImportPath": "github.com/google/cadvisor/collector",
"Comment": "v0.23.5", "Comment": "v0.23.6",
"Rev": "3ecedda96383d3342a5c8e5b8f39c7c9db65982f" "Rev": "4dbefc9b671b81257973a33211fb12370c1a526e"
}, },
{ {
"ImportPath": "github.com/google/cadvisor/container", "ImportPath": "github.com/google/cadvisor/container",
"Comment": "v0.23.5", "Comment": "v0.23.6",
"Rev": "3ecedda96383d3342a5c8e5b8f39c7c9db65982f" "Rev": "4dbefc9b671b81257973a33211fb12370c1a526e"
}, },
{ {
"ImportPath": "github.com/google/cadvisor/container/common", "ImportPath": "github.com/google/cadvisor/container/common",
"Comment": "v0.23.5", "Comment": "v0.23.6",
"Rev": "3ecedda96383d3342a5c8e5b8f39c7c9db65982f" "Rev": "4dbefc9b671b81257973a33211fb12370c1a526e"
}, },
{ {
"ImportPath": "github.com/google/cadvisor/container/docker", "ImportPath": "github.com/google/cadvisor/container/docker",
"Comment": "v0.23.5", "Comment": "v0.23.6",
"Rev": "3ecedda96383d3342a5c8e5b8f39c7c9db65982f" "Rev": "4dbefc9b671b81257973a33211fb12370c1a526e"
}, },
{ {
"ImportPath": "github.com/google/cadvisor/container/libcontainer", "ImportPath": "github.com/google/cadvisor/container/libcontainer",
"Comment": "v0.23.5", "Comment": "v0.23.6",
"Rev": "3ecedda96383d3342a5c8e5b8f39c7c9db65982f" "Rev": "4dbefc9b671b81257973a33211fb12370c1a526e"
}, },
{ {
"ImportPath": "github.com/google/cadvisor/container/raw", "ImportPath": "github.com/google/cadvisor/container/raw",
"Comment": "v0.23.5", "Comment": "v0.23.6",
"Rev": "3ecedda96383d3342a5c8e5b8f39c7c9db65982f" "Rev": "4dbefc9b671b81257973a33211fb12370c1a526e"
}, },
{ {
"ImportPath": "github.com/google/cadvisor/container/rkt", "ImportPath": "github.com/google/cadvisor/container/rkt",
"Comment": "v0.23.5", "Comment": "v0.23.6",
"Rev": "3ecedda96383d3342a5c8e5b8f39c7c9db65982f" "Rev": "4dbefc9b671b81257973a33211fb12370c1a526e"
}, },
{ {
"ImportPath": "github.com/google/cadvisor/container/systemd", "ImportPath": "github.com/google/cadvisor/container/systemd",
"Comment": "v0.23.5", "Comment": "v0.23.6",
"Rev": "3ecedda96383d3342a5c8e5b8f39c7c9db65982f" "Rev": "4dbefc9b671b81257973a33211fb12370c1a526e"
}, },
{ {
"ImportPath": "github.com/google/cadvisor/devicemapper", "ImportPath": "github.com/google/cadvisor/devicemapper",
"Comment": "v0.23.5", "Comment": "v0.23.6",
"Rev": "3ecedda96383d3342a5c8e5b8f39c7c9db65982f" "Rev": "4dbefc9b671b81257973a33211fb12370c1a526e"
}, },
{ {
"ImportPath": "github.com/google/cadvisor/events", "ImportPath": "github.com/google/cadvisor/events",
"Comment": "v0.23.5", "Comment": "v0.23.6",
"Rev": "3ecedda96383d3342a5c8e5b8f39c7c9db65982f" "Rev": "4dbefc9b671b81257973a33211fb12370c1a526e"
}, },
{ {
"ImportPath": "github.com/google/cadvisor/fs", "ImportPath": "github.com/google/cadvisor/fs",
"Comment": "v0.23.5", "Comment": "v0.23.6",
"Rev": "3ecedda96383d3342a5c8e5b8f39c7c9db65982f" "Rev": "4dbefc9b671b81257973a33211fb12370c1a526e"
}, },
{ {
"ImportPath": "github.com/google/cadvisor/healthz", "ImportPath": "github.com/google/cadvisor/healthz",
"Comment": "v0.23.5", "Comment": "v0.23.6",
"Rev": "3ecedda96383d3342a5c8e5b8f39c7c9db65982f" "Rev": "4dbefc9b671b81257973a33211fb12370c1a526e"
}, },
{ {
"ImportPath": "github.com/google/cadvisor/http", "ImportPath": "github.com/google/cadvisor/http",
"Comment": "v0.23.5", "Comment": "v0.23.6",
"Rev": "3ecedda96383d3342a5c8e5b8f39c7c9db65982f" "Rev": "4dbefc9b671b81257973a33211fb12370c1a526e"
}, },
{ {
"ImportPath": "github.com/google/cadvisor/http/mux", "ImportPath": "github.com/google/cadvisor/http/mux",
"Comment": "v0.23.5", "Comment": "v0.23.6",
"Rev": "3ecedda96383d3342a5c8e5b8f39c7c9db65982f" "Rev": "4dbefc9b671b81257973a33211fb12370c1a526e"
}, },
{ {
"ImportPath": "github.com/google/cadvisor/info/v1", "ImportPath": "github.com/google/cadvisor/info/v1",
"Comment": "v0.23.5", "Comment": "v0.23.6",
"Rev": "3ecedda96383d3342a5c8e5b8f39c7c9db65982f" "Rev": "4dbefc9b671b81257973a33211fb12370c1a526e"
}, },
{ {
"ImportPath": "github.com/google/cadvisor/info/v1/test", "ImportPath": "github.com/google/cadvisor/info/v1/test",
"Comment": "v0.23.5", "Comment": "v0.23.6",
"Rev": "3ecedda96383d3342a5c8e5b8f39c7c9db65982f" "Rev": "4dbefc9b671b81257973a33211fb12370c1a526e"
}, },
{ {
"ImportPath": "github.com/google/cadvisor/info/v2", "ImportPath": "github.com/google/cadvisor/info/v2",
"Comment": "v0.23.5", "Comment": "v0.23.6",
"Rev": "3ecedda96383d3342a5c8e5b8f39c7c9db65982f" "Rev": "4dbefc9b671b81257973a33211fb12370c1a526e"
}, },
{ {
"ImportPath": "github.com/google/cadvisor/machine", "ImportPath": "github.com/google/cadvisor/machine",
"Comment": "v0.23.5", "Comment": "v0.23.6",
"Rev": "3ecedda96383d3342a5c8e5b8f39c7c9db65982f" "Rev": "4dbefc9b671b81257973a33211fb12370c1a526e"
}, },
{ {
"ImportPath": "github.com/google/cadvisor/manager", "ImportPath": "github.com/google/cadvisor/manager",
"Comment": "v0.23.5", "Comment": "v0.23.6",
"Rev": "3ecedda96383d3342a5c8e5b8f39c7c9db65982f" "Rev": "4dbefc9b671b81257973a33211fb12370c1a526e"
}, },
{ {
"ImportPath": "github.com/google/cadvisor/manager/watcher", "ImportPath": "github.com/google/cadvisor/manager/watcher",
"Comment": "v0.23.5", "Comment": "v0.23.6",
"Rev": "3ecedda96383d3342a5c8e5b8f39c7c9db65982f" "Rev": "4dbefc9b671b81257973a33211fb12370c1a526e"
}, },
{ {
"ImportPath": "github.com/google/cadvisor/manager/watcher/raw", "ImportPath": "github.com/google/cadvisor/manager/watcher/raw",
"Comment": "v0.23.5", "Comment": "v0.23.6",
"Rev": "3ecedda96383d3342a5c8e5b8f39c7c9db65982f" "Rev": "4dbefc9b671b81257973a33211fb12370c1a526e"
}, },
{ {
"ImportPath": "github.com/google/cadvisor/manager/watcher/rkt", "ImportPath": "github.com/google/cadvisor/manager/watcher/rkt",
"Comment": "v0.23.5", "Comment": "v0.23.6",
"Rev": "3ecedda96383d3342a5c8e5b8f39c7c9db65982f" "Rev": "4dbefc9b671b81257973a33211fb12370c1a526e"
}, },
{ {
"ImportPath": "github.com/google/cadvisor/metrics", "ImportPath": "github.com/google/cadvisor/metrics",
"Comment": "v0.23.5", "Comment": "v0.23.6",
"Rev": "3ecedda96383d3342a5c8e5b8f39c7c9db65982f" "Rev": "4dbefc9b671b81257973a33211fb12370c1a526e"
}, },
{ {
"ImportPath": "github.com/google/cadvisor/pages", "ImportPath": "github.com/google/cadvisor/pages",
"Comment": "v0.23.5", "Comment": "v0.23.6",
"Rev": "3ecedda96383d3342a5c8e5b8f39c7c9db65982f" "Rev": "4dbefc9b671b81257973a33211fb12370c1a526e"
}, },
{ {
"ImportPath": "github.com/google/cadvisor/pages/static", "ImportPath": "github.com/google/cadvisor/pages/static",
"Comment": "v0.23.5", "Comment": "v0.23.6",
"Rev": "3ecedda96383d3342a5c8e5b8f39c7c9db65982f" "Rev": "4dbefc9b671b81257973a33211fb12370c1a526e"
}, },
{ {
"ImportPath": "github.com/google/cadvisor/storage", "ImportPath": "github.com/google/cadvisor/storage",
"Comment": "v0.23.5", "Comment": "v0.23.6",
"Rev": "3ecedda96383d3342a5c8e5b8f39c7c9db65982f" "Rev": "4dbefc9b671b81257973a33211fb12370c1a526e"
}, },
{ {
"ImportPath": "github.com/google/cadvisor/summary", "ImportPath": "github.com/google/cadvisor/summary",
"Comment": "v0.23.5", "Comment": "v0.23.6",
"Rev": "3ecedda96383d3342a5c8e5b8f39c7c9db65982f" "Rev": "4dbefc9b671b81257973a33211fb12370c1a526e"
}, },
{ {
"ImportPath": "github.com/google/cadvisor/utils", "ImportPath": "github.com/google/cadvisor/utils",
"Comment": "v0.23.5", "Comment": "v0.23.6",
"Rev": "3ecedda96383d3342a5c8e5b8f39c7c9db65982f" "Rev": "4dbefc9b671b81257973a33211fb12370c1a526e"
}, },
{ {
"ImportPath": "github.com/google/cadvisor/utils/cloudinfo", "ImportPath": "github.com/google/cadvisor/utils/cloudinfo",
"Comment": "v0.23.5", "Comment": "v0.23.6",
"Rev": "3ecedda96383d3342a5c8e5b8f39c7c9db65982f" "Rev": "4dbefc9b671b81257973a33211fb12370c1a526e"
}, },
{ {
"ImportPath": "github.com/google/cadvisor/utils/cpuload", "ImportPath": "github.com/google/cadvisor/utils/cpuload",
"Comment": "v0.23.5", "Comment": "v0.23.6",
"Rev": "3ecedda96383d3342a5c8e5b8f39c7c9db65982f" "Rev": "4dbefc9b671b81257973a33211fb12370c1a526e"
}, },
{ {
"ImportPath": "github.com/google/cadvisor/utils/cpuload/netlink", "ImportPath": "github.com/google/cadvisor/utils/cpuload/netlink",
"Comment": "v0.23.5", "Comment": "v0.23.6",
"Rev": "3ecedda96383d3342a5c8e5b8f39c7c9db65982f" "Rev": "4dbefc9b671b81257973a33211fb12370c1a526e"
}, },
{ {
"ImportPath": "github.com/google/cadvisor/utils/docker", "ImportPath": "github.com/google/cadvisor/utils/docker",
"Comment": "v0.23.5", "Comment": "v0.23.6",
"Rev": "3ecedda96383d3342a5c8e5b8f39c7c9db65982f" "Rev": "4dbefc9b671b81257973a33211fb12370c1a526e"
}, },
{ {
"ImportPath": "github.com/google/cadvisor/utils/oomparser", "ImportPath": "github.com/google/cadvisor/utils/oomparser",
"Comment": "v0.23.5", "Comment": "v0.23.6",
"Rev": "3ecedda96383d3342a5c8e5b8f39c7c9db65982f" "Rev": "4dbefc9b671b81257973a33211fb12370c1a526e"
}, },
{ {
"ImportPath": "github.com/google/cadvisor/utils/sysfs", "ImportPath": "github.com/google/cadvisor/utils/sysfs",
"Comment": "v0.23.5", "Comment": "v0.23.6",
"Rev": "3ecedda96383d3342a5c8e5b8f39c7c9db65982f" "Rev": "4dbefc9b671b81257973a33211fb12370c1a526e"
}, },
{ {
"ImportPath": "github.com/google/cadvisor/utils/sysinfo", "ImportPath": "github.com/google/cadvisor/utils/sysinfo",
"Comment": "v0.23.5", "Comment": "v0.23.6",
"Rev": "3ecedda96383d3342a5c8e5b8f39c7c9db65982f" "Rev": "4dbefc9b671b81257973a33211fb12370c1a526e"
}, },
{ {
"ImportPath": "github.com/google/cadvisor/utils/tail", "ImportPath": "github.com/google/cadvisor/utils/tail",
"Comment": "v0.23.5", "Comment": "v0.23.6",
"Rev": "3ecedda96383d3342a5c8e5b8f39c7c9db65982f" "Rev": "4dbefc9b671b81257973a33211fb12370c1a526e"
}, },
{ {
"ImportPath": "github.com/google/cadvisor/validate", "ImportPath": "github.com/google/cadvisor/validate",
"Comment": "v0.23.5", "Comment": "v0.23.6",
"Rev": "3ecedda96383d3342a5c8e5b8f39c7c9db65982f" "Rev": "4dbefc9b671b81257973a33211fb12370c1a526e"
}, },
{ {
"ImportPath": "github.com/google/cadvisor/version", "ImportPath": "github.com/google/cadvisor/version",
"Comment": "v0.23.5", "Comment": "v0.23.6",
"Rev": "3ecedda96383d3342a5c8e5b8f39c7c9db65982f" "Rev": "4dbefc9b671b81257973a33211fb12370c1a526e"
}, },
{ {
"ImportPath": "github.com/google/gofuzz", "ImportPath": "github.com/google/gofuzz",

View File

@ -150,8 +150,8 @@ func (w *ThinPoolWatcher) Refresh() error {
} }
const ( const (
thinPoolDmsetupStatusTokens = 11
thinPoolDmsetupStatusHeldMetadataRoot = 6 thinPoolDmsetupStatusHeldMetadataRoot = 6
thinPoolDmsetupStatusMinFields = thinPoolDmsetupStatusHeldMetadataRoot + 1
) )
// checkReservation checks to see whether the thin device is currently holding // checkReservation checks to see whether the thin device is currently holding
@ -163,14 +163,14 @@ func (w *ThinPoolWatcher) checkReservation(poolName string) (bool, error) {
return false, err return false, err
} }
tokens := strings.Split(string(output), " ") // we care about the field at fields[thinPoolDmsetupStatusHeldMetadataRoot],
// Split returns the input as the last item in the result, adjust the // so make sure we get enough fields
// number of tokens by one fields := strings.Fields(string(output))
if len(tokens) != thinPoolDmsetupStatusTokens+1 { if len(fields) < thinPoolDmsetupStatusMinFields {
return false, fmt.Errorf("unexpected output of dmsetup status command; expected 11 fields, got %v; output: %v", len(tokens), string(output)) return false, fmt.Errorf("unexpected output of dmsetup status command; expected at least %d fields, got %v; output: %v", thinPoolDmsetupStatusMinFields, len(fields), string(output))
} }
heldMetadataRoot := tokens[thinPoolDmsetupStatusHeldMetadataRoot] heldMetadataRoot := fields[thinPoolDmsetupStatusHeldMetadataRoot]
currentlyReserved := heldMetadataRoot != "-" currentlyReserved := heldMetadataRoot != "-"
return currentlyReserved, nil return currentlyReserved, nil
} }