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 #14313

Merged
merged 1 commit into from
Apr 5, 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.

@codecov
Copy link

codecov bot commented Apr 5, 2023

Codecov Report

Merging #14313 (b9e397f) into master (4e56d12) will increase coverage by 0.00%.
The diff coverage is n/a.

Flag Coverage Δ
postgres 92.24% <ø> (+0.25%) ⬆️

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

@github-actions
Copy link

github-actions bot commented Apr 5, 2023

Test Results

     28 files   -      545       28 suites   - 545   46m 39s ⏱️ - 4h 24m 44s
   187 tests  -   5 102     186 ✔️  -   4 938    1 💤  -      79  0  - 10 
2 632 runs   - 12 282  2 548 ✔️  - 10 221  84 💤  - 1 976  0  - 10 

Results for commit b9e397f. ± Comparison against base commit 4e56d12.

This pull request removes 5102 tests.
active_directory.tests.test_active_directory ‑ test_basic_check
active_directory.tests.test_unit ‑ test
activemq.tests.test_check ‑ test_activemq_metrics
activemq.tests.test_check ‑ test_artemis_metrics
activemq_xml.tests.test_activemq_xml ‑ test_e2e
activemq_xml.tests.test_activemq_xml ‑ test_integration
aerospike.tests.test_aerospike ‑ test_check
aerospike.tests.test_aerospike ‑ test_e2e
aerospike.tests.test_aerospike ‑ test_metrics_warning
aerospike.tests.test_aerospike ‑ test_openmetrics_e2e
…

@jmeunier28 jmeunier28 merged commit 951c259 into master Apr 5, 2023
@jmeunier28 jmeunier28 deleted the jmeunier/DBM-2001-postgres branch April 5, 2023 19:20
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.

2 participants