[8.19](backport #45944) chore: fix formatting issues in logp printf-style calls#46124
[8.19](backport #45944) chore: fix formatting issues in logp printf-style calls#46124pierrehilbert merged 13 commits into8.19from
Conversation
* fix go vet errors for logp formatting issues * chore: fix formatting for logp calls * revert elastic-agent-libs bump * Apply suggestions from code review Co-authored-by: Tiago Queiroz <me@tiago.life> --------- Co-authored-by: Tiago Queiroz <me@tiago.life> Co-authored-by: Blake Rouse <blake.rouse@elastic.co> (cherry picked from commit a5be2a8) # Conflicts: # metricbeat/module/kubernetes/util/kubernetes.go # x-pack/filebeat/input/awss3/sqs_s3_event.go
|
Cherry-pick of a5be2a8 has failed: 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 |
|
Pinging @elastic/elastic-agent-data-plane (Team:Elastic-Agent-Data-Plane) |
|
This pull request is now in conflicts. Could you fix it? 🙏 |
|
/test |
|
This pull request has not been merged yet. Could you please review and merge it @mauri870? 🙏 |
|
/test |
1 similar comment
|
/test |
|
@mauri870 there is a known issue with 8.19 CI - which is that if your PR consists of changes made to both |
|
This pull request has not been merged yet. Could you please review and merge it @mauri870? 🙏 |
Aha, thanks! I missed your comment at first and was digging into why this was failing. I’ll add it to my notes and ping them tomorrow since they’re out today. |
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
CHANGELOG.next.asciidocorCHANGELOG-developer.next.asciidoc.How to test
Related issues
This is an automatic backport of pull request #45944 done by [Mergify](https://mergify.com).