Skip to content

RANGER-5698 : Docker changes for enabling PAM authentication by default - #16

Open
fimugdha wants to merge 1 commit into
apache:mainfrom
fimugdha:RANGER-5698
Open

RANGER-5698 : Docker changes for enabling PAM authentication by default#16
fimugdha wants to merge 1 commit into
apache:mainfrom
fimugdha:RANGER-5698

Conversation

@fimugdha

Copy link
Copy Markdown

What changes were proposed in this pull request?

This pull-request is handling review provided : apache/ranger#1171 (comment) to create the default shadow group add ranger user to that group for docker setup.

How was this patch tested?

Tested the shell script logic on linux machine manually.

echo "Starting user and group creation..."

# Create ranger group and users
create_users_and_groups "ranger" "ranger rangeradmin rangerusersync rangertagsync rangerkms rangerauditserver"

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.

Passing shadow here in the second param will add it to ranger group, if this works the new function may not be necessary. please check.

@fimugdha fimugdha Sep 11, 2026

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

@kumaab,

The existing function is not suitable for this use-case. The group shadow will be added as the ranger user primary group (-g) if ranger user it not present.

We need ranger user to be part of shadow group as an additional/secondary group (-aG).

Therefore created a new function.

@fimugdha
fimugdha requested a review from kumaab September 11, 2026 15:25
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.

2 participants