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

Updated install-kubeadm.md, removed redundant kubectl installation #4006

Merged
merged 2 commits into from
Jun 6, 2017
Merged

Updated install-kubeadm.md, removed redundant kubectl installation #4006

merged 2 commits into from
Jun 6, 2017

Conversation

mcwienczek
Copy link
Contributor

@mcwienczek mcwienczek commented Jun 6, 2017

Updated install-kubeadm.md:

  • Removed redundant installation of kubectl. By following instructions you were installing kubectl twice: once in separate paragraph "Installing kubectl " and then while installing kubelet and kubeadm

^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
For 1.7 Features: set Milestone to 1.7 and Base Branch to release-1.7
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

NOTE: Please check the “Allow edits from maintainers” box below to allow
reviewers fix problems on your patch and speed up the review process.
Please delete this note before submitting the pull request.


This change is Reviewable

@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Jun 6, 2017
@mcwienczek mcwienczek changed the title Updaated install-kubeadm.md, removed redundant kubectl installation Updated install-kubeadm.md, removed redundant kubectl installation Jun 6, 2017
@mengqiy
Copy link
Member

mengqiy commented Jun 6, 2017

/assign @chenopis

@mengqiy
Copy link
Member

mengqiy commented Jun 6, 2017

LGTM

@chenopis chenopis merged commit 9305c4a into kubernetes:master Jun 6, 2017
chenopis added a commit that referenced this pull request Jun 7, 2017
…hub.io into release-1.7

* 'master' of https://github.com/kubernetes/kubernetes.github.io:
  Updaated install-kubeadm.md, removed redundant kubectl installation (#4006)
  Add Chocolatey Package Manager for Windows (#4000)
chenopis added a commit that referenced this pull request Jun 7, 2017
…hub.io into release-1.6

* 'master' of https://github.com/kubernetes/kubernetes.github.io:
  Updaated install-kubeadm.md, removed redundant kubectl installation (#4006)
  Add Chocolatey Package Manager for Windows (#4000)
  Make issue title automatically a link
  Update directions for macOS bash completion
chenopis added a commit to iphutch/kubernetes.github.io that referenced this pull request Jun 26, 2017
…hub.io

* 'master' of https://github.com/kubernetes/kubernetes.github.io: (859 commits)
  fixed the line giving the template for creating configmap, should include the word configmap as part of the creation process.
  Fixes issue kubernetes#3939
  Add CRI runtime support
  Updaated install-kubeadm.md, removed redundant kubectl installation (kubernetes#4006)
  Add Chocolatey Package Manager for Windows (kubernetes#4000)
  Make issue title automatically a link
  Update directions for macOS bash completion
  Add TCP socket health check example (kubernetes#3977)
  Update networking.md (kubernetes#3950)
  [authorization] fix invalid href for webhook
  Remove self-referential link
  Add fibre channel volume explanation (kubernetes#3967)
  Improve RBAC doc (kubernetes#3951)
  Clarify CNI acronym
  Fix 404 to kubectl install link
  install-kubectl.md: Fix Windows texts, link to latest macOS binaries
  Create Installing kubeadm topic. (kubernetes#3709)
  Fix example in the "Adding ConfigMap data to a Volume" section
  Update ExternalName example
  Completes PR kubernetes#3325
  ...

# Conflicts:
#	_data/tasks.yml
#	docs/tasks/administer-cluster/namespaces-walkthrough.md
#	docs/tasks/administer-cluster/namespaces.md
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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants