Skip to content
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

Add logs to reports #87

Open
rafaeljusto opened this issue Jun 21, 2017 · 4 comments
Open

Add logs to reports #87

rafaeljusto opened this issue Jun 21, 2017 · 4 comments
Assignees
Milestone

Comments

@rafaeljusto
Copy link
Owner

It would be useful to have all logs from an action performed by toglacier in the report.

@rafaeljusto
Copy link
Owner Author

We could encapsulate the current logger into a reportLogger, that just save the messages in the report before calling the real logger.

@rafaeljusto rafaeljusto added this to the v3.2.0 milestone Jun 22, 2017
@rafaeljusto
Copy link
Owner Author

I'm a little concern about the verbosity in e-mail reports if we start adding all levels of logs. Maybe we could just add the warning level.

rafaeljusto added a commit that referenced this issue Jul 10, 2017
To make it easy to understand what is going on with the tool remotly, we are
now adding all log entries to the report.

See #87
rafaeljusto pushed a commit that referenced this issue Jul 30, 2017
@rafaeljusto
Copy link
Owner Author

  • Set the log scope for only one task
  • Add logs to report output (plain and html)
  • Make report log level configurable

@rafaeljusto
Copy link
Owner Author

To busy to work on this right now, so I will move this to a future release.

@rafaeljusto rafaeljusto modified the milestones: v4.0.0, v3.2.0 Aug 11, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant