Skip to content

Use fluent-package paths instead of td-agent paths in configuration examples - #708

Merged
kenhys merged 1 commit into
fluent:1.0from
Watson1978:fix-td-agent-paths-in-examples
Oct 8, 2026
Merged

kenhys merged 1 commit into
fluent:1.0from
Watson1978:fix-td-agent-paths-in-examples

Conversation

@Watson1978

Copy link
Copy Markdown
Contributor

td-agent has reached EOL, but several configuration examples still used /var/log/td-agent and C:\opt\td-agent. On Linux, fluent-package runs as a non-root user, so it cannot create /var/log/td-agent and a copied in_tail example fails to start with a permission error.

This changes those paths to /var/log/fluent and C:\opt\fluent, which fluent-package uses. Other td-agent mentions in the text are left for follow-up PRs.

🤖 Generated with Claude Code

td-agent has reached EOL, but several examples still pointed at
/var/log/td-agent and C:\opt\td-agent. fluent-package uses
/var/log/fluent and C:\opt\fluent instead. On Linux, the service runs
as a non-root user and cannot create /var/log/td-agent, so a copied
in_tail example fails to start with a permission error.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Signed-off-by: Shizuo Fujita <fujita@clear-code.com>
@netlify

netlify Bot commented Oct 8, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for fluentd-docs ready!

Name Link
🔨 Latest commit 5bdda7a
🔍 Latest deploy log https://app.netlify.com/projects/fluentd-docs/deploys/6ac6ed72ce3b260008e04592
😎 Deploy Preview https://deploy-preview-708--fluentd-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@Watson1978
Watson1978 requested a review from kenhys October 8, 2026 01:12
@kenhys
kenhys merged commit 24ba6bc into fluent:1.0 Oct 8, 2026
6 checks passed
@Watson1978
Watson1978 deleted the fix-td-agent-paths-in-examples branch October 8, 2026 02:14
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