Add KylinSecServer6 OS baseline support#14788
Conversation
|
Hi @jcm-123. Thanks for your PR. I'm waiting for a ComplianceAsCode member to verify that this patch is reasonable to test. If it is, they should reply with Regular contributors should join the org to skip this step. Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. DetailsInstructions 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-sigs/prow repository. |
Mab879
left a comment
There was a problem hiding this comment.
This PR needs a rebase and some adjustment. This PR is removing many features and changing the version back to 0.1.81. Please review.
0508b01 to
58cb5d6
Compare
Hi @Mab879, thanks for the review. I have rebased this branch onto the latest master and fixed the issues you pointed out: Restored SSG_PATCH_VERSION to 0.1.82 (it was incorrectly reverted to 0.1.81 during the previous rebase) Could you please trigger /ok-to-test when you have a moment? Thanks! |
58cb5d6 to
b828a8f
Compare
Add security baseline content for KylinSec Server OS: - Product definitions for kylinsecserver6 and kylinserver10 - CPE mappings and SCAP profiles - Applicability checks and OVAL rules - Build system integration (CMakeLists.txt, build_product) - PAM faillock and IPv6 sysctl rules - Emergency target authentication rules
b828a8f to
475bdd8
Compare
|
Hi @Mab879 and reviewers, I've rebased the branch onto the latest master and pushed the updated commit. Summary of changes: Rebase & sync Rebased onto current upstream master (branch was ~45 commits behind) Root cause: modifying the shared rule service_rsyncd_disabled triggered CTF to run Automatus tests on Debian 12, where the default package name rsync-daemon does not exist (Debian uses rsync) Added kylinsecserver6 to the Gate Fedora build list in gate_fedora.yml Could you please trigger /ok-to-test and re-review when you have a moment? Thanks! |
Add security baseline content for KylinSec Server OS:
Description:
Rationale:
Rationale here. Replace this text. Don't use the italics format!
Fixes # Issue number here (e.g. Updating sysctl XCCDF naming #26) or remove this line if no issue exists.
Review Hints:
Review hints here. Replace this text. Don't use the italics format!
Use this optional section to give any relevant information which could help the reviewer to more quickly and assertively understand and test the changes.
Good examples are useful commands, if it is better to review all commits together or in a suggested sequence, any relevant discussion in other PRs or issues, etc.