Skip to content

CI: Move docker files and remove deprecated ci scripts#82273

Merged
maxknv merged 1 commit intomasterfrom
ci_cleanup_docker_and_tests_ci
Jun 21, 2025
Merged

CI: Move docker files and remove deprecated ci scripts#82273
maxknv merged 1 commit intomasterfrom
ci_cleanup_docker_and_tests_ci

Conversation

@maxknv
Copy link
Copy Markdown
Member

@maxknv maxknv commented Jun 20, 2025

Changelog category (leave one):

  • Not for changelog (changelog entry is not required)

  • moves all docker test images into ./ci/docker
    NOTE: Docker files are still a topic for refactoring - test-util and test-base should be merged into one image, many job can reuse the same image, like performance, stress, upgrade, fuzzer; docker images must not include scripts, ch configs, etc. - It is to be addressed in follow up PRs

  • removes files from legacy ci ./tests/ci that no longer used

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):

...

Documentation entry for user-facing changes

  • Documentation is written (mandatory for new features)

@maxknv maxknv marked this pull request as draft June 20, 2025 11:31
@maxknv maxknv force-pushed the ci_cleanup_docker_and_tests_ci branch from d93e222 to a9a8a4f Compare June 20, 2025 11:31
@clickhouse-gh
Copy link
Copy Markdown
Contributor

clickhouse-gh Bot commented Jun 20, 2025

Workflow [PR], commit [3b843c7]

Summary:

job_name test_name status info comment
Stress test (amd_msan) failure
Server died FAIL
Hung check failed, possible deadlock found (see hung_check.log) FAIL
Cannot start clickhouse-server FAIL
Server failed to start (see application_errors.txt and clickhouse-server.clean.log) FAIL
Killed by signal (in clickhouse-server.log) FAIL
Fatal message in clickhouse-server.log (see fatal_messages.txt) FAIL
Killed by signal (output files) FAIL
Found signal in gdb.log FAIL

@clickhouse-gh clickhouse-gh Bot added the pr-not-for-changelog This PR should not be mentioned in the changelog label Jun 20, 2025
@maxknv maxknv force-pushed the ci_cleanup_docker_and_tests_ci branch 3 times, most recently from d4bbff9 to da60f7d Compare June 20, 2025 13:13
@maxknv maxknv force-pushed the ci_cleanup_docker_and_tests_ci branch from da60f7d to 3b843c7 Compare June 20, 2025 17:14
@maxknv maxknv marked this pull request as ready for review June 20, 2025 20:02
@maxknv maxknv enabled auto-merge June 20, 2025 20:03
@maxknv maxknv added this pull request to the merge queue Jun 21, 2025
Merged via the queue into master with commit a296f7f Jun 21, 2025
119 of 122 checks passed
@maxknv maxknv deleted the ci_cleanup_docker_and_tests_ci branch June 21, 2025 11:47
@robot-clickhouse-ci-2 robot-clickhouse-ci-2 added the pr-synced-to-cloud The PR is synced to the cloud repo label Jun 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-not-for-changelog This PR should not be mentioned in the changelog pr-synced-to-cloud The PR is synced to the cloud repo

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants