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

Enable PCI-DSS in test-farm tests #11257

Merged
merged 1 commit into from
Nov 9, 2023

Conversation

marcusburghardt
Copy link
Member

Description:

We want in increase the adoption of pcidss_4 control file.
This control file is more agnostic to products and therefore more collaboration on it is expected.
It is good to include profile tests in Upstream in order to detect any possible issue as earlier as possible.

Rationale:

  • Complement CI tests to find eventual issues earlier.

@marcusburghardt marcusburghardt added the Test Suite Update in Test Suite. label Nov 8, 2023
Copy link

github-actions bot commented Nov 8, 2023

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

Fedora Environment
Open in Gitpod

Oracle Linux 8 Environment
Open in Gitpod

@mildas
Copy link
Contributor

mildas commented Nov 8, 2023

PCI DSS tests are running (see TF tests). However, they are failing in Ansible test in rpm_verify_hashes. I've proposed a change for the downstream test - https://src.fedoraproject.org/tests/scap-security-guide/pull-request/45
@marcusburghardt please check the PR. When it gets merged, we can re-run.

@mildas mildas self-assigned this Nov 8, 2023
@marcusburghardt
Copy link
Member Author

PCI DSS tests are running (see TF tests). However, they are failing in Ansible test in rpm_verify_hashes. I've proposed a change for the downstream test - https://src.fedoraproject.org/tests/scap-security-guide/pull-request/45 @marcusburghardt please check the PR. When it gets merged, we can re-run.

It worked @mildas . Thanks

Copy link

codeclimate bot commented Nov 9, 2023

Code Climate has analyzed commit 72bc715 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 58.8%.

View more on Code Climate.

@marcusburghardt
Copy link
Member Author

Gate (AR / RHEL) / Build, Lint Ansible Roles on Fedora Latest (Container) (pull_request) failure is not related to this PR.

@mildas mildas merged commit 053350f into ComplianceAsCode:master Nov 9, 2023
37 of 38 checks passed
@marcusburghardt marcusburghardt deleted the pcidss_ci_tests branch November 9, 2023 15:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Test Suite Update in Test Suite.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants