-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Examples in Couchbase documentation don't work out of the box #934
Comments
Hey Jens,
Also I assume you are using Testcontainers 1.9.1? Worst case you might need to add a breakpoint somewhere and look into the container logs to see why Couchbase isn't coming up. I have no Couchbase experience, so it might be related to the Bucket definiton? |
Yes, that version does work. Unfortunately, I never use Couchbase before either. :-/ I took the code almost literally from the readme page of the couchbase module: https://github.com/testcontainers/testcontainers-java/tree/master/modules/couchbase |
Again disclaimer, no Couchbase user here, but this works 🙂
Can you give it a try? Basically specifying password for the bucket. |
Also, maybe @Kaidowei knows what happens |
Ok, that is interesting. It does work. I thought it would configure a user (and password) to be created. |
This happens for every bucket: Lines 255 to 272 in 05b3c01
Since the classes involved in this operation are not from Testcontainers but from Couchbase, I don't think this is a Testcontainers related problem. We could probably improve UX by failing early and with more specific errors. |
lol, it's actually super simple and because of the default password policy:
Sorry for the confusion. |
Thanks, that clears it up. Having a dedicated error would indeed help. Thanks again. |
You are right. I've changed the title of the issue, so that we at least cleanup the examples. |
) Add WARN log if provided Couchbase password doesn't meet requirements See #934
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. If you believe this is a mistake, please reply to this comment to keep it open. If there isn't one already, a PR to fix or at least reproduce the problem in a test case will always help us get back on track to tackle this. |
This issue has been automatically closed due to inactivity. We apologise if this is still an active problem for you, and would ask you to re-open the issue if this is the case. |
I've created the following test
At first without the lines marked (1).
It failed with an
InvalidPasswordException
. Adding the line (1) seemed to have fixed that.But now gives:
The text was updated successfully, but these errors were encountered: