ManageIQ / manageiq-rpm_build

Code to build RPMs for ManageIQ appliances and container images
Apache License 2.0
10 stars 25 forks source link

Simplify permissions for kafka and zookeeper #446

Closed bdunne closed 8 months ago

bdunne commented 8 months ago

Both need to read/write the same directories and config files. Previously zookeeper was failing to start because it didn't have permission to create a new log file.

Fixes https://github.com/ManageIQ/manageiq-appliance/issues/380

miq-bot commented 8 months ago

Checked commit https://github.com/bdunne/manageiq-rpm_build/commit/267887483a8b72016fbe322a785723e11fc343c6 with ruby 2.7.8, rubocop 1.56.3, haml-lint 0.51.0, and yamllint 0 files checked, 0 offenses detected Everything looks fine. :+1:

Fryguy commented 8 months ago

Backported to radjabov in commit 7b20e2ee08363659fc7e5cdb236704a8a34aa398.

commit 7b20e2ee08363659fc7e5cdb236704a8a34aa398
Author: Adam Grare <adam@grare.com>
Date:   Wed Feb 28 10:20:05 2024 -0500

    Merge pull request #446 from bdunne/kafka_zookeeper

    Simplify permissions for kafka and zookeeper

    (cherry picked from commit e8ad2449c3d6704140a7bff3a0460938dc08c7eb)