Skip to content

Remove cluster-stack endpoint - #4432

Merged
uglide merged 1 commit into
masterfrom
im/remove-cluster-stack-endpoint
Feb 16, 2026
Merged

Remove cluster-stack endpoint#4432
uglide merged 1 commit into
masterfrom
im/remove-cluster-stack-endpoint

Conversation

@uglide

@uglide uglide commented Feb 13, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

The cluster consumes significant VM resources, so it's better to use the cluster-stable endpoint, since the test is guarded by a version check.

@github-actions

github-actions Bot commented Feb 13, 2026 •
edited
Loading

Copy link
Copy Markdown

Test Results

   302 files  ±0     302 suites  ±0   12m 10s ⏱️ +3s
10 910 tests ±0  10 693 ✅  - 159  217 💤 +159  0 ❌ ±0 
 2 842 runs  ±0   2 833 ✅  -   5    9 💤 +  5  0 ❌ ±0 

Results for commit 86194f6. ± Comparison against base commit cc42541.

This pull request skips 159 tests. 
redis.clients.jedis.commands.jedis.ClusterHotkeysCommandsTest ‑ hotkeysGetNotSupportedInCluster
redis.clients.jedis.commands.jedis.ClusterHotkeysCommandsTest ‑ hotkeysResetNotSupportedInCluster
redis.clients.jedis.commands.jedis.ClusterHotkeysCommandsTest ‑ hotkeysStartNotSupportedInCluster
redis.clients.jedis.commands.jedis.ClusterHotkeysCommandsTest ‑ hotkeysStopNotSupportedInCluster
redis.clients.jedis.commands.jedis.ClusterHotkeysCommandsTest ‑ hotkeysWithSlotsOnSingleClusterNode
redis.clients.jedis.commands.jedis.HotkeysCommandsTest[1] ‑ hotkeysBothMetrics
redis.clients.jedis.commands.jedis.HotkeysCommandsTest[1] ‑ hotkeysDurationOption
redis.clients.jedis.commands.jedis.HotkeysCommandsTest[1] ‑ hotkeysGetBeforeStart
redis.clients.jedis.commands.jedis.HotkeysCommandsTest[1] ‑ hotkeysLifecycle
redis.clients.jedis.commands.jedis.HotkeysCommandsTest[1] ‑ hotkeysResponseFields
…

♻️ This comment has been updated with latest results.

@ggivo

ggivo commented Feb 16, 2026 •
edited
Loading

Copy link
Copy Markdown
Collaborator

The intention was to use the cluster-stack endpoint for module tests in the cluster environment.
Including search module tests for older envs 7.x

ggivo
ggivo approved these changes Feb 16, 2026
View reviewed changes

@ggivo ggivo left a comment

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.

Discussed with the team and agreed that since official module support for clustered environments was introduced in 8.0+, the tests will be migrated to run against 8.0+ only. There is no longer a need to test against 7.x (Redis Stack).

@uglide
uglide merged commit f6b4175 into master Feb 16, 2026
20 of 22 checks passed
@uglide
uglide deleted the im/remove-cluster-stack-endpoint branch February 16, 2026 11:53
@ggivo ggivo added maintenance skip-changelog Ignore pull request from release note labels Feb 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ggivo ggivo ggivo approved these changes

Assignees

No one assigned

Labels

maintenance skip-changelog Ignore pull request from release note

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@uglide @ggivo