Skip to content

Add a rationale for package_sssd_installed#14856

Open
Mab879 wants to merge 1 commit into
ComplianceAsCode:masterfrom
Mab879:fix_RHEL-178437
Open

Add a rationale for package_sssd_installed#14856
Mab879 wants to merge 1 commit into
ComplianceAsCode:masterfrom
Mab879:fix_RHEL-178437

Conversation

@Mab879

@Mab879 Mab879 commented Jul 7, 2026

Copy link
Copy Markdown
Member

Description:

Add a rationale for package_sssd_installed

Rationale:

Fixes RHEL-178437

@Mab879 Mab879 added this to the 0.1.82 milestone Jul 7, 2026
@Mab879 Mab879 added the bugfix Fixes to reported bugs. label Jul 7, 2026
@Mab879 Mab879 force-pushed the fix_RHEL-178437 branch from 55d073d to 52dd04c Compare July 7, 2026 16:19
{{{ describe_package_install(package="sssd") }}}
rationale: ""
rationale: |-

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

The rationale says briefly what sssd does. But it needs to explain why users need to install this package. I think this rationale won't satisfy the reporters of the linked RHEL ticket.

Is the package optional or a convenience package? What security capabilities will it bring? Why do STIG and other policies require this package?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I have updated the rationale.

@Mab879 Mab879 force-pushed the fix_RHEL-178437 branch from 52dd04c to 9655e42 Compare July 8, 2026 15:38

This package is needed so that authentication from
remote directories is possible. Without this package
the system will unable to use many centralized

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
the system will unable to use many centralized
the system will be unable to use many centralized

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Fixed

@Mab879 Mab879 force-pushed the fix_RHEL-178437 branch from 9655e42 to 6c6ce91 Compare July 8, 2026 17:20
@openshift-ci

openshift-ci Bot commented Jul 8, 2026

Copy link
Copy Markdown

@Mab879: 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/4.20-images 6c6ce91 link true /test 4.20-images
ci/prow/4.19-images 6c6ce91 link true /test 4.19-images
ci/prow/e2e-aws-openshift-platform-compliance 6c6ce91 link true /test e2e-aws-openshift-platform-compliance
ci/prow/e2e-aws-openshift-node-compliance 6c6ce91 link true /test e2e-aws-openshift-node-compliance

Full PR test history. Your PR dashboard.

Details

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-sigs/prow repository. I understand the commands that are listed here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix Fixes to reported bugs.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants