Skip to content

test: redis-version to 7.2-rc1#2532

Merged
monkey92t merged 4 commits into
redis:masterfrom
monkey92t:update_redis_server
Apr 12, 2023
Merged

test: redis-version to 7.2-rc1#2532
monkey92t merged 4 commits into
redis:masterfrom
monkey92t:update_redis_server

Conversation

@monkey92t

Copy link
Copy Markdown
Collaborator

No description provided.

Signed-off-by: monkey92t <golang@88.com>
@SoulPancake

Copy link
Copy Markdown
Contributor

Should we raise the updated versions ( 7.2 compatible versions of XINFO Consumers and XINFO streams ) and then upgrade the redis server? @monkey92t
I can quickly raise the PRs for those and update the tests

Signed-off-by: monkey92t <golang@88.com>
Signed-off-by: monkey92t <golang@88.com>
@monkey92t

monkey92t commented Apr 3, 2023

Copy link
Copy Markdown
Collaborator Author

In the redis:latest image, the ENV REDIS_VERSION=7.0.10 version is used.

I corrected the image-version: redis:7.2-rc

@SoulPancake SoulPancake left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@SoulPancake

Copy link
Copy Markdown
Contributor

Can we merge this? @monkey92t For proceeding with the other PRs for version 7.2

@SoulPancake

Copy link
Copy Markdown
Contributor

Any updates on this ? @monkey92t

@monkey92t monkey92t requested review from chayim and vmihailenco April 11, 2023 02:46
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.

3 participants