[9.4](backport #49838) idxmgmt: remove redundant beatPaths parameter

[mergify]

Proposed commit message

idxmgmt: remove redundant beatPaths parameter

NewESClientHandler and NewFileClientHandler accepted both beat.Info
and *paths.Path. Now that Info carries Paths, drop the separate
argument and pass info.Paths to the downstream template loaders.

Checklist

• My code follows the style guidelines of this project
• I have commented my code, particularly in hard-to-understand areas
• I have made corresponding changes to the documentation
• I have made corresponding change to the default configuration files
• I have added tests that prove my fix is effective or that my feature works.
• I have added an entry in ./changelog/fragments using the changelog tool.

Disruptive User Impact

None. Internal API change only.

How to test this PR locally

Related issues

• Relates [beatreceiver] move paths object into Beat.Info #49803
• Blocked by Move Paths from beat.Beat to beat.Info #49836

  ---

  This is an automatic backport of pull request idxmgmt: remove redundant beatPaths parameter #49838 done by Mergify.

[github-actions]

🤖 GitHub comments

Just comment with:

• run docs-build : Re-trigger the docs validation. (use unformatted text in the comment!)
• /test : Run the Buildkite pipeline.

[elasticmachine]

Pinging @elastic/elastic-agent-data-plane (Team:Elastic-Agent-Data-Plane)

[mergify]

This pull request has not been merged yet. Could you please review and merge it @orestisfl? 🙏

[github-actions]

TL;DR

golangci-lint failed on all three runners because this backport branch includes code that violates current forbidigo rules in libbeat/cmd/instance/beat.go (logp.NewLogger, paths.Paths, and paths.InitPaths). Immediate action: backport the follow-up fix commit(s) that removed those forbidden usages (or apply the equivalent changes directly in this PR).

Remediation

• Update libbeat/cmd/instance/beat.go on this branch to remove the three forbidden calls:
  • logp.NewLogger at libbeat/cmd/instance/beat.go:188
  • paths.Paths at libbeat/cmd/instance/beat.go:769
  • paths.InitPaths at libbeat/cmd/instance/beat.go:1509
• Re-run the golangci-lint workflow after backporting those changes.

Investigation details

Root Cause

The workflow itself is healthy; lint fails due to code-level policy violations from forbidigo. The same three violations are reported consistently on ubuntu-latest, windows-latest, and macos-latest, so this is not runner-specific.

Evidence

• Workflow: https://github.com/elastic/beats/actions/runs/24982726999
• Job/step: lint (ubuntu-latest) → golangci-lint (also reproduced in lint (windows-latest) and lint (macos-latest))
• Key log excerpt:
  • libbeat/cmd/instance/beat.go:188:5: use of \logp.NewLogger` forbidden because "accept a *logp.Logger as a parameter instead of creating one with logp.NewLogger" (forbidigo)`
  • libbeat/cmd/instance/beat.go:769:17: use of \paths.Paths` forbidden because "use a per-beat *paths.Path instance instead of the global paths.Paths" (forbidigo)`
  • libbeat/cmd/instance/beat.go:1509:12: use of \paths.InitPaths` forbidden because "use a per-beat *paths.Path instance instead of the global paths.InitPaths" (forbidigo)`

Validation

• Local lint/tests not run in this environment; diagnosis is based on the failing workflow job logs.

Follow-up

If this PR is a clean backport of #49838, check whether the source branch got an additional post-merge lint-fix commit and backport that commit as well.

---

What is this? | From workflow: PR Actions Detective

Give us feedback! React with 🚀 if perfect, 👍 if helpful, 👎 if not.

[orestisfl]

Backport of #49838