-
Notifications
You must be signed in to change notification settings - Fork 27
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
Updated proposal; Add 2 new Linux Tests #88
Conversation
Modified original proposal. Removed lsmod and modprobe, replaced with single kernelmodule construct. 2nd test is the sestatus check.
Hi @wmunyan, why make selinux_status an EntityItemStringType instead of an EntityItemSEStatusModeType? The only possible values should be enabled and disabled, right? |
@solind I think what I'll do is add another enumeration for |
Created new enumeration type describing possible values for SELinux status (enabled, disabled) and associated `selinux_status` element to that new type in both definitions and system characteristics schemas.
Added a field for "policy_from_config_file"
Looks good to me. |
If this can be changed into a PR for the develop branch, @isimluk can make a consensus call so we can move forward and merge. |
Consensus call period elapsed. None of the feedback received was identified to be required to inform changes to this PR. This is good to merge. However, we may want to consider merging this first to the |
Hi @isimluk, it's not a suggestion. After the consensus call, the community process calls for merging into the development branch. |
Modified original proposal. Removed
lsmod
andmodprobe
, replaced with singlekernelmodule
construct.The 2nd test in the PR is a new construct, proposing the
sestatus_(test|object|state|item)
. This test allows implementations to collect and evaluate information stemming from the output of thesestatus
command.