-
Notifications
You must be signed in to change notification settings - Fork 53
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
CI - Minimal and full CI matrix impl #2051
base: main
Are you sure you want to change the base?
Conversation
82bfbe1
to
e07f1f0
Compare
e07f1f0
to
e78b9b8
Compare
e78b9b8
to
218b369
Compare
.github/workflows/npm-cd.yml
Outdated
@@ -11,6 +11,7 @@ on: | |||
- .github/workflows/install-rust-and-protoc/action.yml | |||
- .github/workflows/install-shared-dependencies/action.yml | |||
- .github/workflows/install-valkey/action.yml | |||
- .github/workflows/create-test-matrices/action.yml |
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.
please fix indentation
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.
will do
@@ -9725,7 +9725,7 @@ async def cluster_route_custom_command_slot_route( | |||
route_class = SlotKeyRoute if is_slot_key else SlotIdRoute | |||
route_second_arg = "foo" if is_slot_key else 4000 | |||
primary_res = await glide_client.custom_command( | |||
["CLUSTER", "NODES"], route_class(SlotType.PRIMARY, route_second_arg) | |||
["CLUSTER", "NODES"], route_class(SlotType.PRIMARY, route_second_arg) # type: ignore |
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.
an artifact of merge?
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.
Fixes for the failing python CI picked from release branch
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.
Do you want to backport #2453 to main as another PR?
Something went wrong with node CI |
@Yury-Fridlyand Probably again differences between old Redis |
218b369
to
0ef69b8
Compare
…ed schedule and label runs Signed-off-by: avifenesh <[email protected]>
0ef69b8
to
f0eb3ee
Compare
Signed-off-by: avifenesh <[email protected]>
f0eb3ee
to
e9886c8
Compare
This pull request includes extensive updates to the CI/CD workflow documentation and configuration files. The changes focus on improving the CI/CD pipeline's flexibility, efficiency, and clarity, particularly by enhancing the matrix configurations for testing across different environments and languages.
Documentation Enhancements:
.github/DEVELOPER.md
: Added a comprehensive guide on the CI/CD workflow, detailing workflow triggers, test coverage levels, language-specific workflows, shared components, and customization options.Configuration Updates:
.github/json_matrices/build-matrix.json
: Updated build matrix to include new fields such asrun
andlanguages
, and added support for Amazon Linux. [1] [2].github/json_matrices/engine-matrix.json
: Added new engine versions and configurations for Valkey and Redis, including therun
field to specify always-run configurations..github/json_matrices/supported-language-versions.json
: Defined supported versions for multiple languages, specifying which versions should always be run.Workflow Enhancements:
.github/workflows/create-test-matrices/action.yml
: Created a new action for dynamically generating test matrices based on inputs and event types, supporting flexible CI/CD workflows..github/workflows/csharp.yml
: Updated workflow to use the new dynamic matrix generation action, added new inputs for manual triggers, and adjusted job configurations for improved testing efficiency. [1] [2]