Skip to content

Make s3 session token test not flaky#96234

Merged
scanhex12 merged 1 commit intomasterfrom
scanhex12-patch-1
Feb 6, 2026
Merged

Make s3 session token test not flaky#96234
scanhex12 merged 1 commit intomasterfrom
scanhex12-patch-1

Conversation

@scanhex12
Copy link
Copy Markdown
Member

Changelog category (leave one):

  • CI Fix or Improvement (changelog entry is not required)

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

Make s3 session token test not flaky

Documentation entry for user-facing changes

  • Documentation is written (mandatory for new features)

@clickhouse-gh
Copy link
Copy Markdown
Contributor

clickhouse-gh Bot commented Feb 6, 2026

Workflow [PR], commit [1bf5e4c]

Summary:

@clickhouse-gh clickhouse-gh Bot added the pr-ci label Feb 6, 2026
Copy link
Copy Markdown
Member

@rienath rienath left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you link the issue this PR closes?

Copy link
Copy Markdown
Contributor

@grantholly-clickhouse grantholly-clickhouse left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@grantholly-clickhouse grantholly-clickhouse self-assigned this Feb 6, 2026
@scanhex12
Copy link
Copy Markdown
Member Author

@rienath there is no issue for this test. I noticed that this test can be flaky when making related changes, such as in this PR: #95069

@scanhex12 scanhex12 enabled auto-merge February 6, 2026 19:00
@scanhex12 scanhex12 added this pull request to the merge queue Feb 6, 2026
Merged via the queue into master with commit dc26df6 Feb 6, 2026
133 checks passed
@scanhex12 scanhex12 deleted the scanhex12-patch-1 branch February 6, 2026 20:19
@robot-clickhouse robot-clickhouse added the pr-synced-to-cloud The PR is synced to the cloud repo label Feb 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-ci 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.

4 participants