-
-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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
Couchbase v5 #3227
Couchbase v5 #3227
Conversation
ci/run-couchbase-server-v5.sh
Outdated
#while sleep 9m; do echo -e '\n=====[ Gradle build is still running ]====='; done & | ||
|
||
echo "Pulling Couchbase docker image..." | ||
docker pull couchbase/server:4.6.4 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@dima767 how did you validate against Couchbase 5? This is still pulling 4.6.4, no?
docker pull couchbase/server:4.6.4 | ||
|
||
echo "Running Couchbase docker image..." | ||
docker run -d --name couchbase -p 8091-8094:8091-8094 -p 11210:11210 couchbase/server:5.1.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah, I see you're pulling/running the image here. Cool. Ignore comment :)
Verified ticket registry and services registry against both v4 and v5 of Couchbase by making unit tests (integration tests against dockerized Couchbase instance) pass for both versions. No code changes required except the script for docker image to use password for bucket for v4 and user password for v5 (username added must match the bucket name in use), and unit tests to reflect the bucket names and passwords by adding additional properties values right inside the @SpringBootTest annotation.
The separate
ci/run-couchbase-server-v5.sh
is introduced, but I'm not sure how to incorporate that into the travis machinery, etc.