Skip to content

[otel] add awslogsencodingextension#11107

Merged
ishleenk17 merged 5 commits intoelastic:mainfrom
mykola-elastic:edot-add-awslogsencodingextension-again
Nov 12, 2025
Merged

[otel] add awslogsencodingextension#11107
ishleenk17 merged 5 commits intoelastic:mainfrom
mykola-elastic:edot-add-awslogsencodingextension-again

Conversation

@mykola-elastic
Copy link
Contributor

@mykola-elastic mykola-elastic commented Nov 10, 2025

What does this PR do?

Adds awslogsencodingextension to EDOT collector. Redo of #11078 (due to severe merge conflicts - EDOT got moved into a separate package #10922)

Why is it important?

awss3receiver (added in #10515) is quite useless without awslogsencodingextension which unmarshals logs encoded in formats produced by AWS services.

Adding awslogsencodingextension would enable us to create input otel package for AWS VPC Flow Logs, ELB Access Logs, Cloudtrail logs, etc.

cc @ishleenk17, @tommyers-elastic

Checklist

  • I have read and understood the pull request guidelines of this project.
  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have made corresponding change to the default configuration files
  • I have added tests that prove my fix is effective or that my feature works
  • I have added an entry in ./changelog/fragments using the changelog tool
  • I have added an integration test or an E2E test
@mergify
Copy link
Contributor

mergify bot commented Nov 10, 2025

This pull request does not have a backport label. Could you fix it @mykola-elastic? 🙏
To fixup this pull request, you need to add the backport labels for the needed
branches, such as:

  • backport-./d./d is the label that automatically backports to the 8./d branch. /d is the digit
  • backport-active-all is the label that automatically backports to all active branches.
  • backport-active-8 is the label that automatically backports to all active minor branches for the 8 major.
  • backport-active-9 is the label that automatically backports to all active minor branches for the 9 major.
@mykola-elastic mykola-elastic marked this pull request as ready for review November 10, 2025 13:54
@mykola-elastic mykola-elastic requested a review from a team as a code owner November 10, 2025 13:54
@pierrehilbert pierrehilbert added the Team:Elastic-Agent-Control-Plane Label for the Agent Control Plane team label Nov 10, 2025
@elasticmachine
Copy link
Contributor

Pinging @elastic/elastic-agent-control-plane (Team:Elastic-Agent-Control-Plane)

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

cc @mykola-elastic

@mykola-elastic
Copy link
Contributor Author

@swiatekm thanks for the review. Feel free to merge (I don't have the permission)

@ishleenk17 ishleenk17 merged commit 0657336 into elastic:main Nov 12, 2025
21 checks passed
@ebeahan
Copy link
Member

ebeahan commented Nov 12, 2025

@mykola-elastic confirming - do we need to backport this change? Currently it'll only come out in 9.3 by going into main.

@mykola-elastic
Copy link
Contributor Author

@ebeahan thanks, no backporting. awss3receiver (merged) will also only be available beginning 9.3

hayotbisonai pushed a commit to hayotbisonai/elastic-agent that referenced this pull request Nov 23, 2025
* [otel] add awslogsencodingextension

* add changelog

* mage notice

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

Labels

Team:Elastic-Agent-Control-Plane Label for the Agent Control Plane team

6 participants