Repository navigation
Conversation
Collaborator
|
I guess these can then also be removed from |
Three of the eight entries are removed. Each is either already satisfied
transitively or is dead weight:
libselinux-utils already a dependency of selinux-policy, and the
stig-hardening role never invokes any of its binaries
ntp installed but never configured - no task in the role
notifies the time-sync handlers, and 0 of the 107
PHTN-50-xxxxxx controls cover time sync
libgcrypt a workaround for aide declaring only an unversioned
Requires: libgcrypt; that belongs in aide.spec
The same three are dropped from examples/ova/packages_stig.json, so the OVA
package list and the installer's own list say the same thing. Keeping the two
in step matters: a list restated in two places is how the media and the
installer drifted apart in the first place.
Verified on four real STIG installs - two on photon-os-installer 2.8, two on
latest - whose manifests carry the reduced additional_packages list
[audit, rsyslog, openssl-fips-provider, selinux-policy, aide]:
libselinux-utils installed, transitively
libgcrypt installed, transitively
ntp / ntpsec absent, and time sync works via
systemd-timesyncd
aide's dependency is now a versioned Requires: libgcrypt >= 1.10.4, confirmed
in aide-0.19-3.ph5, so nothing rests on the unversioned form any more.
Signed-off-by: Daniel Casota <dcasota@gmail.com>
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01JW73JTCUGRcaNTUEQcAMtf
dcasota
force-pushed
the
upstream/stig-drop-redundant-packages
branch
from
September 2, 2026 06:07
d3ac861 to
964162b
Compare
Contributor
Author
|
The PR has been modified and tested with minimal ISO, kickstart, x86_64, without/with STIG, prebuilt canister. The ongoing tests with a full iso x86_64 should be fine, too. aarch64 hasn't been tested. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Removes three of the eight entries in
KS_STIG_PACKAGES, and the same three fromexamples/ova/packages_stig.json. Each is either already satisfied transitively or is dead weight.libselinux-utilsselinux-policy; the stig-hardening role never invokes any of its binariesntpPHTN-50-xxxxxxcontrols cover time synclibgcryptaidedeclaring only an unversionedRequires: libgcrypt; that belongs in aide.specOn
ntpspecificallyChecked against the role Photon 5.0 actually ships,
stig-hardening-2.1-10.ph5(source:vmware/dod-compliance-and-automation,photon/5.0/ansible/vmware-photon-5.0-stig-ansible-hardening):tasks/contains 107 distinctPHTN-50-xxxxxxcontrols and zero occurrences ofntp,chronyortimesynchandlers/main.ymldefinesrestart ntpd,restart chronyandrestart timesyncd. Of the 25notify:in the role, they are referenced 0 times — the only handlers ever notified arerestart sshd(21),restart rsyslog(2),restart resolved(1) andreload auditd(1)var_ntp_serversappears only in README.md, never in a taskSo installing
ntpsatisfies no control — it just adds a package.Also dropped from
examples/ova/packages_stig.jsonPer review feedback. The OVA package list and the installer's own declaration now say the same thing. Keeping the two in step matters here: a list restated in two places is the drift that left the STIG menu offering an option the media could not honour.
Verified on real installs
Ten unattended permutation-matrix rows on both installer variants —
photon-os-installer-2.8-7and2.9-3— each a full ISO build plus a kickstart install in a VM, with the STIG rows (k03,k04,k07,k08) exercising this change directly:additional_packagesin the install manifest is exactly[audit, rsyslog, openssl-fips-provider, selinux-policy, aide]— 5 entrieslibselinux-utilsandlibgcryptare still installed, as ordinary transitive dependencies; dropping them from the explicit list changed nothing about what lands on the systemntp/ntpsecare not installed, and time sync works:systemd-timesyncdactive,System clock synchronized: yesError(1011)on any rowOne note on SELinux, since an earlier revision of this description got it wrong: these installs boot Permissive, and that is correct rather than a regression.
selinux-policyshipsSELINUX=permissiveat subrelease ≥ 92 by design; only builds at subrelease ≤ 90 boot Enforcing. This change does not affect the mode either way.Related
The aide side of this is a versioned
Requires: libgcrypt >= 1.10.4guarded byphoton_subrelease >= 91, filed separately against the photon repo (dcasota/photon#21).