Skip to content

fix: use gpasswd instead of removed groupmems for local groups - #282

Merged
jakub-vavra-cz merged 1 commit into
SSSD:masterfrom
jakub-vavra-cz:groupmems
Oct 23, 2026
Merged

jakub-vavra-cz merged 1 commit into
SSSD:masterfrom
jakub-vavra-cz:groupmems

Conversation

@jakub-vavra-cz

Copy link
Copy Markdown
Contributor

groupmems was dropped from recent Fedora; manage /etc/group membership with gpasswd -a/-d for users and gpasswd -M for nested group members.

Assisted-by: Cursor

@alexey-tikhonov

Copy link
Copy Markdown
Member

A note for reviewer: please make sure it works on C8S

@thalman thalman left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Also re-base please.

@jakub-vavra-cz
jakub-vavra-cz force-pushed the groupmems branch 2 times, most recently from b1637cd to 3390074 Compare October 13, 2026 06:46
@jakub-vavra-cz

jakub-vavra-cz commented Oct 20, 2026 •

Copy link
Copy Markdown
Contributor Author

Works on RHEL 8.10:
Only the the known failure: test_basic__hostname_excluded[fqdn] (bare_client) present.
Rebased.

@jakub-vavra-cz

Copy link
Copy Markdown
Contributor Author

ping @thalman

groupmems was dropped from recent Fedora; manage /etc/group membership
with gpasswd -a/-d for users and gpasswd -M for nested group members.

Assisted-by: Cursor

@thalman thalman left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Works for me, thanks

@jakub-vavra-cz
jakub-vavra-cz merged commit 8af604a into SSSD:master Oct 23, 2026
6 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants