Skip to content

[8.19] (backport #9732) chore: fix formatting issues in logp printf-style calls#9741

Merged
mauri870 merged 3 commits into8.19from
mergify/bp/8.19/pr-9732
Sep 4, 2025
Merged

[8.19] (backport #9732) chore: fix formatting issues in logp printf-style calls#9741
mauri870 merged 3 commits into8.19from
mergify/bp/8.19/pr-9732

Conversation

@mergify
Copy link
Contributor

@mergify mergify bot commented Sep 4, 2025

Proposed commit message

This PR fixes fatal go vet errors on calls to logp functions. Release v0.22.1 of elastic-agent-libs enabled proper go vet support for printf style calls in logp, and that uncovered many misuses of printing directives.

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.next.asciidoc or CHANGELOG-developer.next.asciidoc.

How to test

$ go get -u github.com/elastic/elastic-agent-libs@v0.22.1
$ go vet ./...

Related issues

(cherry picked from commit b646814)

# Conflicts:
#	internal/pkg/agent/application/upgrade/upgrade.go
@mergify mergify bot added backport conflicts There is a conflict in the backported pull request labels Sep 4, 2025
@mergify mergify bot requested a review from a team as a code owner September 4, 2025 13:57
@mergify mergify bot requested review from pchila and ycombinator and removed request for a team September 4, 2025 13:57
@mergify mergify bot added conflicts There is a conflict in the backported pull request backport labels Sep 4, 2025
@mergify
Copy link
Contributor Author

mergify bot commented Sep 4, 2025

Cherry-pick of b646814 has failed:

On branch mergify/bp/8.19/pr-9732
Your branch is up to date with 'origin/8.19'.

You are currently cherry-picking commit b646814d6.
  (fix conflicts and run "git cherry-pick --continue")
  (use "git cherry-pick --skip" to skip this patch)
  (use "git cherry-pick --abort" to cancel the cherry-pick operation)

Changes to be committed:
	modified:   internal/pkg/agent/application/actions/handlers/handler_action_policy_reassign.go
	modified:   internal/pkg/agent/application/upgrade/artifact/download/http/verify_backoff_rtt.go
	modified:   internal/pkg/agent/migration/migrate_config.go
	modified:   internal/pkg/composable/providers/kubernetes/kubernetes.go
	modified:   internal/pkg/config/loader.go
	modified:   internal/pkg/remote/client.go
	modified:   pkg/testing/ess/serverless_provisioner.go

Unmerged paths:
  (use "git add <file>..." to mark resolution)
	both modified:   internal/pkg/agent/application/upgrade/upgrade.go

To fix up this pull request, you can check it out locally. See documentation: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally

@mauri870 mauri870 enabled auto-merge (squash) September 4, 2025 18:51
@elastic-sonarqube
Copy link

@mauri870 mauri870 merged commit 7cc6ec7 into 8.19 Sep 4, 2025
19 checks passed
@mauri870 mauri870 deleted the mergify/bp/8.19/pr-9732 branch September 4, 2025 20:42
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

History

cc @mauri870

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport conflicts There is a conflict in the backported pull request skip-changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants