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.

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

Improve host aliases page #5443

Merged
merged 3 commits into from
Sep 15, 2017
Merged

Improve host aliases page #5443

merged 3 commits into from
Sep 15, 2017

Conversation

kairen
Copy link

@kairen kairen commented Sep 13, 2017

Fixes issue #5442. Add more detail command, and fix confusion code block.


This change is Reviewable

@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Sep 13, 2017
@k8sio-netlify-preview-bot
Copy link
Collaborator

k8sio-netlify-preview-bot commented Sep 13, 2017

Deploy preview ready!

Built with commit 5a6f947

https://deploy-preview-5443--kubernetes-io-master-staging.netlify.com

@kairen
Copy link
Author

kairen commented Sep 14, 2017

@xiangpengzhao Could you help me to review this changes?

@xiangpengzhao
Copy link
Contributor

ping @rickypai who is the author of this doc for reviewing :)

@evie404
Copy link
Contributor

evie404 commented Sep 14, 2017

/lgtm

@k8s-ci-robot
Copy link
Contributor

@rickypai: changing LGTM is restricted to assignees, and only kubernetes org members may be assigned issues.

In response to this:

/lgtm

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

$ kubectl apply -f hostaliases-pod.yaml
pod "hostaliases-pod" created

$ kubectl get pod -a
Copy link
Contributor

Choose a reason for hiding this comment

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

can you do -o=wide here so it prints out the pod IP. that way the IP of this entry

10.244.3.80	hostaliases-pod

becomes clearer

Copy link
Author

Choose a reason for hiding this comment

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

@rickypai Okay!

@evie404
Copy link
Contributor

evie404 commented Sep 14, 2017

thanks for fixing this btw

@kairen
Copy link
Author

kairen commented Sep 14, 2017

@xiangpengzhao Thanks you for your help!!

@kairen
Copy link
Author

kairen commented Sep 14, 2017

@rickypai I added wide output, thanks you for review.

@kairen
Copy link
Author

kairen commented Sep 15, 2017

@rickypai Could you help me to review again?

@chenopis
Copy link
Contributor

I think this is good to go.

@chenopis chenopis merged commit 5a0c301 into kubernetes:master Sep 15, 2017
chenopis added a commit that referenced this pull request Sep 16, 2017
…hub.io into release-1.8

* 'master' of https://github.com/kubernetes/kubernetes.github.io:
  ZTE-SH-CN-debug-pod-replication-controller-2017-09-14-14
  ZTE-SH-CN-define-command-argument-container (#5381)
  Update index.md
  Update Kompose docs
  Improve host aliases page (#5443)
  Add link to AlwaysPullImages admission controller
  Revert "create homepage for user journeys"
  create homepage for user journeys
  fix the command output
  ZTE-SH-CN-run-application-run-single-instance-stateful-application-pr-2017-… (#5363)
  Add a note to static pod manifest scanning
  revert WordPress and MySQL PV doc changes to use apps/v1beta2 APIs (#5461)
  Fix pod probes yaml file
  Redirect v1 docs. (#5460)
  Fix error link (#5459)
  Fix typo in deployment documentation

# Conflicts:
#	docs/tutorials/stateful-application/basic-stateful-set.md
chenopis added a commit that referenced this pull request Sep 16, 2017
…hub.io into release-1.7

* 'master' of https://github.com/kubernetes/kubernetes.github.io:
  ZTE-SH-CN-debug-pod-replication-controller-2017-09-14-14
  ZTE-SH-CN-define-command-argument-container (#5381)
  Update index.md
  Update Kompose docs
  Improve host aliases page (#5443)
  Add link to AlwaysPullImages admission controller
  Revert "create homepage for user journeys"
  create homepage for user journeys
  fix the command output
  ZTE-SH-CN-run-application-run-single-instance-stateful-application-pr-2017-… (#5363)
  Add a note to static pod manifest scanning
  revert WordPress and MySQL PV doc changes to use apps/v1beta2 APIs (#5461)
  Fix pod probes yaml file
  Redirect v1 docs. (#5460)
  Fix error link (#5459)
  Fix typo in deployment documentation
chenopis added a commit that referenced this pull request Sep 18, 2017
…hub.io into chenopis-404-20170915

* 'master' of https://github.com/kubernetes/kubernetes.github.io:
  Adding Treasure Data to KCSP page (#5488)
  Update docs/admin/authentication.md (#5408)
  ZTE-SH-CN-debug-pod-replication-controller-2017-09-14-14
  ZTE-SH-CN-define-command-argument-container (#5381)
  Update index.md
  Update Kompose docs
  Improve host aliases page (#5443)
  Add link to AlwaysPullImages admission controller
chenopis added a commit that referenced this pull request Sep 19, 2017
…hub.io into chenopis-user-journeys

* 'master' of https://github.com/kubernetes/kubernetes.github.io: (21 commits)
  Remove User FAQ refer links (#5467)
  fix link of secret
  ZTE-SH-CN-debug-stateful-set-2017-09-19
  use para. page.version to replace specific version
  update the admission-controllers.md index.md what-is-kubernetes.md link
  Fix incorrect links
  update aggregated-api-servers.md link
  update api-group link
  fix version.md link
  Monthly 404 fix (#5494)
  Fix spelling mistake.
  Adding Treasure Data to KCSP page (#5488)
  Update docs/admin/authentication.md (#5408)
  ZTE-SH-CN-debug-pod-replication-controller-2017-09-14-14
  ZTE-SH-CN-define-command-argument-container (#5381)
  Update index.md
  Update Kompose docs
  Improve host aliases page (#5443)
  Add link to AlwaysPullImages admission controller
  Revert "create homepage for user journeys"
  ...

# Conflicts:
#	docs/home/index.md
jesscodez pushed a commit that referenced this pull request Sep 22, 2017
* Fix confusion code block

* Add more detail command

* Add pod wide output
@kairen kairen deleted the patch-hostaliases-1 branch October 3, 2017 13:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants