Open acald-creator opened 1 year ago
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.
This issue is being automatically closed due to inactivity. However, you may choose to reopen this issue.
bump
working on the logging part now
Checking in... it has been awhile since we've heard from you on this issue. Are you still working on it? Please let us know and please don't hesitate to contact a MeshMate or any other community member for assistance.
Be sure to join the community, if you haven't yet and please leave a :star: star on the project :smile:
@acald-creator Is this issue still open?
UPDATE:
slog
is currently available in Go1.21
, and we should make the effort to migrate to useslog
.Current Behavior
meshkit
useslogrus
for custom logging, and we created a custom logger that only returns certain fields such aswarn
info
error
debug
But we're lacking on the other fields that
logrus
has and we're using a mixture oflogrus
and our custom logger.meshkit
will benefit from having more fields to call from, when utilizing the custom logging tool inmeshery
. This will help migratemesheryctl
to use the custom logger, instead of callinglogrus
directly.For this purpose of tracking the migration, we will be migrating to use
slog
instead, which is slightly basic but gives us a bit more ability to customize logging and use more fields.Here are the fields that
slog
currently supportsInfo
InfoCtx
Warn
WarnCtx
Debug
DebugCtx
Error
ErrorCtx
There is no built-in support for formatting strings, but
slog
allows us to create our own fields.Desired Behavior
Opening this issue to track the progress.
Resources
Alternatives / Additional Context