-
-
Notifications
You must be signed in to change notification settings - Fork 1.8k
Couchbase sometimes fails rename on startup. #7049
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
base: main
Are you sure you want to change the base?
Couchbase sometimes fails rename on startup. #7049
Conversation
This will retry the rename after sleeping for 10 seconds up to 6 times. Closes testcontainers#6938.
|
Any updates on this? |
|
This looks very similar to the issue addressed in testcontainers-dotnet: testcontainers/testcontainers-dotnet#1062 |
|
Hi, is the Couchbase nature to take that long to perform that operation? Sorry, but I think I am missing something regarding to the context where it fails. It could be not enough resources when parallelizing tests? Other alternative:
Although, it could be the only way to fix it I kind of disagree with the solution given that the worst case scenario is 60 seconds, which will not provide a good experience to users. My understanding is that proposal has been done because it is actually possible but still could fail. I would like to understand more about it and hear other alternatives if any. |
|
could it be that those operations are executed when the container is starting and should be when the container is started instead? |
|
Hello everyone, @eddumelendez coming back to this a bit late, but yes most setup operations are asynchronous, must be done in order, so we have to wait, and have no other way right now. |
|
Hi, any updates on that. The issue still happened frequently, any workaround except just waiting for a certain time? |
|
The pull-request for fixing it it (retry up to one minute) is waiting for approval. |
|
@eddumelendez maybe take another look?
That is better than flaky failing in tests for user experience I think, or has to retry on user code level. |
This will retry the rename after sleeping for 10 seconds up to 6 times.
Closes #6938.