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

Update CIS L2 test for configure_crypto_policy #9163

Conversation

yuumasato
Copy link
Member

@yuumasato yuumasato commented Jul 18, 2022

Description:

  • Since CIS RHEL8 v2.0.0 the policy doesn't require FUTURE in L2 anymore.

Rationale:

  • Fix failing test scenarios:
ERROR - Script policy_future_cis_l2.pass.sh using profile xccdf_org.ssgproject.content_profile_cis found issue:
ERROR - Rule evaluation resulted in fail, instead of expected pass during initial stage 
ERROR - The initial scan failed for rule 'xccdf_org.ssgproject.content_rule_configure_crypto_policy'.
ERROR - Script policy_future_cis_l2.pass.sh using profile xccdf_org.ssgproject.content_profile_cis_workstation_l2 found issue:
ERROR - Rule evaluation resulted in fail, instead of expected pass during initial stage 
ERROR - The initial scan failed for rule 'xccdf_org.ssgproject.content_rule_configure_crypto_policy'.

Related #9154 diffL585

From CIS RHEL8 v2.0.0 the policy doesn't require FUTURE in L2 anymore.
@yuumasato yuumasato added the productization-issue Issue found in upstream stabilization process. label Jul 18, 2022
@yuumasato yuumasato added this to the 0.1.63 milestone Jul 18, 2022
@github-actions
Copy link

Start a new ephemeral environment with changes proposed in this pull request:

rhel8 (from CTF) Environment (using Fedora as testing environment)
Open in Gitpod

Fedora Testing Environment
Open in Gitpod

Oracle Linux 8 Environment
Open in Gitpod

@Mab879 Mab879 self-assigned this Jul 18, 2022
@Mab879 Mab879 added Test Suite Update in Test Suite. CIS CIS Benchmark related. labels Jul 18, 2022
@codeclimate
Copy link

codeclimate bot commented Jul 18, 2022

Code Climate has analyzed commit d3e8cb9 and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (50% is the threshold).

This pull request will bring the total coverage in the repository to 42.7% (0.0% change).

View more on Code Climate.

@Mab879 Mab879 merged commit eb69d8c into ComplianceAsCode:master Jul 18, 2022
@yuumasato yuumasato deleted the update_rename_crypto_policy_cis_l2_test branch July 18, 2022 15:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CIS CIS Benchmark related. productization-issue Issue found in upstream stabilization process. Test Suite Update in Test Suite.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants