Skip to content

telegraf: update to 1.37.0#28043

Merged
hnyman merged 2 commits intoopenwrt:masterfrom
niklasthorild:telegraf-bump
Dec 9, 2025
Merged

telegraf: update to 1.37.0#28043
hnyman merged 2 commits intoopenwrt:masterfrom
niklasthorild:telegraf-bump

Conversation

@niklasthorild
Copy link
Contributor

@niklasthorild niklasthorild commented Dec 9, 2025

📦 Package Details

Maintainer: me

Description:

  • Update Telegraf to v1.37.0
  • Set home environment variable in uci-defaults script
    • This is to avoid telegraf trying to call getent when generating the config, a package that's not installed by default in OpenWRT. It would produce the following non-critical error: ERROR : Couldn't find configuration directory nor home directory: exec: "getent": executable file not found in $PATH"

🧪 Run Testing Details

  • OpenWrt Version: SNAPSHOT
  • OpenWrt Target/Subtarget: x86/64
  • OpenWrt Device: LXC container

✅ Formalities

  • I have reviewed the CONTRIBUTING.md file for detailed contributing guidelines.

- Update Telegraf to v1.37.0

Signed-off-by: Niklas Thorild <niklas@thorild.se>
@hnyman
Copy link
Contributor

hnyman commented Dec 9, 2025

HOME is defined in the standard /etc/profile, so I wonder if your export change would possibly change that for some other process, too.

HOME=$(grep -E "^${USER:-root}:" /etc/passwd | cut -d ":" -f 6)
HOME=${HOME:-/root}
export HOME

So, do you need to export the HOME variable?
(possibly it would stay for more permanent usage by other scripts launched from the same shell. Not quite sure.)

I think that it would be enough to declare it in the uci-defaults context where telegraf config is immediately launched after defining the variable. In style of:
HOME=/root /usr/bin/telegraf config create > /etc/telegraf.conf

avoid a non-critical error caused by getent not being installed

Signed-off-by: Niklas Thorild <niklas@thorild.se>
@niklasthorild
Copy link
Contributor Author

You're totally right. The change you suggested has been tested and applied.

@hnyman hnyman merged commit aecb53a into openwrt:master Dec 9, 2025
13 checks passed
@niklasthorild niklasthorild deleted the telegraf-bump branch December 10, 2025 15:19
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.

2 participants