ConsoleMsg is not set by open source or AIStor, we should remove this an… #5186
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
…d correctly show Message only
Community Contribution License
All community contributions in this pull request are licensed to the project maintainers
under the terms of the Apache 2 license.
By creating this pull request I represent that I have the right to license the
contributions to the project maintainers under the Apache 2 license.
Description
mc admin logs
is missing messages, which are being sent in the json payload but not being printedMotivation and Context
This results in the following:
Notice that the "message" field in the JSON contains the message, and the "ConsoleMsg" field is empty. MinIO and AIStor do not set this field. However,
mc
is still looking for it. The first commandmc admin logs local
showed lines for API, Time and Deployment but not the actual message.After this PR, it shows this:
How to test this PR?
mc admin logs
Types of changes
Checklist:
commit-id
orPR #
here)