[9.3] (backport #13436) Replace deprecated LoggingWithTypedOutputs

[mergify]

What does this PR do?

LoggingWithTypedOutputs is deprecated in favor of the Local variant which returns a localized logger instance. Use the returned logger directly via Named() instead of re-reading from global state with logp.NewLogger().

Why is it important?

Clean up of deprecated endpoints. Elastic Agent is the only caller in Elastic: https://github.com/search?q=org%3Aelastic%20LoggingWithTypedOutputs&type=code

Checklist

• I have read and understood the pull request guidelines of this project.
• 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
• I have added an integration test or an E2E test

Disruptive User Impact

There shouldn't be any. I've tested locally and both the elastic-agent build from main and with my change seem to produce the same format of logs.

How to test this PR locally

./elastic-agent install --develop --force

and read the *.ndjson logs. Compare with main.

Related issues

• Relates to Remove global paths from logp file output elastic-agent-libs#397

  ---

  This is an automatic backport of pull request Replace deprecated LoggingWithTypedOutputs #13436 done by Mergify.

[elasticmachine]

Pinging @elastic/elastic-agent-control-plane (Team:Elastic-Agent-Control-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 pkg/core/logger/logger.go due to gosec rule G115 (int -> uint32 overflow-risk conversion). Update the file permission variable to use os.FileMode directly and rerun lint.

Remediation

• In pkg/core/logger/logger.go, replace the inferred int permissions variable with os.FileMode (for example: permissions := os.FileMode(0o600) and permissions = 0o660), then pass it directly to file.Permissions(permissions).
• Re-run mage check:lint (or the PR golangci-lint workflow) to confirm G115 is resolved.

Investigation details

Root Cause

The permissions literal is currently inferred as int and later converted in a way that triggers gosec G115 for potential integer overflow on conversion to an unsigned 32-bit mode representation.

Evidence

• Workflow: https://github.com/elastic/elastic-agent/actions/runs/24539908244
• Job/step: lint (ubuntu-latest) → golangci-lint
• Key log excerpt:
  • pkg/core/logger/logger.go:223:31: G115: integer overflow conversion int -> uint32 (gosec)
  • file.Permissions(os.FileMode(permissions)),

Validation

• Not run locally in this workflow (read-only Actions triage).

Follow-up

• macos-latest and windows-latest lint jobs were canceled after the Ubuntu failure; rerunning after this fix should exercise all matrix targets.

---

What is this? | From workflow: PR Actions Detective

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

[elasticmachine]

💚 Build Succeeded

• Buildkite Build
• Commit: d295444

History

• 💔 Build #36931 failed 10e12da

cc @orestisfl