Skip to content
This repository has been archived by the owner on Sep 30, 2020. It is now read-only.

[DOC] Fix destroy doc #637

Merged
merged 2 commits into from
May 16, 2017
Merged

[DOC] Fix destroy doc #637

merged 2 commits into from
May 16, 2017

Conversation

jorge07
Copy link
Contributor

@jorge07 jorge07 commented May 10, 2017

No description provided.

@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label May 10, 2017
@codecov-io
Copy link

codecov-io commented May 10, 2017

Codecov Report

Merging #637 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #637   +/-   ##
=======================================
  Coverage   38.26%   38.26%           
=======================================
  Files          51       51           
  Lines        3316     3316           
=======================================
  Hits         1269     1269           
  Misses       1845     1845           
  Partials      202      202

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 40f4e05...2fafd43. Read the comment docs.

@@ -1,8 +1,8 @@
## Destroy the cluster

When you are done with your cluster, run `kube-aws node-pools destroy --node-pool-name <name-of-your-node-pool>` and then `kube-aws destroy` to destroy all the cluster components.
When you are done with your cluster, run `kube-aws destroy node-pools <name-of-your-node-pool>` and then `kube-aws destroy` to destroy all the cluster components.
Copy link
Contributor

Choose a reason for hiding this comment

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

Thanks for the PR but kube-aws node-pools sub-command is removed since v0.9.4!
We can't (and won't need to) destroy only node pools. kube-aws destroy destroys all the cfn stacks including a control-plane and worker node pool(s).

@mumoshu mumoshu modified the milestones: backlog, wip, tbd May 12, 2017
@mumoshu mumoshu added this to To be reminded in v0.9.7 May 12, 2017
@mumoshu mumoshu modified the milestones: tbd, v0.9.7-rc.<tbd> May 12, 2017
@jorge07 jorge07 changed the title [DOC] Fix destroy node-pools doc [DOC] Fix destroy doc May 16, 2017
@mumoshu
Copy link
Contributor

mumoshu commented May 16, 2017

@jorge07 Great! Thanks for taking your time 👍

@mumoshu mumoshu merged commit 45e3e1c into kubernetes-retired:master May 16, 2017
@mumoshu mumoshu modified the milestones: v0.9.7-rc.1, v0.9.7-rc.<tbd> May 16, 2017
camilb added a commit to camilb/kube-aws that referenced this pull request May 18, 2017
* kubernetes-incubator/master:
  [DOC] Fix destroy doc (kubernetes-retired#637)
  Use new K8S 1.6 AWS tagging scheme
  Label masters with 'node-role.kubernetes.io/master' label
  Update versions for various components.
camilb added a commit to camilb/kube-aws that referenced this pull request Aug 1, 2017
…on-updates

* kubernetes-incubator/master:
  [DOC] Fix destroy doc (kubernetes-retired#637)
  Use new K8S 1.6 AWS tagging scheme
  Label masters with 'node-role.kubernetes.io/master' label
kylehodgetts pushed a commit to HotelsDotCom/kube-aws that referenced this pull request Mar 27, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
awaiting reply cncf-cla: yes Indicates the PR's author has signed the CNCF CLA.
Projects
No open projects
v0.9.7
To be reminded
Development

Successfully merging this pull request may close these issues.

None yet

4 participants