Skip to content

DBI-829: MariaDB as iteration dimension#4516

Closed
pfcoperez wants to merge 1 commit into
mainfrom
CP-829/mariadb-9-12-13
Closed

DBI-829: MariaDB as iteration dimension#4516
pfcoperez wants to merge 1 commit into
mainfrom
CP-829/mariadb-9-12-13

Conversation

@pfcoperez

@pfcoperez pfcoperez commented Jun 30, 2026

Copy link
Copy Markdown
Member

alternative draft to #4504
TODO: Figure out the best way to run MySQL tests with maria flavor

@codecov

codecov Bot commented Jun 30, 2026

Copy link
Copy Markdown

⚠️ JUnit XML file not found

The CLI was unable to find any JUnit XML files to upload.
For more help, visit our troubleshooting guide.

@github-actions

Copy link
Copy Markdown
Contributor

❌ Test Failure

Analysis: Not flaky — all matrix jobs deterministically fail at the DB container setup because DB_IMG/DB_ENV/DB_PARAMS env vars are empty, making docker run ... -e error with "flag needs an argument: 'e' in -e" (exit 125) before any test runs.
Confidence: 0.95

⚠️ This appears to be a real bug - manual intervention needed

View workflow run

pfcoperez added a commit that referenced this pull request Jun 30, 2026
Changes:
* Incorporate #4516 making Maria a separate DB in the matrix
* Spawn out the maria-roots of e2e suites
* Introduce TestIntegraion.../mysql and .../mariadb naming pattern for
integration tests, filter from tilt with a matcher. MySQL-only are
TestMySQLOnlyIntegration... to avoid running them from maria targets as
parent tests are first ran to discover subtests. Unit tests are out of
scope for tilt as they don't need any infra

Also GitHub seems to not handle indentation diffs very well, most of the
line changes are just subtest indentation

---------

Co-authored-by: Pablo Francisco Pérez Hidalgo <pablo.perez@clickhouse.com>
@pfcoperez

Copy link
Copy Markdown
Member Author

Superseded by #4517

@pfcoperez pfcoperez closed this Jun 30, 2026
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.

2 participants