Skip to content

Fix integration-base image#44853

Merged
qoega merged 2 commits intomasterfrom
fix-integration-base
Jan 5, 2023
Merged

Fix integration-base image#44853
qoega merged 2 commits intomasterfrom
fix-integration-base

Conversation

@Felixoid
Copy link
Copy Markdown
Member

@Felixoid Felixoid commented Jan 2, 2023

Changelog category (leave one):

  • Build/Testing/Packaging Improvement

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

Fix zookeeper downloading, update the version, and optimize the image size.

@robot-ch-test-poll3 robot-ch-test-poll3 added the pr-build Pull request with build/testing/packaging improvement label Jan 2, 2023
@Felixoid Felixoid force-pushed the fix-integration-base branch 4 times, most recently from 832b59b to 665666b Compare January 3, 2023 10:37
@Felixoid Felixoid force-pushed the fix-integration-base branch from 665666b to 5257ff6 Compare January 3, 2023 18:32
@alexey-milovidov
Copy link
Copy Markdown
Member

@qoega, the images updated successfully, and the integration tests work, but there were many failures after we disabled retries. If this is only related to retries, let's merge.

@qoega
Copy link
Copy Markdown
Member

qoega commented Jan 5, 2023

Lets dig

Integration tests (asan) [1/6]

Integration tests (asan) [2/6] — fail: 2, passed: 248, flaky: 3

Integration tests (asan) [4/6] — fail: 32, passed: 238, flaky: 2

Integration tests (asan) [5/6] — fail: 1, passed: 465, flaky: 2

@qoega
Copy link
Copy Markdown
Member

qoega commented Jan 5, 2023

Integration tests (asan) [6/6] — fail: 2, passed: 388, flaky: 2

        # 3G limit is a big one. However, we can hit it anyway with parallel s3 writes enabled.
        # Also actual value can be bigger because of memory drift.
        # Increase it a little bit if test fails.
>       assert int(max_usage) < 3e9
E       AssertionError: assert 3356058560 < 3000000000.0
E        +  where 3356058560 = int('3356058560\n')

Integration tests (release) [3/4] — fail: 1, passed: 461, flaky: 0

Integration tests (release) [4/4] — fail: 1, passed: 354, flaky: 0

Integration tests (tsan)

@qoega qoega merged commit c552ef0 into master Jan 5, 2023
@qoega qoega deleted the fix-integration-base branch January 5, 2023 11:26
@robot-clickhouse robot-clickhouse added the pr-backports-created Backport PRs are successfully created, it won't be processed by CI script anymore label Jan 11, 2023
Felixoid added a commit that referenced this pull request Jan 11, 2023
Backport #44853 to 22.12: Fix integration-base image
@Felixoid Felixoid added v22.3-must-backport and removed pr-backports-created Backport PRs are successfully created, it won't be processed by CI script anymore labels Jan 31, 2023
@robot-ch-test-poll robot-ch-test-poll added the pr-backports-created Backport PRs are successfully created, it won't be processed by CI script anymore label Jan 31, 2023
Felixoid added a commit that referenced this pull request Feb 1, 2023
Backport #44853 to 22.3: Fix integration-base image
@Felixoid Felixoid removed the pr-backports-created Backport PRs are successfully created, it won't be processed by CI script anymore label Feb 2, 2023
@robot-ch-test-poll1 robot-ch-test-poll1 added the pr-backports-created Backport PRs are successfully created, it won't be processed by CI script anymore label Feb 2, 2023
Felixoid added a commit that referenced this pull request Feb 2, 2023
Backport #44853 to 22.11: Fix integration-base image
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-backports-created Backport PRs are successfully created, it won't be processed by CI script anymore pr-build Pull request with build/testing/packaging improvement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants