Skip to content

docs(telegraf): Fix typo#18722

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

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

Conversation

@SAY-5

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

Copy link
Copy Markdown
Contributor

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

telegraf-tiger Bot commented Apr 15, 2026

Copy link
Copy Markdown
Contributor

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

telegraf-tiger Bot commented Apr 15, 2026

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 commented Apr 16, 2026

Copy link
Copy Markdown
Member

@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

SAY-5 commented Apr 16, 2026

Copy link
Copy Markdown
Contributor Author

@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

srebhan commented Apr 27, 2026

Copy link
Copy Markdown
Member

!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

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

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

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

@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
srebhan pushed a commit that referenced this pull request May 11, 2026
@SAY-5 @srebhan
docs(telegraf): Fix typo (#18722)
17f2ec3 
Co-authored-by: SAY-5 <saiasish.cnp@gmail.com>
(cherry picked from commit 956f3a1)
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