Relax restriction on [admins]
in dev local.ini
#4893
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Overview
Currently,
dev/run
only starts CouchDB if the[admins]
section is last inlocal.ini
. Otherwise, it will fail, and log this message:looking at the
local.ini
in question, we see:where the admin user has been appended to the end of the file instead of in the
[admins]
section.This relaxes that restriction by inserting the admin user just after the
[admin]
section line, rather than at the end of the file.Testing recommendations
should actually start, and e.g.
tail -20 dev/lib/node1/etc/local.ini
should contain something likeRelated Issues or Pull Requests
Checklist
rel/overlay/etc/default.ini
src/docs
folder