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

ntp/chrony remove server remediations and test scenarios #7631

Merged
merged 4 commits into from
Sep 24, 2021

Conversation

mildas
Copy link
Contributor

@mildas mildas commented Sep 23, 2021

Description:

chronyd_specify_remote_server
Update platform to multi_platform_rhel and remove unnecessary empty lines from tests

ntpd_specify_remote_server
Add remediations and test scenarios that are basically same as for chronyd_specify_remote_server. Remediations use same remote server variable as chronyd - var_multiple_time_servers

@mildas
Copy link
Contributor Author

mildas commented Sep 23, 2021

Updated ntpd test scenarios platform to RHEL 7, because ntp package is not available in RHEL8 and Fedora 33.

@jan-cerny
Copy link
Collaborator

Updated ntpd test scenarios platform to RHEL 7, because ntp package is not available in RHEL8 and Fedora 33.

If the ntp package isn't available on some systems, then we most likely should limit the platform of rule ntpd_specify_remote_server in rule.yml to only the systems where this package is available.

@jan-cerny jan-cerny self-assigned this Sep 24, 2021
@jan-cerny jan-cerny added this to the 0.1.59 milestone Sep 24, 2021
@openshift-ci
Copy link

openshift-ci bot commented Sep 24, 2021

@mildas: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/e2e-aws-rhcos4-e8 67063f1 link true /test e2e-aws-rhcos4-e8
ci/prow/e2e-aws-rhcos4-moderate 67063f1 link true /test e2e-aws-rhcos4-moderate

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Remediations are not enabled on sle12 as we haven't tested
them on this system.
@jan-cerny jan-cerny merged commit 27b3a74 into ComplianceAsCode:master Sep 24, 2021
@marcusburghardt marcusburghardt added the Test Suite Update in Test Suite. label Jun 23, 2022
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.

None yet

4 participants