-
Notifications
You must be signed in to change notification settings - Fork 8
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 setup for sos report #64
Conversation
SOS Report needs a special file in the /etc/sos.extras.d/ folder to be able to collect the log files for the worker. Since it won't have a special plugin in sosreport due to RHEL 7 being in maintenance mode, we will be using this strategy to make sure that sosreport will be able to collect our logs. Signed-off-by: Rodolfo Olivieri <[email protected]>
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## main #64 +/- ##
==========================================
- Coverage 66.23% 64.84% -1.39%
==========================================
Files 5 5
Lines 311 330 +19
==========================================
+ Hits 206 214 +8
- Misses 91 99 +8
- Partials 14 17 +3
Flags with carried forward coverage won't be shown. Click here to find out more.
☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just few questions, I am good with overall implementation and test ! 👍
Signed-off-by: Rodolfo Olivieri <[email protected]>
* Update parameter for setupSosExtrasReport to match what is being passed to the function. * Include a comment to specify what is the expected format for the sosreport file Signed-off-by: Rodolfo Olivieri <[email protected]>
SOS Report needs a special file in the /etc/sos.extras.d/ folder to be able to collect the log files for the worker. Since it won't have a special plugin in sosreport due to RHEL 7 being in maintenance mode, we will be using this strategy to make sure that sosreport will be able to collect our logs.