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.

Sign up for GitHub

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

Jump to bottom

Dmesg change should cause the process execution to be re-run. #319

Open
vext01 opened this issue Jul 20, 2017 · 2 comments
Open

Dmesg change should cause the process execution to be re-run. #319

vext01 opened this issue Jul 20, 2017 · 2 comments
Labels
enhancement

Comments

@vext01
Copy link
Member

vext01 commented Jul 20, 2017

Dmesg change should cause the process execution to be re-run. Currently gets marked as E in manifest yielding no data.
@ltratt
Copy link
Member

ltratt commented Jul 20, 2017

Good idea.

In the long term, we'd probably want to log these errors so people can understand them afterwards. Not in the main output file, I guess, but in some new file? Anyway, one for the future...

@vext01
Copy link
Member Author

vext01 commented Jul 21, 2017

Most of the stuff we are interested in already goes into the log (bad amperf ratios and dmesg changes) thanks to log_and_mail(). Not sure if you had something more grand in mind, but I agree that now is probably not the time to look at this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ltratt @vext01