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

recovery shell: print zerrors since the last command was executed #561

Merged
merged 1 commit into from
Dec 21, 2023

Conversation

zdykstra
Copy link
Member

@zdykstra zdykstra commented Dec 18, 2023

I'm not super happy with how print_kmsg_logs is shaking out, but there's only so much time I'm willing to spend chasing elegance.

This is modestly tested, and could likely use further refinement after people play with it.

Copy link
Member

@ahesford ahesford left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a good idea. Printing these messages in red might be nice, but isn't necessary.

@zdykstra zdykstra merged commit f9b3f70 into master Dec 21, 2023
3 checks passed
@zdykstra zdykstra deleted the error-error-on-the-wall branch December 21, 2023 19:25
@zdykstra
Copy link
Member Author

This is good enough for now. I'll think a bit more about how I want to refine print_kmsg_logs - but this simple solution is fine for now.

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

Successfully merging this pull request may close these issues.

2 participants