Skip to content
This repository was archived by the owner on Oct 17, 2022. It is now read-only.

Fully document _cluster_setup endpoint for 2.1#155

Merged
wohali merged 1 commit intomasterfrom
593-setup-wizard
Jul 30, 2017
Merged

Fully document _cluster_setup endpoint for 2.1#155
wohali merged 1 commit intomasterfrom
593-setup-wizard

Conversation

@wohali
Copy link
Copy Markdown
Member

@wohali wohali commented Jul 30, 2017

Documents the new enable_single_node action as well as the required node_count parameter. Also, adds the full syntax for the /_cluster_setup endpoint to the HTTP API reference.

Closes apache/couchdb#593

@wohali wohali requested a review from janl July 30, 2017 07:52
@wohali wohali force-pushed the 593-setup-wizard branch from 84cdce3 to a2d26bc Compare July 30, 2017 07:55
Even when joining more than 3 nodes into a cluster using this endpoint,
it is **strongly** recommended not to set this value higher than 3, as
it will directly impact the number of replicas of each document in the
cluster.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for raising this issue, but I think that behaviour is weird. I approved the original change, and now understand that we were missing something. I’ve addressed the issue in apache/couchdb-setup#14

If that PR is merged, we can leave out this warning. This would be a nice-to-have for 2.1, but not blocking.

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.

Agree 💯%. Please merge apache/couchdb-setup#14 and I'll remove the warning here.

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.

Merged and removing.

@wohali wohali force-pushed the 593-setup-wizard branch from a2d26bc to e737942 Compare July 30, 2017 21:25
@wohali wohali merged commit 8e1c399 into master Jul 30, 2017
@wohali wohali deleted the 593-setup-wizard branch July 30, 2017 21:25
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants