Skip to content

fix(dev/run): use couchdb.uri only when auto-ports is enabled#5964

Merged
rnewson merged 2 commits into
apache:mainfrom
pgj:fix/dev/run/default.ini-vs-auto-ports
Apr 15, 2026
Merged

fix(dev/run): use couchdb.uri only when auto-ports is enabled#5964
rnewson merged 2 commits into
apache:mainfrom
pgj:fix/dev/run/default.ini-vs-auto-ports

Conversation

@pgj
Copy link
Copy Markdown
Contributor

@pgj pgj commented Apr 15, 2026

On tweaking the default.ini for setting up the launch for the CouchDB cluster, the couchdb.uri file with information on the published ports is being referenced even when it does not exist. It is only created when the --auto-ports flag is set, so we will have to make the reconfiguration of cluster consistent with that to avoid potential errors and making the startup fail.

Testing recommendations

The dev/lib/nodeN/couch.uri should be only in use in default.ini when --auto-ports is passed for dev/run. Note that with this change, couch.uri will not be considered by default any more as --auto-ports is not automatically enabled.

Related Issues or Pull Requests

This is a follow-up to #5687.

Checklist

  • This is my own work, I did not use AI, LLM's or similar technology
  • Code is written and works correctly

On tweaking the `default.ini` for setting up the launch for the
CouchDB cluster, the `couchdb.uri` file with information on the
published ports is being referenced even when it does not exist.
It is only created when the `--auto-ports` flag is set, so we will
have to make the reconfiguration of cluster consistent with that
to avoid potential errors and making the startup fail.
Comment thread Makefile
@pgj pgj force-pushed the fix/dev/run/default.ini-vs-auto-ports branch from cbfc6df to 7e190de Compare April 15, 2026 07:37
Leftover versions of `couch.uri` may create confusion so it is
better remove them alongside all the other node-specific local
data.
@pgj pgj force-pushed the fix/dev/run/default.ini-vs-auto-ports branch from 7e190de to e71c31d Compare April 15, 2026 07:38
@rnewson rnewson merged commit 57964c2 into apache:main Apr 15, 2026
59 checks passed
@pgj pgj deleted the fix/dev/run/default.ini-vs-auto-ports branch April 15, 2026 09:32
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