-
-
Notifications
You must be signed in to change notification settings - Fork 989
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
Added testing with 8.0 #1510
Added testing with 8.0 #1510
Conversation
…v-testing-with-8.0
…v-testing-with-8.0
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks great. Can you summarize the changed in a changelog entry?
Do we want to reduce the number of PHP runs as well? We still need to test against PHP 7.2, but maybe we test fewer Redis versions and only run 8.4 against every version?
@tillkruss I agree, we can make it as a part of this PR. I'll take care of it |
@tillkruss What about keep 7.2, 8.3, 8.4? PHP provides official support for two latest versions, so it's still possible that something might be broken and fixed in 8.3 as well |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you restore testing against 4.0 as well. I doubt people use 3.0 still, but we do advertise it.
@tillkruss Done |
Thanks! 4.0 so we test against some common old versions. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Added testing against 8.0. Changed test infrastructure to use client specific docker images.
Client specific images is a separate docker images provided and maintained by Clients & Ecosystems team within Redis LTD. They're used specifically for unified creation of test infrastructure across different clients, and provides out of the box different types of environments including Standalone, Cluster, Sentinel, Stack. Full support for TLS connections