Skip to content

8.8 (unstable) validation#466

Open
mgravell wants to merge 6 commits intomasterfrom
server-8.8
Open

8.8 (unstable) validation#466
mgravell wants to merge 6 commits intomasterfrom
server-8.8

Conversation

@mgravell
Copy link
Collaborator

@mgravell mgravell commented Mar 20, 2026

Back-compat validation for server 8.8; no functional changes - CI only; compensates for:

  • changes to some search error messages (no such index becomes index not found, from here)
  • changes to info search output (looks like a lot of internal detail sections removed)

Note

Low Risk
Low risk: changes are limited to CI/docker test configuration and test assertions to accommodate Redis 8.8 output/message differences.

Overview
Adds Redis 8.8 to the GitHub Actions integration test matrix and updates the docker test setup to use the redislabs/client-libs-test:unstable-23321515778-debian image (with a new tests/dockers/.env.v8.8).

Adjusts a few tests to be version-aware for Redis 8.8, including new EndpointsFixture.IsAtLeast(...) helper and updated assertions for INFO search output size and FT.DROPINDEX error message text.

Written by Cursor Bugbot for commit cdfc87a. This will update automatically on new commits. Configure here.

@jit-ci
Copy link

jit-ci bot commented Mar 20, 2026

🛡️ Jit Security Scan Results

CRITICAL HIGH MEDIUM

✅ No security findings were detected in this PR


Security scan by Jit

Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

There are 2 total unresolved issues (including 1 from previous review).

Fix All in Cursor

Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant