Skip to content
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.

Sign up for GitHub

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

Jump to bottom

ReactiveCouchbaseTemplateKeyValueIntegrationTests.findByIdWithLock needs longer duration #1662

Closed
mikereiche opened this issue Feb 11, 2023 · 4 comments
Closed

ReactiveCouchbaseTemplateKeyValueIntegrationTests.findByIdWithLock needs longer duration #1662

mikereiche opened this issue Feb 11, 2023 · 4 comments

Comments

@mikereiche
Copy link
Collaborator

The two second duration results in the test failing occasionally. (KV timeout is 2.5 seconds).
@babltiga
Copy link
Contributor

It seems introduced with my test case, let me try to fix this one ...

@babltiga
Copy link
Contributor

Does it fail on data retrieval or on data cleanup? Sadly Jenkins is not publicly accessible for me to see the logs ...

@mikereiche
Copy link
Collaborator Author

Sadly, the integration tests are not ran by jenkins as they need a cb server, and right now the runs in jenkings run without a cb server.

To run theses tests, it's necessary to modify the src/test/resources to point to a running cb server ( change the type=mocked to type=unmanaged and set the username/password and connection string to the cb server, then mvn integration-test)

All that it needs is the two second duration increased to 5 seconds or so.

mikereiche added a commit that referenced this issue Feb 14, 2023
@mikereiche
Bump couchbase sdk to 3_4_3.
9177c87 
Requires some refactoring around @Stability.Internal APIs.
Also fixed a test to get it to pass.

Closes #1661,#1662.
@mikereiche mikereiche mentioned this issue Feb 14, 2023
Merged
babltiga added a commit to babltiga/spring-data-couchbase that referenced this issue Feb 14, 2023
@babltiga
Fixed randomly failing test for get with lock.
63267a1 
Closes spring-projects#1662
Related tickets spring-projects#1662
@babltiga babltiga mentioned this issue Feb 14, 2023
Closed
5 tasks
mikereiche added a commit that referenced this issue Feb 14, 2023
@mikereiche
Bump couchbase sdk to 3_4_3.
78f8f29 
Requires some refactoring around @Stability.Internal APIs.
Also fixed a test to get it to pass.

Closes #1661,#1662.
mikereiche added a commit that referenced this issue Feb 14, 2023
@mikereiche
Bump couchbase sdk to 3_4_3.
78feaaf 
Requires some refactoring around @Stability.Internal APIs.
Also fixed a test to get it to pass.

Closes #1661,#1662.
mikereiche added a commit that referenced this issue Feb 17, 2023
@mikereiche
Bump couchbase sdk to 3_4_3.
05bd415 
Requires some refactoring around @Stability.Internal APIs.
Also fixed a test to get it to pass.

Closes #1661,#1662.
mikereiche added a commit that referenced this issue Feb 17, 2023
@mikereiche
Bump couchbase sdk to 3_4_3.
4b527eb 
Requires some refactoring around @Stability.Internal APIs.
Also fixed a test to get it to pass.

Closes #1661,#1662.
mikereiche added a commit that referenced this issue Feb 17, 2023
@mikereiche
Bump couchbase sdk to 3_4_3. (#1663)
02afaee 
Requires some refactoring around @Stability.Internal APIs.
Also fixed a test to get it to pass.

Closes #1661,#1662.
@mikereiche
Copy link
Collaborator Author

fixed along with #1663

mikereiche added a commit that referenced this issue Feb 28, 2023
@mikereiche
Bump couchbase sdk to 3_4_3. (#1663)
6924392 
Requires some refactoring around @Stability.Internal APIs.
Also fixed a test to get it to pass.

Closes #1661,#1662.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fixed randomly failing test for get with lock. babltiga/spring-data-couchbase
2 participants
@babltiga @mikereiche