Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

display "target" to log msg #797

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Brant-lzh
Copy link

Hello, I'd like to display the 'target' field in the logs for easier debugging.

OLD
ts=2023-12-19T02:53:31.516Z caller=exporter.go:152 level=error msg="Error pinging mysqld" err="Error 1045 (28000): Access denied for user 'testuser'@'127.0.0.1' (using password: YES)" ts=2023-12-19T02:53:35.076Z caller=exporter.go:152 level=error msg="Error pinging mysqld" err="Error 1045 (28000): Access denied for user 'testuser'@'127.0.0.1' (using password: YES)"

NEW
ts=2023-12-19T02:53:31.516Z caller=exporter.go:152 level=error msg="Error pinging mysqld" target=1.2.3.4:3306 err="Error 1045 (28000): Access denied for user 'testuser'@'127.0.0.1' (using password: YES)" ts=2023-12-19T02:53:35.076Z caller=exporter.go:152 level=error msg="Error pinging mysqld" target=1.2.3.5:3306 err="Error 1045 (28000): Access denied for user 'testuser'@'127.0.0.1' (using password: YES)"

@Brant-lzh Brant-lzh changed the title show "target" to log msg display "target" to log msg Dec 19, 2023
@SuperQ
Copy link
Member

SuperQ commented Aug 29, 2024

This needs a rebase.

@Brant-lzh
Copy link
Author

@SuperQ So, what should be done?

@SuperQ
Copy link
Member

SuperQ commented Aug 29, 2024

Signed-off-by: Brant <53347762+Brant-lzh@users.noreply.github.com>
@Brant-lzh Brant-lzh force-pushed the feature/add_log_msg branch from e6eacbc to 9a4dcba Compare August 31, 2024 15:15
@Brant-lzh
Copy link
Author

@SuperQ Done. LGTM !

@SuperQ
Copy link
Member

SuperQ commented Nov 28, 2024

Sorry, this needs to be updated for the new logging system.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants