Skip to content

Conversation

@MoteHue
Copy link
Contributor

@MoteHue MoteHue commented Jan 6, 2026

This conflicts with our configured doca repos, which need gpgcheck disabled.

@MoteHue MoteHue requested a review from a team as a code owner January 6, 2026 14:28
@MoteHue MoteHue added bug Something isn't working xs Extra small (Less than 10 changes) Epoxy labels Jan 6, 2026
Alex-Welsh
Alex-Welsh previously approved these changes Jan 6, 2026
Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request disables the CIS hardening rule that enforces GPG checks on all yum repositories to accommodate a specific repository (doca) that does not support it. While this resolves the immediate conflict, it introduces a security risk by disabling a critical verification step for all repositories. My review includes a suggestion for a more targeted and secure approach that addresses the repository conflict without weakening the overall system security.

@Alex-Welsh
Copy link
Member

Reno would be nice but fine as it is

This conflicts with our configured doca repos, which need gpgcheck
disabled.
@MoteHue MoteHue enabled auto-merge January 6, 2026 14:54
@MoteHue MoteHue merged commit 3cead46 into stackhpc/2025.1 Jan 6, 2026
21 of 22 checks passed
@MoteHue MoteHue deleted the cis-disable-gpgcheck-rule branch January 6, 2026 17:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working Epoxy xs Extra small (Less than 10 changes)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants