-
Notifications
You must be signed in to change notification settings - Fork 3.4k
[Exasol] Enable exasol product tests, after updating testcontainers #27634
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
base: master
Are you sure you want to change the base?
[Exasol] Enable exasol product tests, after updating testcontainers #27634
Conversation
The commit body is too long. https://trino.io/development/process.html#pull-request-and-commit-guidelines- is our commit message's guideline.
|
cc22c1c to
1dec42a
Compare
79381c6 to
4347bcb
Compare
Wrapped. |
|
Is "Update exasol-testcontainers" commit required for green CI? |
The commit is required to fix the CI action: |
4347bcb to
d8b46b8
Compare
Uses a tailored container that uses less disk space, this solves the CI runner disk space issue we were experiencing.
d8b46b8 to
ae10c3d
Compare
Description
This PR aims to re-enable the Exasol product tests.
We have updated the used Exasol test container to a version that uses less disk space. This solves the CI runner space pressure issue that made the product tests fail occasionally and made them flaky.
Additional context and related issues
Release notes
(x) This is not user-visible or is docs only, and no release notes are required.
( ) Release notes are required. Please propose a release note for me.
( ) Release notes are required, with the following suggested text: