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

Let the test suite install OpenSCAP packages #801

Merged
merged 1 commit into from
Nov 22, 2020

Conversation

Bischoff
Copy link
Contributor

@Bischoff Bischoff commented Nov 21, 2020

What does this PR change?

The openSCAP packages were installed half in sumaform, half in the test suite, leading to a huge mess.

This situation is due to the fact that the QAM test suite does not use sumaform's testsuite module. But besides this immediate reason, it has always been wrong to let sumaform prepare ulterior processings. sumaform should prepare machines as pristine as possible.

This PR lets the test suite do all the job. See uyuni-project/uyuni#2880 for implementation on test suite side.

@Bischoff Bischoff merged commit 26d8239 into uyuni-project:master Nov 22, 2020
Copy link
Member

@srbarrios srbarrios left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. We should even remove more packages

@Bischoff
Copy link
Contributor Author

Bischoff commented Nov 23, 2020

LGTM. We should even remove more packages

Completly agree. Created cards: #802 and SUSE/spacewalk#13199

- mgr-cfg-actions
- wget
- require:
- sls: default

{% if grains['os'] == 'SUSE' %}
{% if grains['os'] == 'SUSE' and '12' in grains['osrelease'] %}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is aaa_base-extras something exclusively for SLE 12? Ie. not needed/present on SLE 15?

@Bischoff Bischoff deleted the openscap-move-to-testsuite branch January 25, 2021 12:42
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.

3 participants