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.

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

test: kv test cosmosdb increase timeout while waiting for container #8660

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

nopcoder
Copy link
Contributor

Closes #8659

Limit the kv test using cosmos db to run on windows only.

@nopcoder nopcoder added area/testing Improvements or additions to tests exclude-changelog PR description should not be included in next release changelog area/KV Improvements to the KV store implementation labels Feb 16, 2025
@nopcoder nopcoder requested a review from itaiad200 February 16, 2025 09:42
@nopcoder nopcoder self-assigned this Feb 16, 2025
Copy link

E2E Test Results - Quickstart

11 passed

Copy link

E2E Test Results - DynamoDB Local - Local Block Adapter

14 passed

@nopcoder nopcoder force-pushed the chore/test-cosmosdb branch from dd4cea6 to 9bf455d Compare February 16, 2025 10:02
@nopcoder nopcoder changed the title test: limit kv test on cosmosdb on windows only test: kv test cosmosdb increase timeout while waiting for container Feb 16, 2025
- No magic numbers
- Adjusted the client timeout to 30 seconds for the CosmosDB client in the main_test.go file.
- Updated the client configuration in the cosmosdb.go file to include a timeout of 5 seconds.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/KV Improvements to the KV store implementation area/testing Improvements or additions to tests exclude-changelog PR description should not be included in next release changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Testing using CosmosDB Emulator fails sometimes
1 participant