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

[AWS] Use aws.dimensions.DBInstanceIdentifier as database instance identifier in the RDS dashboard #6440

Conversation

zmoog
Copy link
Contributor

@zmoog zmoog commented Jun 1, 2023

What does this PR do?

Replace aws.rds.db_instance.identifier with aws.dimensions.DBInstanceIdentifier as the field to identify database instances in the RDS dashboard.

Why?

Previously we used aws.rds.db_instance.identifier as the field to identify database instances in the RDS dashboard.

However, since the CloudWatch metricset collects the aws.rds.db_instance.* as metadata calling the DescribeDBInstances API calls, transient problems with permissions or other configuration issues may prevent the dashboard from displaying data.

We decided to replace aws.rds.db_instance.identifier with aws.dimensions.DBInstanceIdentifier since CloudWatch returns this dimension with the metrics values.

Checklist

  • I have reviewed tips for building integrations and this pull request is aligned with them.
  • I have verified that all data streams collect metrics or logs.
  • I have added an entry to my package's changelog.yml file.
  • I have verified that Kibana version constraints are current according to guidelines.

Screenshots

CleanShot 2023-06-01 at 23 47 52@2x

@zmoog zmoog added enhancement New feature or request Team:Cloud-Monitoring Label for the Cloud Monitoring team labels Jun 1, 2023
@zmoog zmoog self-assigned this Jun 1, 2023
@zmoog zmoog marked this pull request as ready for review June 1, 2023 21:48
@zmoog zmoog requested review from a team as code owners June 1, 2023 21:48
@zmoog zmoog changed the title [AWS] Use aws.dimensions.DBInstanceIdentifier as database identifier instances in the RDS dashboard [AWS] Use aws.dimensions.DBInstanceIdentifier as database instance identifier in the RDS dashboard Jun 1, 2023
@elasticmachine
Copy link

elasticmachine commented Jun 1, 2023

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview preview

Expand to view the summary

Build stats

  • Start Time: 2023-06-05T23:05:58.305+0000

  • Duration: 53 min 39 sec

Test stats 🧪

Test Results
Failed 0
Passed 190
Skipped 4
Total 194

🤖 GitHub comments

Expand to view the GitHub comments

To re-run your PR in the CI, just comment with:

  • /test : Re-trigger the build.

@elasticmachine
Copy link

elasticmachine commented Jun 1, 2023

🌐 Coverage report

Name Metrics % (covered/total) Diff
Packages 100.0% (15/15) 💚
Files 93.75% (15/16) 👍 25.75
Classes 93.75% (15/16) 👍 25.75
Methods 85.921% (238/277) 👍 5.405
Lines 85.925% (7387/8597) 👍 3.057
Conditionals 100.0% (0/0) 💚

zmoog added 2 commits June 2, 2023 17:17
Previously we used `aws.rds.db_instance.identifier` as the field to
identify database instances in the RDS dashboard.

However, since the CloudWatch metricset collects this field as
metadata calling the DescribeDBInstances API [1] calls, transient
problems with permissions or configuration issues may prevent the
dashboard from displaying data.

We decided to replace `aws.rds.db_instance.identifier` with
`aws.dimensions.DBInstanceIdentifier` since CloudWatch returns this
dimension with the metrics values.

[1]: https://docs.aws.amazon.com/AmazonRDS/latest/APIReference/API_DescribeDBInstances.html
@zmoog zmoog force-pushed the zmoog/rds-dashboard-use-dbinstanceidentifier-dimension branch from ccc46f0 to 5c0f1e5 Compare June 2, 2023 15:21
@zmoog zmoog merged commit 5771166 into elastic:main Jun 6, 2023
3 checks passed
@zmoog zmoog deleted the zmoog/rds-dashboard-use-dbinstanceidentifier-dimension branch June 6, 2023 06:50
@elasticmachine
Copy link

Package aws - 1.40.2 containing this change is available at https://epr.elastic.co/search?package=aws

sodhikirti07 pushed a commit that referenced this pull request Jun 15, 2023
…identifier in the RDS dashboard (#6440)

* Use aws.dimensions.DBInstanceIdentifier as DB identifier

Previously we used `aws.rds.db_instance.identifier` as the field to
identify database instances in the RDS dashboard.

However, since the CloudWatch metricset collects this field as
metadata calling the DescribeDBInstances API [1] calls, transient
problems with permissions or configuration issues may prevent the
dashboard from displaying data.

We decided to replace `aws.rds.db_instance.identifier` with
`aws.dimensions.DBInstanceIdentifier` since CloudWatch returns this
dimension with the metrics values.

[1]: https://docs.aws.amazon.com/AmazonRDS/latest/APIReference/API_DescribeDBInstances.html
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request Team:Cloud-Monitoring Label for the Cloud Monitoring team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants