Skip to content

[aws_elb] enable agentless deployment for aws elb metrics#18364

Merged
jakubgalecki0 merged 13 commits intoelastic:mainfrom
jakubgalecki0:aws_elb_agentless
Apr 17, 2026
Merged

[aws_elb] enable agentless deployment for aws elb metrics#18364
jakubgalecki0 merged 13 commits intoelastic:mainfrom
jakubgalecki0:aws_elb_agentless

Conversation

@jakubgalecki0
Copy link
Copy Markdown
Contributor

Proposed commit message

Enable agentless deployment for aws elb metrics

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

@jakubgalecki0 jakubgalecki0 requested review from a team as code owners April 13, 2026 09:56
@jakubgalecki0 jakubgalecki0 marked this pull request as draft April 13, 2026 10:26
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 15, 2026

Vale Linting Results

Summary: 2 warnings found

⚠️ Warnings (2)
File Line Rule Message
packages/aws/_dev/build/docs/elb.md 39 Elastic.Latinisms Latin terms and abbreviations are a common source of confusion. Use 'using' instead of 'via'.
packages/aws/docs/elb.md 39 Elastic.Latinisms Latin terms and abbreviations are a common source of confusion. Use 'using' instead of 'via'.

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.

@jakubgalecki0 jakubgalecki0 marked this pull request as ready for review April 15, 2026 12:53
@elastic-vault-github-plugin-prod
Copy link
Copy Markdown

🚀 Benchmarks report

To see the full report comment with /test benchmark fullreport

@andrewkroh andrewkroh added documentation Improvements or additions to documentation. Applied to PRs that modify *.md files. Team:obs-ds-hosted-services Observability Hosted Services team [elastic/obs-ds-hosted-services] labels Apr 15, 2026
Comment thread packages/aws/_dev/build/docs/elb.md Outdated

### Agentless Enabled Integration

Agentless integrations allow you to collect data without having to manage Elastic Agent in your cloud. They make manual agent deployment unnecessary, so you can focus on your data instead of the agent that collects it. For more information, refer to [Agentless integrations](https://www.elastic.co/guide/en/serverless/current/security-agentless-integrations.html) and the [Agentless integrations FAQ](https://www.elastic.co/guide/en/serverless/current/agentless-integration-troubleshooting.html).
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.

Suggested change
Agentless integrations allow you to collect data without having to manage Elastic Agent in your cloud. They make manual agent deployment unnecessary, so you can focus on your data instead of the agent that collects it. For more information, refer to [Agentless integrations](https://www.elastic.co/guide/en/serverless/current/security-agentless-integrations.html) and the [Agentless integrations FAQ](https://www.elastic.co/guide/en/serverless/current/agentless-integration-troubleshooting.html).
Agentless integrations allow you to collect data without having to manage Elastic Agent in your cloud. They make manual agent deployment unnecessary, so you can focus on your data instead of the agent that collects it. For more information, refer to [Agentless integrations](https://www.elastic.co/guide/en/serverless/current/security-agentless-integrations.html)

Copy link
Copy Markdown
Member

@ishleenk17 ishleenk17 Apr 15, 2026

Choose a reason for hiding this comment

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

@jakubgalecki0 : The placeement of this in README doesn't feel right.

## How do I deploy this integration?

Lets use this as example pls

Comment thread packages/aws/changelog.yml Outdated
@@ -1,4 +1,9 @@
# newer versions go on top
- version: "6.5.1"
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.

Suggested change
- version: "6.5.1"
- version: "6.6.0"

Since this is an enhancement rather than a bug fix, we should increment the minor version.

Copy link
Copy Markdown
Member

@ishleenk17 ishleenk17 left a comment

Choose a reason for hiding this comment

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

Looks good!

Copy link
Copy Markdown
Contributor

@kcreddy kcreddy left a comment

Choose a reason for hiding this comment

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

codeowner approval for aws integration.

@ishleenk17 ishleenk17 enabled auto-merge (squash) April 16, 2026 10:26
@ishleenk17 ishleenk17 disabled auto-merge April 16, 2026 10:46
Copy link
Copy Markdown
Contributor

@zmoog zmoog left a comment

Choose a reason for hiding this comment

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

LGTM.

Nit: the PR title mentions ELB metrics only, but this change affects all inputs, right?

@jakubgalecki0
Copy link
Copy Markdown
Contributor Author

@zmoog only metrics are supported in agentless deployment type

@elasticmachine
Copy link
Copy Markdown

💚 Build Succeeded

History

@jakubgalecki0 jakubgalecki0 merged commit dec3355 into elastic:main Apr 17, 2026
10 checks passed
@elastic-vault-github-plugin-prod
Copy link
Copy Markdown

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation. Applied to PRs that modify *.md files. Integration:aws AWS Team:obs-ds-hosted-services Observability Hosted Services team [elastic/obs-ds-hosted-services]

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants