Skip to content

Fix test_storage_s3_queue/test.py::test_alter_settings#75614

Merged
kssenii merged 2 commits intomasterfrom
try-fix-s3queue-test-alter-settings
Feb 5, 2025
Merged

Fix test_storage_s3_queue/test.py::test_alter_settings#75614
kssenii merged 2 commits intomasterfrom
try-fix-s3queue-test-alter-settings

Conversation

@kssenii
Copy link
Copy Markdown
Member

@kssenii kssenii commented Feb 5, 2025 •
edited
Loading

Changelog category (leave one):

  • Not for changelog (changelog entry is not required)

Closes #73263.

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)

Information about CI checks: https://clickhouse.com/docs/en/development/continuous-integration/

CI Settings (Only check the boxes if you know what you are doing)

All builds in Builds_1 and Builds_2 stages are always mandatory and will run independently of the checks below:

  • Only: Stateless tests
  • Only: Integration tests
  • Only: Performance tests
  • Skip: Style check
  • Skip: Fast test
  • Run all checks ignoring all possible failures (Resource-intensive. All test jobs execute in parallel).
  • Disable CI cache

@robot-ch-test-poll robot-ch-test-poll added the pr-not-for-changelog This PR should not be mentioned in the changelog label Feb 5, 2025
@robot-ch-test-poll1
Copy link
Copy Markdown
Contributor

robot-ch-test-poll1 commented Feb 5, 2025 •
edited by robot-ch-test-poll3
Loading

This is an automated comment for commit 4d077cd with description of existing statuses. It's updated for the latest CI running

❌ Click here to open a full report in a separate page

Check nameDescriptionStatus
Integration testsThe integration tests report. In parenthesis the package type is given, and in square brackets are the optional part/total tests❌ failure
Successful checks
Check nameDescriptionStatus
BuildsThere's no description for the check yet, please add it to tests/ci/ci_config.py:CHECK_DESCRIPTIONS✅ success
Style checkRuns a set of checks to keep the code style clean. If some of tests failed, see the related log from the report✅ success

@Michicosun Michicosun self-assigned this Feb 5, 2025
@kssenii kssenii enabled auto-merge February 5, 2025 21:24
@kssenii
Copy link
Copy Markdown
Member Author

kssenii commented Feb 5, 2025

Integration tests (tsan) [5/6] — fail: 1,

test_delayed_replica_failover

@kssenii kssenii added this pull request to the merge queue Feb 5, 2025
Merged via the queue into master with commit 3d13a14 Feb 5, 2025
58 of 59 checks passed
@kssenii kssenii deleted the try-fix-s3queue-test-alter-settings branch February 5, 2025 22:17
@robot-ch-test-poll1 robot-ch-test-poll1 added the pr-synced-to-cloud The PR is synced to the cloud repo label Feb 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Michicosun Michicosun Michicosun approved these changes

Assignees

@Michicosun Michicosun

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

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Test test_storage_s3_queue/test_alter_settings is flaky

4 participants

@kssenii @robot-ch-test-poll1 @Michicosun @robot-ch-test-poll