Skip to content

Remove condition that prevents adjusting SELinux labels#246

Merged
steven-schattenberg-itential merged 3 commits intoitential:mainfrom
steven-schattenberg-itential:redis-se-linux-labels
Oct 2, 2025
Merged

Remove condition that prevents adjusting SELinux labels#246
steven-schattenberg-itential merged 3 commits intoitential:mainfrom
steven-schattenberg-itential:redis-se-linux-labels

Conversation

@steven-schattenberg-itential
Copy link
Contributor

Under certain conditions the SELinux labels were not getting set. This would cause Redis to fail at startup due to SELinux restrictions. This change removes those conditions and sets the appropriate labels on directories when SELinux is enforcing and non-standard directories are used.

@kvelarde-itential
Copy link
Contributor

@steven-schattenberg-itential How did you test these changes? I think we will have to do some regression testing.

  • Install from Remi RPM
  • Install from source
  • Install from source with custom conf, log, data, pid directories
  • Install from source with /var/lib/redis and /var/log/redis partitions

@steven-schattenberg-itential steven-schattenberg-itential merged commit 76bd6ef into itential:main Oct 2, 2025
1 check passed
kvelarde-itential pushed a commit to kvelarde-itential/deployer that referenced this pull request Oct 9, 2025
* Remove condition that prevents adjusting SELinux labels

* Corrected style issue
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