Skip to content

publish node components version information#6211

Merged
vmarmol merged 2 commits intokubernetes:masterfrom
dchen1107:docker
Mar 31, 2015
Merged

publish node components version information#6211
vmarmol merged 2 commits intokubernetes:masterfrom
dchen1107:docker

Conversation

@dchen1107
Copy link
Copy Markdown
Member

With this PR, kubectl describe nodes id includes related version info.

Running: cluster/../cluster/gce/../../cluster/../_output/dockerized/bin/linux/amd64/kubectl describe nodes kubernetes-minion-4vox.c.golden-system-455.internal
Name: kubernetes-minion-4vox.c.golden-system-455.internal
Labels:
CreationTimestamp: Mon, 30 Mar 2015 16:20:29 -0700
Conditions:
Type Status LastProbeTime LastTransitionTime Reason Message
Ready True Tue, 31 Mar 2015 00:35:37 -0700 Mon, 30 Mar 2015 16:20:59 -0700 kubelet is posting ready status
Addresses: 130.211.182.208,104.197.28.200
Capacity:
cpu: 1
memory: 3800824Ki
Version:
Kernel Version: 3.16.0-0.bpo.4-amd64
OS Image: Debian GNU/Linux 7 (wheezy)
Container Runtime Version: docker://1.5.0
Kubelet Version: v0.14.0-10-g3e0cdff97c2e77-dirty
Kube-Proxy Version: v0.14.0-10-g3e0cdff97c2e77-dirty
ExternalID: 9471772650125516840
Pods: (2 in total)
fluentd-to-elasticsearch-kubernetes-minion-4vox
monitoring-influx-grafana-controller-240nz
Events:
FirstSeen LastSeen Count From SubobjectPath Reason Message
Tue, 31 Mar 2015 00:17:25 -0700 Tue, 31 Mar 2015 00:17:25 -0700 1 {kubelet kubernetes-minion-4vox.c.golden-system-455.internal} starting Starting kubelet.

Fix #5948

@vmarmol vmarmol self-assigned this Mar 31, 2015
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we don't need this function, we can probably call cAdvisor directly above. We only export the information through status anyways.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, removed it. Thanks!

@vmarmol
Copy link
Copy Markdown
Contributor

vmarmol commented Mar 31, 2015

Thanks @dchen1107! Minor comment, LGTM otherwise.

@vmarmol
Copy link
Copy Markdown
Contributor

vmarmol commented Mar 31, 2015

LGTM, we can squash the last commit and merge on green :)

@dchen1107
Copy link
Copy Markdown
Member Author

Done with squash!

vmarmol added a commit that referenced this pull request Mar 31, 2015
publish node components version information
@vmarmol vmarmol merged commit d6d370d into kubernetes:master Mar 31, 2015
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 this pull request may close these issues.

Kubelet publish node components' version

3 participants