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

[wip] fix: support couchdb 2.x #55

Closed
wants to merge 2 commits into from

Conversation

garbados
Copy link
Collaborator

This PR add support for 2.x and 3.x installations of CouchDB, which use a _node/${nodeName}/_config path for interacting with configuration settings, while retaining support for 1.x instances. It does this by determining the installation's version number and then setting the configuration path to either _config or _node/_local/_config. A node within a cluster will then distribute these updates to its peers.

These changes depend on this PR in couchdb-configure, and include it as a dependency in order to demonstrate compatibility. This dependency should be changed back to mainline couchdb-configure once that PR is merged, before this PR is merged.

@janl
Copy link
Collaborator

janl commented May 22, 2020

lgtm!

@garbados
Copy link
Collaborator Author

It has been brought to my attention that CouchDB nodes do not gossip config information, so this PR will need to be updated so that config data is written to each node, rather than to the local one only.

@garbados
Copy link
Collaborator Author

I've updated the PR so that each node in a cluster is configured individually.

@jo
Copy link
Owner

jo commented Feb 27, 2021

Hey @garbados, I have merged and released your couchdb-configure adaption to clustered couchdb versions. And this project is now also on GH actions, where I set up a test matrix against various couch and node versions.

Now I have rebased your changes here: #58 - there are some test errors with couch 2 and 3 - could you have a look? I have added you to this repo so that you can work here directly and push to the new branch.

@jo jo closed this Feb 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants