Skip to content

Commit

Permalink
security: call /usr/libexec/fips-setup-helper
Browse files Browse the repository at this point in the history
crypto-policies now ships a helper for anaconda to call
in order to just "do the right thing"
and make it not anaconda's responsibility.
  • Loading branch information
t184256 committed Aug 13, 2024
1 parent 1be740a commit 321d91c
Show file tree
Hide file tree
Showing 4 changed files with 8 additions and 11 deletions.
2 changes: 1 addition & 1 deletion anaconda.spec.in
Original file line number Diff line number Diff line change
Expand Up @@ -136,7 +136,7 @@ Requires: python3-pid

# Required by the systemd service anaconda-fips.
Requires: crypto-policies
Requires: /usr/bin/update-crypto-policies
Requires: crypto-policies-scripts

# required because of the rescue mode and VNC question
Requires: anaconda-tui = %{version}-%{release}
Expand Down
9 changes: 3 additions & 6 deletions pyanaconda/modules/security/installation.py
Original file line number Diff line number Diff line change
Expand Up @@ -159,13 +159,10 @@ def run(self):
log.debug("Don't set up FIPS on %s.", conf.target.type.value)
return

# We use the --no-bootcfg option as we don't want fips-mode-setup
# to modify the bootloader configuration. Anaconda already does
# everything needed & it would require grubby to be available on
# the system.
# Bootloader is not modified. Anaconda already does everything needed.
util.execWithRedirect(
"fips-mode-setup",
["--enable", "--no-bootcfg"],
"/usr/libexec/fips-setup-helper",
["anaconda"],
root=self._sysroot
)

Expand Down
2 changes: 1 addition & 1 deletion pyanaconda/modules/security/security.py
Original file line number Diff line number Diff line change
Expand Up @@ -194,7 +194,7 @@ def collect_requirements(self):
# Add FIPS requirements.
if self.fips_enabled:
requirements.append(Requirement.for_package(
"/usr/bin/fips-mode-setup",
"/usr/libexec/fips-setup-helper",
reason="Required for FIPS compliance."
))

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -322,7 +322,7 @@ def test_fips_requirements(self, kernel_arguments_mock):
assert self.security_interface.CollectRequirements() == [
{
"type": get_variant(Str, "package"),
"name": get_variant(Str, "/usr/bin/fips-mode-setup"),
"name": get_variant(Str, "/usr/libexec/fips-setup-helper"),
"reason": get_variant(Str, "Required for FIPS compliance.")
}
]
Expand Down Expand Up @@ -1014,7 +1014,7 @@ def test_configure_fips_task(self, mock_util):
task.run()

mock_util.execWithRedirect.assert_called_once_with(
"fips-mode-setup",
["--enable", "--no-bootcfg"],
"fips-setup-helper",
["anaconda"],
root="/mnt/sysroot"
)

0 comments on commit 321d91c

Please sign in to comment.