Skip to content

docs(telegraf): Fix typo#18722

Merged
skartikey merged 1 commit intoinfluxdata:masterfrom
SAY-5:fix-telegraf-cmd-occured-comment-typo
Apr 27, 2026
Merged

docs(telegraf): Fix typo#18722
skartikey merged 1 commit intoinfluxdata:masterfrom
SAY-5:fix-telegraf-cmd-occured-comment-typo

Conversation

@SAY-5
Copy link
Copy Markdown
Contributor

@SAY-5 SAY-5 commented Apr 15, 2026 •
edited by srebhan
Loading

Summary

cmd/telegraf/telegraf.go had a comment misspelling occured. Line 191: // if a delete or create occured. → // if a delete or create occurred.

Comment-only change. No behavior, API, or logic change.

Checklist

Related issues

fix(cmd/telegraf): correct 'occured' -> 'occurred' in SIGHUP reload c…
c5606d5 
…omment

Doc comment inside the SIGHUP reload branch in cmd/telegraf/telegraf.go
explaining when the config-files watch list needs updating used
'occured'. Doc-only change; gofmt -l on the file is clean.
@telegraf-tiger
Copy link
Copy Markdown
Contributor

telegraf-tiger Bot commented Apr 15, 2026

Thanks so much for the pull request!
🤝 ✒️ Just a reminder that the CLA has not yet been signed, and we'll need it before merging. Please sign the CLA when you get a chance, then post a comment here saying !signed-cla

@telegraf-tiger telegraf-tiger Bot added the fix pr to fix corresponding bug label Apr 15, 2026
@telegraf-tiger
Copy link
Copy Markdown
Contributor

telegraf-tiger Bot commented Apr 15, 2026

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
Copy link
Copy Markdown
Member

srebhan commented Apr 16, 2026

@SAY-5 please restore the PR description template (especially the AI section) and sign the CLA as we cannot review your PR otherwise.

@srebhan srebhan self-assigned this Apr 16, 2026
@srebhan srebhan added the waiting for response waiting for response from contributor label Apr 16, 2026
@SAY-5
Copy link
Copy Markdown
Contributor Author

SAY-5 commented Apr 16, 2026

@srebhan restored the PR template with the AI disclosure filled in. CLA pending — I need to sign via browser; will do.

@telegraf-tiger telegraf-tiger Bot removed the waiting for response waiting for response from contributor label Apr 16, 2026
@srebhan srebhan changed the title fix(cmd/telegraf): correct 'occured' -> 'occurred' in SIGHUP reload comment docs(telegraf): Fix typo Apr 17, 2026
@telegraf-tiger telegraf-tiger Bot added the docs Issues related to Telegraf documentation and configuration descriptions label Apr 17, 2026
@srebhan srebhan added area/agent waiting for response waiting for response from contributor and removed fix pr to fix corresponding bug labels Apr 17, 2026
@srebhan
Copy link
Copy Markdown
Member

srebhan commented Apr 27, 2026

!signed-cla

@telegraf-tiger telegraf-tiger Bot removed the waiting for response waiting for response from contributor label Apr 27, 2026
srebhan
srebhan approved these changes Apr 27, 2026
View reviewed changes
Copy link
Copy Markdown
Member

@srebhan srebhan left a comment

Choose a reason for hiding this comment

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

Thanks @SAY-5!

@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 Apr 27, 2026
@srebhan srebhan assigned skartikey and unassigned srebhan Apr 27, 2026
@srebhan srebhan mentioned this pull request Apr 27, 2026
Closed
skartikey
skartikey approved these changes Apr 27, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@skartikey skartikey left a comment

Choose a reason for hiding this comment

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

@SAY-5 Thanks for the contribution!

@skartikey skartikey merged commit 956f3a1 into influxdata:master Apr 27, 2026
33 checks passed
@github-actions github-actions Bot added this to the v1.38.4 milestone Apr 27, 2026
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

Labels

area/agent docs Issues related to Telegraf documentation and configuration descriptions 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.38.4

Development

Successfully merging this pull request may close these issues.

3 participants

@SAY-5 @srebhan @skartikey