-
Notifications
You must be signed in to change notification settings - Fork 684
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
rhcos4: Add check for nousb kernel argument #6743
Conversation
Changes identified: Show detailsProfile moderate on rhcos4: Recommended tests to execute: |
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.
Since you are making the template parameter optional, you can update the documentation as well here: https://github.com/ComplianceAsCode/content/blame/master/docs/manual/developer/06_contributing_with_content.md#L1279
/test e2e-aws-ocp4-moderate |
/test e2e-aws-ocp4-moderate |
@ggbecker I think this is good to go, can you revisit this? |
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.
One more nitpick regarding documentation.
This adds a check that verifies if the nousb kernel argument is set. If it isn't it should generate an appropraite remediation. This also enhances the `coreos_kernel_option` template and the `oval_argument_value_in_line_criterion` and `oval_argument_value_in_line_test` macros to not require `arg_value` anymore. If this parameter is not given, it'll only use `arg_name`. Signed-off-by: Juan Antonio Osorio Robles <jaosorior@redhat.com>
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.
LGTM, thanks for addressing the requests. I pushed one typo fix and as soon the build finishes I'll merge it.
This adds a check that verifies if the nousb kernel argument is set. If
it isn't it should generate an appropraite remediation.
This also enhances the
coreos_kernel_option
template and theoval_argument_value_in_line_criterion
andoval_argument_value_in_line_test
macros to not requirearg_value
anymore. If this parameter is not given, it'll only use
arg_name
.Signed-off-by: Juan Antonio Osorio Robles jaosorior@redhat.com