Skip to content

[aws_{ec2,rds,elb_metrics,sqs,ecs,lambda}_otel] Add initial packages for AWS metrics OTEL - #19462

Merged
mrg-elastic merged 20 commits into
elastic:mainfrom
mrg-elastic:aws_otel
Jun 17, 2026
Merged

[aws_{ec2,rds,elb_metrics,sqs,ecs,lambda}_otel] Add initial packages for AWS metrics OTEL#19462
mrg-elastic merged 20 commits into
elastic:mainfrom
mrg-elastic:aws_otel

Conversation

@mrg-elastic

@mrg-elastic mrg-elastic commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

Overview

Adds initial aws_{service}_otel packages with dashboards for metrics: EC2, RDS, ELB, SQS, ECS, Lambda.

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.
  • I have verified that any added dashboard complies with Kibana's Dashboard good practices

Author's Checklist

  • [ ]

How to test this PR locally

Related issues

Screenshots

Comment thread packages/aws_ec2_otel/changelog.yml Outdated
@github-actions

github-actions Bot commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

Elastic Docs Style Checker (Vale)

Summary: 6 suggestions found

💡 Suggestions (6): Optional style improvements. Apply when helpful.
File Line Rule Message
packages/aws_ec2_otel/changelog.yml 1 Elastic.Versions Use 'later versions' instead of 'newer versions' when referring to versions.
packages/aws_ecs_otel/changelog.yml 1 Elastic.Versions Use 'later versions' instead of 'newer versions' when referring to versions.
packages/aws_elb_metrics_otel/changelog.yml 1 Elastic.Versions Use 'later versions' instead of 'newer versions' when referring to versions.
packages/aws_lambda_otel/changelog.yml 1 Elastic.Versions Use 'later versions' instead of 'newer versions' when referring to versions.
packages/aws_rds_otel/changelog.yml 1 Elastic.Versions Use 'later versions' instead of 'newer versions' when referring to versions.
packages/aws_sqs_otel/changelog.yml 1 Elastic.Versions Use 'later versions' instead of 'newer versions' when referring to versions.

The Vale linter checks documentation changes against the Elastic Docs style guide. To use Vale locally or report issues, refer to Elastic style guide for Vale.

@andrewkroh andrewkroh added documentation Improvements or additions to documentation. Applied to PRs that modify *.md files. New Integration Issue or pull request for creating a new integration package. labels Jun 9, 2026
@github-actions

This comment has been minimized.

@mykola-elastic mykola-elastic changed the title [aws_otel] Add initial package Jun 11, 2026
@github-actions

This comment has been minimized.

@mykola-elastic
mykola-elastic marked this pull request as ready for review June 11, 2026 18:28
@mykola-elastic
mykola-elastic requested a review from a team as a code owner June 11, 2026 18:28
@github-actions

Copy link
Copy Markdown
Contributor

TL;DR

Buildkite is failing for two concrete metadata checks: six new package changelog links still point to /pull/999, and the six new aws_*_otel packages are missing matching package-path entries in .github/CODEOWNERS.

Remediation

  • Update these files to https://github.com/elastic/integrations/pull/19462 at link: line 6:
    • packages/aws_ec2_otel/changelog.yml
    • packages/aws_ecs_otel/changelog.yml
    • packages/aws_elb_metrics_otel/changelog.yml
    • packages/aws_lambda_otel/changelog.yml
    • packages/aws_rds_otel/changelog.yml
    • packages/aws_sqs_otel/changelog.yml
  • Add CODEOWNERS entries for the new packages (same owner already used elsewhere):
    • /packages/aws_ec2_otel @elastic/obs-infraobs-integrations``
    • /packages/aws_ecs_otel @elastic/obs-infraobs-integrations``
    • /packages/aws_elb_metrics_otel @elastic/obs-infraobs-integrations``
    • /packages/aws_lambda_otel @elastic/obs-infraobs-integrations``
    • /packages/aws_rds_otel @elastic/obs-infraobs-integrations``
    • /packages/aws_sqs_otel @elastic/obs-infraobs-integrations``
  • Re-run:
    • .buildkite/scripts/check_changelog_entries.sh
    • .buildkite/scripts/check_sources.sh
Investigation details

Root Cause

  1. Check changelog PR links fails because the new package changelogs still use a placeholder PR link (/pull/999) instead of this PR (/pull/19462).
  2. Check go sources fails in dev/codeowners.Check because package manifests declare owner elastic/obs-infraobs-integrations, but there are no package-specific CODEOWNERS rules for the six newly added package paths.

Evidence

  • Build: https://buildkite.com/elastic/integrations/builds/44420
  • Job/step: :scroll: Check changelog PR links
    • Log shows expected link https://github.com/elastic/integrations/pull/19462
    • Log reports mismatches in the six files above, each with ERROR: unexpected link: 'https://github.com/elastic/integrations/pull/999'
  • Job/step: :white_check_mark: Check go sources
    • Log: Error: ... error checking manifest 'packages/aws_ec2_otel': owner "elastic/obs-infraobs-integrations" ... is not in ".github/CODEOWNERS"
  • PR branch file evidence:
    • packages/aws_ec2_otel/changelog.yml:6 (and same line in the other five): link: https://github.com/elastic/integrations/pull/999
    • packages/aws_ec2_otel/manifest.yml:30 (and same line in the other five): github: elastic/obs-infraobs-integrations
    • .github/CODEOWNERS has /packages/aws_otel @elastic/obs-infraobs-integrations`` near line 618, but no entries for /packages/aws_ec2_otel, `/packages/aws_ecs_otel`, `/packages/aws_elb_metrics_otel`, `/packages/aws_lambda_otel`, `/packages/aws_rds_otel`, `/packages/aws_sqs_otel`.

