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.

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

Fix test_prometheus_endpoint #32476

Merged
merged 1 commit into from
Dec 10, 2021

Conversation

vdimir
Copy link
Member

@vdimir vdimir commented Dec 9, 2021

Changelog category (leave one):

  • Not for changelog (changelog entry is not required)

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

Detailed description / Documentation draft:
...

@robot-clickhouse robot-clickhouse added the pr-not-for-changelog This PR should not be mentioned in the changelog label Dec 9, 2021
@@ -30,7 +30,7 @@ def parse_response_line(line):

if line.startswith("#"):
return {}
match = re.match('^([a-zA-Z_:][a-zA-Z0-9_:]+)(\{.*\})? (\d)', line)
match = re.match('^([a-zA-Z_:][a-zA-Z0-9_:]+)(\{.*\})? -?(\d)', line)
Copy link
Member

Choose a reason for hiding this comment

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

Ok. This is for negative numbers.

@vdimir
Copy link
Member Author

vdimir commented Dec 10, 2021

@Mergifyio rebase

@mergify
Copy link
Contributor

mergify bot commented Dec 10, 2021

rebase

✅ Branch has been successfully rebased

@vdimir vdimir force-pushed the fix_test_prometheus_endpoint branch from 59cc3e4 to a0488d2 Compare December 10, 2021 11:58
@vdimir vdimir merged commit 6b0e6dd into ClickHouse:master Dec 10, 2021
@vdimir vdimir deleted the fix_test_prometheus_endpoint branch December 10, 2021 11:59
robot-clickhouse pushed a commit that referenced this pull request Jan 22, 2022
robot-clickhouse pushed a commit that referenced this pull request Jan 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr-not-for-changelog This PR should not be mentioned in the changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants