Skip to content

Revert "Add test for Service Ping generated queries for Snowflake"

Piotr Skorupa requested to merge revert-50e842fa into master

What does this MR do and why?

This reverts !82757 (merged), which didn't account for developers with an active license. When they try regenerating the fixture file using the provided rake task, it updates the fixture with their license information.

These metrics should be ignored and not added to the fixture file.

I'm asking for a revert to prevent creating problems for team members adding or changing database metrics before a fix is made. After that, a fixed implementation can be reintroduced.

Screenshots or screen recordings

These are strongly recommended to assist reviewers and reduce the time to merge your change.

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Piotr Skorupa

Merge request reports