Skip to content
This repository has been archived by the owner on Jan 10, 2019. It is now read-only.

Add Uninstall docs #108

Merged
merged 1 commit into from
Jun 1, 2015
Merged

Add Uninstall docs #108

merged 1 commit into from
Jun 1, 2015

Conversation

BenWhitehead
Copy link
Contributor

No description provided.

@BenWhitehead
Copy link
Contributor Author

@ssk2 Please review. /cc @air

@air
Copy link

air commented May 27, 2015

I think this is a little cryptic:
Zookeeper Node provided to CASSANDRA_ZK
Using zkCli run the rmr command

  1. how do I find that value CASSANDRA_ZK
  2. where can I find zkCli, on what machine do I run it, what rmr args do I need.

This is great btw, will serve as reference example for others.

ssk2
ssk2 reviewed May 28, 2015
View reviewed changes
docs/docs/uninstall.md

1. Run `dcos package uninstall cassandra`
2. Zookeeper Node provided to `CASSANDRA_ZK`
* Using zkCli run the `rmr` command
Copy link
Contributor

Choose a reason for hiding this comment

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

What path do we need to rmr?

@BenWhitehead
Copy link
Contributor Author

@ssk2 I incorporated all your proposed edits and added some of the notes you mentioned. Please review.

@air In the case of DCOS I've added in the default values for the rmr command. The point about how to find the values I think plays into "How do we expose configuration values for Services", WDYT?

ssk2
ssk2 reviewed Jun 1, 2015
View reviewed changes
docs/docs/uninstall.md

To uninstall the Cassandra Mesos Framework complete the following actions:

1. Delete marathon app
Copy link
Contributor

Choose a reason for hiding this comment

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

marathon -> Marathon

@ssk2
Copy link
Contributor

ssk2 commented Jun 1, 2015

One small correction, otherwise good!

BenWhitehead added a commit that referenced this pull request Jun 1, 2015
@BenWhitehead
Merge pull request #108 from BenWhitehead/uninstall-doc
0b8d3a9 
Add Uninstall docs
@BenWhitehead BenWhitehead merged commit 0b8d3a9 into mesosphere-backup:master Jun 1, 2015
@BenWhitehead BenWhitehead deleted the uninstall-doc branch June 1, 2015 19:21
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@BenWhitehead @air @ssk2