Skip to content

feat(processors.execd): Add log prefixing - #17576

Merged
skartikey merged 1 commit into
influxdata:masterfrom
emilvonck:execd_processor_log
Sep 9, 2025
Merged

feat(processors.execd): Add log prefixing#17576
skartikey merged 1 commit into
influxdata:masterfrom
emilvonck:execd_processor_log

Conversation

@emilvonck

@emilvonck emilvonck commented Sep 7, 2025

Copy link
Copy Markdown
Contributor

Summary

This PR adds the same og prefixing functionality which inputs.execd has.

Checklist

  • No AI generated code was used in this PR

Related issues

resolves #17569

@telegraf-tiger telegraf-tiger Bot added area/execd Issues related to execd or plugins that would be better suited to be used through execd feat Improvement on an existing feature such as adding a new setting/mode to an existing plugin plugin/processor labels Sep 7, 2025
@telegraf-tiger

telegraf-tiger Bot commented Sep 7, 2025

Copy link
Copy Markdown
Contributor

Download PR build artifacts for linux_amd64.tar.gz, darwin_arm64.tar.gz, and windows_amd64.zip.
Downloads for additional architectures and packages are available below.

☺️ This pull request doesn't significantly change the Telegraf binary size (less than 1%)

📦 Click here to get additional PR build artifacts

Artifact URLs

. DEB . RPM . TAR . GZ . ZIP
[[amd64.deb aarch64.rpm darwin_amd64.tar.gz windows_amd64.zip] [arm64.deb armel.rpm darwin_arm64.tar.gz windows_arm64.zip] [armel.deb armv6hl.rpm freebsd_amd64.tar.gz windows_i386.zip] [armhf.deb i386.rpm freebsd_armv7.tar.gz ] [i386.deb ppc64le.rpm freebsd_i386.tar.gz ] [mips.deb riscv64.rpm linux_amd64.tar.gz ] [mipsel.deb s390x.rpm linux_arm64.tar.gz ] [ppc64el.deb x86_64.rpm linux_armel.tar.gz ] [riscv64.deb linux_armhf.tar.gz ] [s390x.deb linux_i386.tar.gz ] [ linux_mips.tar.gz ] [ linux_mipsel.tar.gz ] [ linux_ppc64le.tar.gz ] [ linux_riscv64.tar.gz ] [ linux_s390x.tar.gz ]]

srebhan
srebhan approved these changes Sep 8, 2025
View reviewed changes

@srebhan srebhan left a comment

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.

Nice addition @emilvonck! Thank you!

@srebhan srebhan added the ready for final review This pull request has been reviewed and/or tested by multiple users and is ready for a final review. label Sep 8, 2025
skartikey
skartikey approved these changes Sep 9, 2025
View reviewed changes

@skartikey skartikey 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.

@emilvonck Nice addition!

@skartikey
skartikey merged commit fe2f0eb into influxdata:master Sep 9, 2025
27 checks passed
@github-actions github-actions Bot added this to the v1.37.0 milestone Sep 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@skartikey skartikey skartikey approved these changes
@srebhan srebhan srebhan approved these changes

Assignees

@skartikey skartikey

@mstrandboge mstrandboge

Labels

area/execd Issues related to execd or plugins that would be better suited to be used through execd feat Improvement on an existing feature such as adding a new setting/mode to an existing plugin plugin/processor ready for final review This pull request has been reviewed and/or tested by multiple users and is ready for a final review.

Projects

None yet

Milestone

v1.37.0

Development

Successfully merging this pull request may close these issues.

feat (processors.execd): prefix logging

4 participants

@emilvonck @skartikey @srebhan @mstrandboge