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
Epic logging #1186
Merged
Merged
Epic logging #1186
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* refactor: remove Debug config option * feat: check if Debug is enabled for big payloads
* feat: add logrus.Fields to feature trace output * feat: remove logrus.Fields param from feature trace
* Combine multiple debug messages about devices * Move NFS error from debug to warn
* feat: add "Sampler" feature trace * refactor: use TraceSamplerStructureDetails for sampler traces
* added logging on discovery * fixed tests
* add integration_name field to executor logger * renaming configuration option
* feat: replace Fields by log.Entry in tracer * consider when fields is nil
* feat: add WithTrace fields The fields added with that method will only be displayed when the Trace level is enabled * refactor: move json logged payloads to trace fields * feat: remove unused helper function and apply suggestion
* refactor: move feature traces usages to log.Trace * refactor: remove feature flags package * refactor: remove FeatureFlags configuration option
* feat: add LogConfig structure for logging options * fix: check logfile definition for windows * feat: remove wrong documentation line * feat: add smart limit option * feat: apply typos suggestions
* feat: add logging filter configuration option Adds a new configuration option for logging to filter by fields. It also includes a logrus.Formatter implementation that performs this filtering. * feat: populate log config option * feat: use config.Log object to configure formatter
* add context for heartbeat logs * Heartbeat msg as warning. Refactor naming Co-authored-by: Ruben Ruiz de Gauna <rubenruizdegauna@gmail.com>
* docs: add log configuration option * docs: document with log differences
* feat: add include and exclude logging filters * added include/exclude log filtering options * fixed unit test * update code docs * fixed unsupported logging config * improved logging filtering * added more test * fixed failing tests * improve log filtering tests * fix databind * revert Co-authored-by: Roger Coll <rogercoll@protonmail.com>
rubenruizdegauna
approved these changes
May 27, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
No description provided.