Skip to content

Conversation

jovial
Copy link
Contributor

@jovial jovial commented Dec 8, 2022

No description provided.

@jovial jovial marked this pull request as ready for review January 11, 2023 16:08
@jovial jovial requested a review from a team as a code owner January 11, 2023 16:08
- src: stackhpc.vxlan
- name: ansible-lockdown.rhel8_cis
src: https://github.com/ansible-lockdown/RHEL8-CIS
version: 1.3.0
Copy link
Contributor

Choose a reason for hiding this comment

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

This is from July 2021, would be nice to update to 2.1.0 when there is time to validate it.

Copy link
Member

Choose a reason for hiding this comment

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

Newer versions do not support CentOS Stream anymore, but it would be good to have a newer version on Ubuntu/Rocky.

Copy link
Contributor

Choose a reason for hiding this comment

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

Doh. Probably something to update in zed/Antelope then.

Copy link
Member

Choose a reason for hiding this comment

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

Note that it doesn't support inject_facts_as_vars = False.

Copy link
Contributor

@markgoddard markgoddard Feb 22, 2023

Choose a reason for hiding this comment

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

hmm. The playbook should at least have a comment to that effect. I suggested to @GregWhiteyBialas to use the config lookup plugin to check that the option is true for the Wazuh playbook.

@markgoddard markgoddard merged commit 41ec9cf into stackhpc/yoga Feb 21, 2023
@markgoddard markgoddard deleted the feature/cis branch February 21, 2023 12:33
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.

4 participants