Skip to content
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

Misc RHEL-10 fixes, rhc-worker-playbook, OSPP disablement #255

Merged
merged 6 commits into from
Aug 20, 2024

Conversation

comps
Copy link
Contributor

@comps comps commented Aug 16, 2024

The OSPP change is a result of ComplianceAsCode/content#12223.

@comps
Copy link
Contributor Author

comps commented Aug 16, 2024

Regarding the disablement of OSPP and the need to change so many files -- unfortunately, there is not reasonable solution for us for now, we really do need teemtee/tmt#1793 .

We could do ie. PROFILE=blabla /hardening/oscap, but that specification can't be easily paralellized via runcontest, for example, because it doesn't represent FMF tests, only env+test combos, and (even then) the test specification would break some assumptions about test/result name prefixing.

Make ansible-core required (makes sense for ansible-specific
tests), but leave rhc-worker-playbook always as recommend,
letting lib/ansible.py install dependencies from ansible galaxy
if the RPM is not available.

Signed-off-by: Jiri Jaburek <[email protected]>
Signed-off-by: Jiri Jaburek <[email protected]>
This fixes it for RHEL-10 which has java-21-openjdk.

Signed-off-by: Jiri Jaburek <[email protected]>
It seems --template was removed in OpenSCAP v1.4, and --fix-type
seems to be a better fit here anyway.

    OpenSCAP/openscap#2134

It's also used in other ansible tests instead of --template.

The resulting playbook is exactly the same.

Signed-off-by: Jiri Jaburek <[email protected]>
@mildas mildas merged commit f3e6c98 into main Aug 20, 2024
3 checks passed
@mildas mildas deleted the rhel_10_additions branch August 20, 2024 07:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants