Skip to content

return 200 when replicas status is ok and verbose = 1#13722

Merged
alexey-milovidov merged 5 commits intoClickHouse:masterfrom
javisantana:fix/replicas_status_verbose
Aug 25, 2020
Merged

return 200 when replicas status is ok and verbose = 1#13722
alexey-milovidov merged 5 commits intoClickHouse:masterfrom
javisantana:fix/replicas_status_verbose

Conversation

@javisantana
Copy link
Copy Markdown
Contributor

@javisantana javisantana commented Aug 14, 2020

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en

A previous change made /replicas_status endpoint return 500 when verbose=1 even when the replication lag is 0

Changelog category (leave one):

  • Bug Fix

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
Fixes /replicas_status endpoint response status code when verbose=1

@robot-clickhouse robot-clickhouse added the pr-bugfix Pull request with bugfix, not backported by default label Aug 14, 2020
@qoega
Copy link
Copy Markdown
Member

qoega commented Aug 14, 2020

Can you provide integration test as well?

@abyss7 abyss7 self-assigned this Aug 18, 2020
@alexey-milovidov
Copy link
Copy Markdown
Member

Description check — Changelog entry is empty

Fixed.

@alexey-milovidov alexey-milovidov merged commit 75a3bbf into ClickHouse:master Aug 25, 2020
robot-clickhouse pushed a commit that referenced this pull request Aug 25, 2020
robot-clickhouse pushed a commit that referenced this pull request Aug 25, 2020
robot-clickhouse pushed a commit that referenced this pull request Aug 25, 2020
alexey-milovidov added a commit that referenced this pull request Aug 31, 2020
Backport #13722 to 20.5: return 200 when replicas status is ok and verbose = 1
alexey-milovidov added a commit that referenced this pull request Aug 31, 2020
Backport #13722 to 20.6: return 200 when replicas status is ok and verbose = 1
alexey-milovidov added a commit that referenced this pull request Aug 31, 2020
Backport #13722 to 20.7: return 200 when replicas status is ok and verbose = 1
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-bugfix Pull request with bugfix, not backported by default

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants