CLDSRV-739: Skip S3C flaky and run with vformat and null-compat#5927
CLDSRV-739: Skip S3C flaky and run with vformat and null-compat#5927bert-e merged 2 commits intodevelopment/9.0from
Conversation
BourgoisMickael
commented
Sep 3, 2025
- Run S3C job with matrix of v0, v1, null compat
- Skip flaky test until fixed
Hello bourgoismickael,My role is to assist you with the merge of this Available options
Available commands
Status report is not available. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## development/9.0 #5927 +/- ##
================================================
Coverage 83.19% 83.19%
================================================
Files 188 188
Lines 12095 12095
================================================
Hits 10063 10063
Misses 2032 2032
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
Request integration branchesWaiting for integration branch creation to be requested by the user. To request integration branches, please comment on this pull request with the following command: Alternatively, the |
There was a problem hiding this comment.
Pull Request Overview
This PR enhances the S3C test suite to run with different versioning format configurations and null compatibility modes. The changes enable testing across multiple configurations (v0, v1, and null compatibility) to ensure comprehensive coverage.
- Adds matrix strategy to GitHub workflow for running S3C tests with v0/v1 formats and optional null compatibility
- Temporarily skips flaky tests for S3C v0 configuration to prevent build failures
- Updates configuration management to support dynamic versioning format selection
Reviewed Changes
Copilot reviewed 3 out of 4 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| .github/workflows/tests.yaml | Implements matrix strategy for S3C tests with multiple vformat and null-compat configurations |
| .github/docker/docker-compose.sse.yaml | Updates metadata config file reference to support dynamic configuration |
| tests/multipleBackend/routes/routeBackbeatForReplication.js | Conditionally skips flaky tests for S3C v0 configuration |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
|
/approve |
Integration data createdI have created the integration data for the additional destination branches.
The following branches will NOT be impacted:
You can set option The following options are set: approve |
|
I have successfully merged the changeset of this pull request
The following branches have NOT changed:
Please check the status of the associated issue CLDSRV-739. Goodbye bourgoismickael. The following options are set: approve |