Skip to content

Java: add CI to run modules - #2388

Merged
Yury-Fridlyand merged 12 commits into
mainfrom
java/yuryf-modules-ci
Oct 7, 2024
Merged

Yury-Fridlyand merged 12 commits into
mainfrom
java/yuryf-modules-ci

Conversation

@Yury-Fridlyand

Copy link
Copy Markdown

Based on and includes #2156

Signed-off-by: Yury-Fridlyand <yury.fridlyand@improving.com>
Signed-off-by: Yury-Fridlyand <yury.fridlyand@improving.com>
Signed-off-by: Yury-Fridlyand <yury.fridlyand@improving.com>
@Yury-Fridlyand Yury-Fridlyand added java ☕ issues and fixes related to the java client CI/CD ⚒️ CI/CD related labels Oct 3, 2024
@Yury-Fridlyand
Yury-Fridlyand requested a review from a team as a code owner October 3, 2024 22:18
Comment thread java/integTest/src/test/java/glide/modules/BasicTests.java Outdated
Comment thread java/integTest/src/test/java/glide/modules/BasicTests.java Outdated
@acarbonetto
acarbonetto self-requested a review October 3, 2024 22:29
Comment thread java/DEVELOPER.md

To run IT tests against an existing cluster and/or standalone endpoint, use:
```bash
./gradlew :integTest:test -Dcluster-endpoints=localhost:7000 -Dstandalone-endpoints=localhost:6379

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why do we have only 1 cluster endpoint?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is an example. Actually, glide core can detect (and connection to) all cluster nodes having address of only one of them.

@Yury-Fridlyand
Yury-Fridlyand marked this pull request as draft October 4, 2024 17:12
@Yury-Fridlyand
Yury-Fridlyand marked this pull request as ready for review October 7, 2024 21:49
@Yury-Fridlyand
Yury-Fridlyand merged commit abc8666 into main Oct 7, 2024
@Yury-Fridlyand
Yury-Fridlyand deleted the java/yuryf-modules-ci branch October 7, 2024 21:49
prateek-kumar-improving pushed a commit that referenced this pull request Oct 8, 2024
Signed-off-by: Yury-Fridlyand <yury.fridlyand@improving.com>
@Yury-Fridlyand Yury-Fridlyand mentioned this pull request Oct 21, 2024
6 tasks
affonsov pushed a commit that referenced this pull request Aug 21, 2026
Signed-off-by: Yury-Fridlyand <yury.fridlyand@improving.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI/CD ⚒️ CI/CD related java ☕ issues and fixes related to the java client

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants