Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

FCGI paths should not be messed with on RHEL/CenOS 7. #299

Closed
wants to merge 1 commit into from

Conversation

haimgel
Copy link
Contributor

@haimgel haimgel commented Dec 7, 2014

CentOS 7 (and recent Fedoras) have Apache 2.4, where FCGI socket path
and shared memory path is managed adequately without further involvment
neccessary (subdirectory is created under /var/run/httpd).

However, when the path is specified manually, that subdirectory is NOT
created automatically if it doesn't already exist and Apache fails to
start. Since in recent RHEL systems /var/run is on tmpfs, end result is
that chef-created subdirectory disappears on shutdown, and Apache fails
to start after server reboot. Best to leave out these two settings
unset then.

CentOS 7 (and recent Fedoras) have Apache 2.4, where FCGI socket path
and shared memory path is managed adequately without further involvment
neccessary (subdirectory is created under /var/run/httpd).

However, when the path is specified manually, that subdirectory is NOT
created automatically if it doesn't already exist and Apache fails to
start. Since in recent RHEL systems /var/run is on tmpfs, end result is
that chef-created subdirectory disappears on shutdown, and Apache fails
to start after server reboot. Best to leave out these two settings
unset then.
@josephholsten
Copy link
Contributor

This seems reasonable, but needs to be updated onto the current code.

group node['apache']['group']
recursive true
only_if { node['platform_version'].to_i >= 6 }
if ((node['platform_family'] == 'rhel') && (node['platform_version'].to_i == 6)) ||

This comment was marked as outdated.

recursive true
only_if { node['platform_version'].to_i >= 6 }
if ((node['platform_family'] == 'rhel') && (node['platform_version'].to_i == 6)) ||
((node['platform_family'] == 'fedora') && (node['platform_version'].to_i < 20))

This comment was marked as outdated.

Copy link
Member

@damacus damacus left a comment

Choose a reason for hiding this comment

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

This seems great, can you resolve the merge conflicts please?

If you can't do the library helper, we'll just pull this in and fix after.

Thank you!

@damacus
Copy link
Member

damacus commented Mar 5, 2018

Closing due to inactivity.

Please feel free to open against the current code base. We'd love to have this contribution.

@lock
Copy link

lock bot commented Oct 23, 2019

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators Oct 23, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants