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

[Unified Instance Tagging]: Add cloud_metadata to DBM event payloads #14312

Merged
merged 4 commits into from
Apr 14, 2023

Conversation

jmeunier28
Copy link
Contributor

What does this PR do?

This PR adds cloud_metadata fields to all DBM event payloads, this adds necessary context in order link DBM event data with related cloud resource data in our backend.

The goal here is to allow customers to filter event data by a new universal tag database_instance, which will be applied across DBM and cloud telemetry.

@jmeunier28 jmeunier28 requested review from a team as code owners April 5, 2023 00:33
@ghost ghost added the integration/mysql label Apr 5, 2023
@github-actions
Copy link

github-actions bot commented Apr 5, 2023

The validations job has failed; please review the Files changed tab for possible suggestions to resolve.

@github-actions
Copy link

github-actions bot commented Apr 5, 2023

Test Results

     20 files       20 suites   21m 49s ⏱️
   126 tests    126 ✔️   0 💤 0
1 260 runs  1 248 ✔️ 12 💤 0

Results for commit 4f3435e.

♻️ This comment has been updated with latest results.

iliakur
iliakur previously approved these changes Apr 6, 2023
@alexandre-normand alexandre-normand dismissed stale reviews from iliakur and themself via c224454 April 12, 2023 16:20
@ghost ghost added the integration/postgres label Apr 12, 2023
@github-actions
Copy link

The validations job has failed; please review the Files changed tab for possible suggestions to resolve.

@codecov
Copy link

codecov bot commented Apr 12, 2023

Codecov Report

Merging #14312 (4f3435e) into master (4e56d12) will decrease coverage by 0.13%.
The diff coverage is n/a.

Flag Coverage Δ
mysql 87.78% <ø> (+0.41%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

@yzhan289 yzhan289 merged commit 9ba0a50 into master Apr 14, 2023
@yzhan289 yzhan289 deleted the jmeunier/DBM-2001 branch April 14, 2023 13:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants