-
Notifications
You must be signed in to change notification settings - Fork 698
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
SLE add ability to configure emergency via dropin #10482
SLE add ability to configure emergency via dropin #10482
Conversation
This datastream diff is auto generated by the check Click here to see the full diffbash remediation for rule 'xccdf_org.ssgproject.content_rule_require_emergency_target_auth' differs.
--- xccdf_org.ssgproject.content_rule_require_emergency_target_auth
+++ xccdf_org.ssgproject.content_rule_require_emergency_target_auth
@@ -3,7 +3,9 @@
service_file="/usr/lib/systemd/system/emergency.service"
+
sulogin="/usr/lib/systemd/systemd-sulogin-shell emergency"
+
if grep "^ExecStart=.*" "$service_file" ; then
sed -i "s%^ExecStart=.*%ExecStart=-$sulogin%" "$service_file" |
d763dda
to
e96150f
Compare
@teacup-on-rockingchair could you rebase this PR with the current master, please? This should solve the failed CI tests. Thanks |
For SLE platforms add the possibilty to configure the emergency target auth via drop-in configuration file, rather than directly in the /usr/lib/systemd.. unit
e96150f
to
23af3a8
Compare
linux_os/guide/system/accounts/accounts-physical/require_emergency_target_auth/oval/shared.xml
Show resolved
Hide resolved
linux_os/guide/system/accounts/accounts-physical/require_emergency_target_auth/oval/shared.xml
Outdated
Show resolved
Hide resolved
linux_os/guide/system/accounts/accounts-physical/require_emergency_target_auth/oval/shared.xml
Outdated
Show resolved
Hide resolved
Thanks @marcusburghardt for raising the flag
Code Climate has analyzed commit 02f4dc8 and detected 0 issues on this pull request. The test coverage on the diff in this pull request is 100.0% (50% is the threshold). This pull request will bring the total coverage in the repository to 52.4% (0.0% change). View more on Code Climate. |
Description:
Rationale: