-
Notifications
You must be signed in to change notification settings - Fork 122
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
internal/pkg/agent/application/coordinator: fix logging calls #3679
Conversation
Pinging @elastic/elastic-agent (Team:Elastic-Agent) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice catch!
buildkite test this |
@blakerouse Are you able to force merge this? The integration tests are failing for unrelated reasons. |
@Mergifyio update |
✅ Branch has been successfully updated |
SonarQube Quality Gate |
Merging, seems like the Windows tests aborted themselves without explanation, I do see the Linux tests passed.
|
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com> (cherry picked from commit 91c03aa)
…c#3679) (elastic#3774) Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com> (cherry picked from commit 91c03aa) Co-authored-by: Dan Kortschak <90160302+efd6@users.noreply.github.com> Co-authored-by: Pierre HILBERT <pierre.hilbert@elastic.co>
What does this PR do?
Fixes logging calls that expect to be able to interpret formatting verbs.
Why is it important?
Current code is incorrect.
Checklist
./changelog/fragments
using the changelog toolAuthor's Checklist
How to test this PR locally
Related issues
Use cases
Screenshots
Logs
Questions to ask yourself