Skip to content

Update cloudwatch logs receiver with latest upstream changes #3909

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

Merged
merged 1 commit into from
Jul 7, 2025

Conversation

boernd
Copy link
Contributor

@boernd boernd commented Jun 26, 2025

PR Description

Starting with Alloy 1.9.x the cloudwatch logs receiver starts reading from the start which causes duplicate logs in case of restarts.

  • Add start_from arg
  • Add storage capabilities

Relates-To: open-telemetry/opentelemetry-collector-contrib#39007

PR Checklist

  • CHANGELOG.md updated
  • Documentation added
  • Tests updated
  • Config converters updated

@boernd boernd requested review from clayton-cornell and a team as code owners June 26, 2025 14:58
@CLAassistant
Copy link

CLAassistant commented Jun 26, 2025

CLA assistant check
All committers have signed the CLA.

@ptodev ptodev self-assigned this Jul 4, 2025
Copy link
Contributor

@ptodev ptodev left a comment

Choose a reason for hiding this comment

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

Thank you, looks good! Could you please just rebase the PR?

Starting with Alloy 1.9.x the cloudwatch logs receiver starts reading
from the start which causes duplicate logs in case of restarts.

* Add `start_from` arg
* Add storage capabilities

Relates-To: open-telemetry/opentelemetry-collector-contrib#39007
@boernd boernd force-pushed the cw-receiver-start-from branch from b64c3ad to 4abb86f Compare July 7, 2025 15:43
@boernd
Copy link
Contributor Author

boernd commented Jul 7, 2025

@ptodev Done

@ptodev ptodev merged commit c6718de into grafana:main Jul 7, 2025
36 of 37 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants