fix(mango): allow for overriding the host on running the tests
#5219
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
pgj
force-pushed
the
fix/mango/tests/COUCH_HOST
branch
from
c248595
to
dda8799
Compare
nickva
approved these changes
Sep 5, 2024
Yes, that is the idea behind the change. For example, it can come handy if one deployed a new CouchDB cluster and somehow wants to validate if it works as expected. Especially if Clouseau (Search) has been added as well. Of course, other use cases might be possible. I heard about a company who used integration tests as health checks by running and reporting on them on a schedule, i.e. hourly or daily. |
pgj
force-pushed
the
fix/mango/tests/COUCH_HOST
branch
from
dda8799
to
a8be97f
Compare
|
@pgj ah definitely, that's a good case. Using tests as a health check is also a very neat idea. |
Currently it is not possible to configure the host where CouchDB is located so that the integration tests could always be run against the instance on `localhost`, typically launched by `dev/run`. As an answer to this, consider the value of the `COUCH_HOST` environment variable, which is in alignment with `COUCH_USER` and `COUCH_PASS`, introduced in the past. That is completely optional, it is going to fall back to the previously assumed implict value for the `localhost`.
pgj
force-pushed
the
fix/mango/tests/COUCH_HOST
branch
from
a8be97f
to
43feeba
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Currently it is not possible to configure the host where CouchDB is located so that the integration tests could always be run against the instance on
localhost, typically launched bydev/run.As an answer to this, consider the value of the
COUCH_HOSTenvironment variable, which is in alignment withCOUCH_USERandCOUCH_PASS, introduced in the past. That is completely optional, it is going to fall back to the previously assumed implict value for thelocalhost.