Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Disable "Too many redirects" error retry #59099

Merged
merged 3 commits into from Jan 24, 2024
Merged

Disable "Too many redirects" error retry #59099

merged 3 commits into from Jan 24, 2024

Conversation

skyoct
Copy link
Contributor

@skyoct skyoct commented Jan 23, 2024

Changelog category (leave one):

  • Bug Fix (user-visible misbehavior in an official stable release)

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

fixs: #58967

@skyoct skyoct changed the title Fix redirect retry Fix "Too many redirects" retry Jan 23, 2024
@skyoct skyoct changed the title Fix "Too many redirects" retry Disable "Too many redirects" error retry Jan 23, 2024
@Avogar Avogar self-assigned this Jan 23, 2024
@Avogar Avogar added the can be tested Allows running workflows for external contributors label Jan 23, 2024
@robot-ch-test-poll3 robot-ch-test-poll3 added the pr-bugfix Pull request with bugfix, not backported by default label Jan 23, 2024
@robot-ch-test-poll3
Copy link
Contributor

robot-ch-test-poll3 commented Jan 23, 2024 •
edited by robot-ch-test-poll1

This is an automated comment for commit 718064b 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
CI runningA meta-check that indicates the running CI. Normally, it's in success or pending state. The failed status indicates some problems with the PR⏳ pending

@Avogar Avogar merged commit 6c0615d into ClickHouse:master Jan 24, 2024
2 of 4 checks passed
@robot-ch-test-poll1 robot-ch-test-poll1 added the pr-synced-to-cloud The PR is synced to the cloud repo label Jan 24, 2024
@skyoct skyoct deleted the fix/fix_mult_redirect branch January 24, 2024 12:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tavplubix tavplubix tavplubix left review comments

@Avogar Avogar Avogar approved these changes

Assignees

@Avogar Avogar

Labels
can be tested Allows running workflows for external contributors pr-bugfix Pull request with bugfix, not backported by default 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.

None yet
5 participants
@skyoct @robot-ch-test-poll3 @tavplubix @Avogar @robot-ch-test-poll1