Skip to content

Commit

Permalink
Merge pull request #49339 from MrHohn/automated-cherry-pick-of-#49316-#…
Browse files Browse the repository at this point in the history
…49330-upstream-release-1.7

Automated cherry pick of #49316 #49330
  • Loading branch information
wojtek-t authored Jul 21, 2017
2 parents e7bb571 + 6fc8749 commit 66bf0de
Show file tree
Hide file tree
Showing 3 changed files with 11 additions and 8 deletions.
2 changes: 2 additions & 0 deletions cluster/gce/gci/configure-helper.sh
Original file line number Diff line number Diff line change
Expand Up @@ -987,6 +987,8 @@ function start-node-problem-detector {
flags+=" --logtostderr"
flags+=" --system-log-monitors=${km_config},${dm_config}"
flags+=" --apiserver-override=https://${KUBERNETES_MASTER_NAME}?inClusterConfig=false&auth=/var/lib/node-problem-detector/kubeconfig"
local -r npd_port=${NODE_PROBLEM_DETECTOR_PORT:-20256}
flags+=" --port=${npd_port}"

# Write the systemd service file for node problem detector.
cat <<EOF >/etc/systemd/system/node-problem-detector.service
Expand Down
4 changes: 2 additions & 2 deletions pkg/cloudprovider/providers/gce/gce_healthchecks.go
Original file line number Diff line number Diff line change
Expand Up @@ -37,8 +37,8 @@ var (
)

func init() {
if v, err := utilversion.ParseGeneric("1.7.0"); err != nil {
panic(err)
if v, err := utilversion.ParseGeneric("1.7.2"); err != nil {
glog.Fatalf("Failed to parse version for minNodesHealthCheckVersion: %v", err)
} else {
minNodesHealthCheckVersion = v
}
Expand Down
13 changes: 7 additions & 6 deletions pkg/cloudprovider/providers/gce/gce_healthchecks_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -27,8 +27,9 @@ func TestIsAtLeastMinNodesHealthCheckVersion(t *testing.T) {
version string
expect bool
}{
{"v1.7.1", true},
{"v1.7.0-alpha.2.597+276d289b90d322", true},
{"v1.7.3", true},
{"v1.7.2", true},
{"v1.7.2-alpha.2.597+276d289b90d322", true},
{"v1.6.0-beta.3.472+831q821c907t31a", false},
{"v1.5.2", false},
}
Expand All @@ -52,14 +53,14 @@ func TestSupportsNodesHealthCheck(t *testing.T) {
{
Status: v1.NodeStatus{
NodeInfo: v1.NodeSystemInfo{
KubeProxyVersion: "v1.7.1",
KubeProxyVersion: "v1.7.2",
},
},
},
{
Status: v1.NodeStatus{
NodeInfo: v1.NodeSystemInfo{
KubeProxyVersion: "v1.7.0-alpha.2.597+276d289b90d322",
KubeProxyVersion: "v1.7.2-alpha.2.597+276d289b90d322",
},
},
},
Expand Down Expand Up @@ -92,14 +93,14 @@ func TestSupportsNodesHealthCheck(t *testing.T) {
{
Status: v1.NodeStatus{
NodeInfo: v1.NodeSystemInfo{
KubeProxyVersion: "v1.7.1",
KubeProxyVersion: "v1.7.3",
},
},
},
{
Status: v1.NodeStatus{
NodeInfo: v1.NodeSystemInfo{
KubeProxyVersion: "v1.7.0-alpha.2.597+276d289b90d322",
KubeProxyVersion: "v1.7.2-alpha.2.597+276d289b90d322",
},
},
},
Expand Down

0 comments on commit 66bf0de

Please sign in to comment.