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

Add support for the firstboot remediation - RHEL8 #190

Merged
merged 1 commit into from Mar 7, 2022

Conversation

matejak
Copy link
Contributor

@matejak matejak commented Jan 17, 2022

Port of #189 to RHEL8

@matejak matejak added this to the 1.2.2 milestone Jan 17, 2022
@pep8speaks
Copy link

pep8speaks commented Jan 17, 2022

Hello @matejak! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

There are currently no PEP 8 issues detected in this Pull Request. Cheers! 🍻

Comment last updated at 2022-03-04 16:00:37 UTC

@scrutinizer-notifier
Copy link

The inspection completed: 1 new issues, 4 updated code elements

OpenSCAP can support a new remediation type -
a classical scan + remediate that is executed during the first boot.
This PR adds support for this functionality,
and allows to control it via a remediate kickstart property.
@jan-cerny
Copy link
Member

I have tested with the latest update and it works fine with the latest RHEL 8.6.

@jan-cerny jan-cerny merged commit 652bc63 into OpenSCAP:rhel8-branch Mar 7, 2022
OSCAP_REMEDIATE_HTML_REPORT='{report_path}'
""")
if ds_id:
config += "OSCAP_REMEDIATE_DATASTREAM_ID='{ds_id}'\n"
Copy link
Contributor Author

@matejak matejak Mar 8, 2022

Choose a reason for hiding this comment

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

ouch, we miss the tiny, but important letter f here and on lines below. The ds_id doesn't get substituted.

Copy link
Member

Choose a reason for hiding this comment

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

Then how its possible that it worked? What did I test?

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.

None yet

4 participants