Verification

  • Not run locally in this investigation; conclusions are based on Buildkite failure logs plus direct inspection of the PR branch content.

Follow-up

  • After updating links and CODEOWNERS entries, trigger a rebuild to confirm both failing checks are green.

Note

🔒 Integrity filter blocked 2 items

The following items were blocked because they don't meet the GitHub integrity level.

To allow these resources, lower min-integrity in your GitHub frontmatter:

tools:
  github:
    min-integrity: approved  # merged | approved | unapproved | none

What is this? | From workflow: PR Buildkite Detective

Give us feedback! React with 🚀 if perfect, 👍 if helpful, 👎 if not.

@ishleenk17

Copy link
Copy Markdown
Member

@mykola-elastic : Can we have separate PR's for each of the content package. Easier to review and keep merging the CP's service wise.
Else we will be stuck for longer.

Also, currently I see a lot of inconsistencies with the datastream used, package name etc. I assume this is the draft version.

Comment thread packages/aws_ec2_otel/docs/README.md Outdated
Comment thread packages/aws_ec2_otel/manifest.yml Outdated
@mykola-elastic

Copy link
Copy Markdown
Contributor

/test

Comment thread .github/CODEOWNERS
/packages/aws_bedrock/data_stream/runtime @elastic/obs-infraobs-integrations
/packages/aws_billing @elastic/obs-infraobs-integrations
/packages/aws_cloudtrail_otel @elastic/obs-infraobs-integrations
/packages/aws_ec2_otel @elastic/obs-infraobs-integrations

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Which naming are we following here ?
Since I suppose we have separate content packages for say ec2 logs and ec2 metrics. So maybe the content package names should also depict that difference.

@lalit-satapathy lalit-satapathy left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code owner approval for first cut

@elastic-vault-github-plugin-prod

Copy link
Copy Markdown
Contributor

✅ All changelog entries have the correct PR link.

@elasticmachine

Copy link
Copy Markdown

💚 Build Succeeded

History

cc @mykola-elastic @mrg-elastic

@mrg-elastic
mrg-elastic merged commit c0d6549 into elastic:main Jun 17, 2026
11 checks passed
@andrewkroh andrewkroh added the dashboard Relates to a Kibana dashboard bug, enhancement, or modification. label Jun 17, 2026
@elastic-vault-github-plugin-prod

Copy link
Copy Markdown
Contributor

Package aws_ec2_otel - 0.1.0 containing this change is available at https://epr.elastic.co/package/aws_ec2_otel/0.1.0/

@elastic-vault-github-plugin-prod

Copy link
Copy Markdown
Contributor

Package aws_ecs_otel - 0.1.0 containing this change is available at https://epr.elastic.co/package/aws_ecs_otel/0.1.0/

@elastic-vault-github-plugin-prod

Copy link
Copy Markdown
Contributor

Package aws_elb_metrics_otel - 0.1.0 containing this change is available at https://epr.elastic.co/package/aws_elb_metrics_otel/0.1.0/

@elastic-vault-github-plugin-prod

Copy link
Copy Markdown
Contributor

Package aws_lambda_otel - 0.1.0 containing this change is available at https://epr.elastic.co/package/aws_lambda_otel/0.1.0/

@elastic-vault-github-plugin-prod

Copy link
Copy Markdown
Contributor

Package aws_rds_otel - 0.1.0 containing this change is available at https://epr.elastic.co/package/aws_rds_otel/0.1.0/

@elastic-vault-github-plugin-prod

Copy link
Copy Markdown
Contributor

Package aws_sqs_otel - 0.1.0 containing this change is available at https://epr.elastic.co/package/aws_sqs_otel/0.1.0/

@andrewkroh andrewkroh added Integration:aws_lambda_otel AWS Lambda Metrics OpenTelemetry Assets Integration:aws_elb_metrics_otel AWS ELB Metrics OpenTelemetry Assets Integration:aws_rds_otel AWS RDS Metrics OpenTelemetry Assets Integration:aws_ecs_otel AWS ECS Metrics OpenTelemetry Assets Integration:aws_ec2_otel AWS EC2 Metrics OpenTelemetry Assets Integration:aws_sqs_otel AWS SQS Metrics OpenTelemetry Assets labels Jun 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dashboard Relates to a Kibana dashboard bug, enhancement, or modification. documentation Improvements or additions to documentation. Applied to PRs that modify *.md files. Integration:aws_ec2_otel AWS EC2 Metrics OpenTelemetry Assets Integration:aws_ecs_otel AWS ECS Metrics OpenTelemetry Assets Integration:aws_elb_metrics_otel AWS ELB Metrics OpenTelemetry Assets Integration:aws_lambda_otel AWS Lambda Metrics OpenTelemetry Assets Integration:aws_rds_otel AWS RDS Metrics OpenTelemetry Assets Integration:aws_sqs_otel AWS SQS Metrics OpenTelemetry Assets New Integration Issue or pull request for creating a new integration package.

6 participants