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 wrong cloudsql/WARN/2023_003 MQL query #75

Conversation

m3adow
Copy link
Contributor

@m3adow m3adow commented Jul 13, 2023

This changes the MQL query for cloudsql/WARN/2023_003 from aggregate (which adds the value up to the thousands) to max which I assume was the intended way this query works.

Also, I fixed two small typos I found when I accidentally opened the wrong file.

Edit: Fixes #74

@ropeck
Copy link
Collaborator

ropeck commented Jul 29, 2023

Thank you for the pull request. Please check to be sure you have completed the contribution license agreement so we can merge your change into the repository

https://cla.developers.google.com/clas

@junggil
Copy link
Collaborator

junggil commented Jul 31, 2023

LGTM, thanks for the fix

@junggil junggil added the ready to pull Change can be pulled into Google's internal source repo label Jul 31, 2023
@copybara-service copybara-service bot merged commit 01878e5 into GoogleCloudPlatform:main Jul 31, 2023
4 checks passed
ropeck pushed a commit that referenced this pull request Feb 26, 2024
This changes the MQL query for cloudsql/WARN/2023_003 from `aggregate` (which adds the value up to the thousands) to `max` which I assume was the intended way this query works.

Also, I fixed two small typos I found when I accidentally opened the wrong file.

Edit: Fixes #74

Merge da4af2a into 620a060

Closes #75

FUTURE_COPYBARA_INTEGRATE_REVIEW=#75 from m3adow:fix/correct-2023_003-mql-query da4af2a
ORIGINAL_AUTHOR=Till Wiese <1872304+m3adow@users.noreply.github.com>
GitOrigin-RevId: b9bebbf
Change-Id: I4a206f4c8ede86c8f650ce8b7dbb0483b97ab870
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready to pull Change can be pulled into Google's internal source repo
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[cloudsql/WARN/2023_003]: MQL query wrong, leading to false positives
3 participants