This repository was archived by the owner on Mar 8, 2023. It is now read-only.
Nginx/Gunicorn support for RedHat and other small things #237
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Changes from upstream:
and RedHat
versions are installed.
gr_web_server_remove_default
-- default config isonly wiped when the variable is true, or when it is
undef
andgr_web_server_port
== 80gr_apache_port
andgr_apache_port_https
togr_web_server_port
andgr_web_server_port
respectively. If the old configuration values are used, puppetthrows an error telling you to use the new ones. (If you don't want to bump a major
version number for these changes, I can change it to a deprecation warning, but
that's less elegant.)
Changes since fork that are in line with latest commits:
python-tzlocal
package for RedHatTo test the changes, I provisioned stock VMs with puppet and pip. I did two runs of
puppet apply per OS:
include graphite
class { 'graphite': gr_web_server => 'nginx', }
If I could display a metric in the Graphite Composer, then the test passed.
Now for all the caveats:
/usr/bin/pip-python
to/usr/bin/pip
(see:PUP-3829 for CentOS 6/7))
nginx.conf
includes aserver
section listening on port80
.Thus, it is not possible to set up graphite without modifying the package-provided
configuration file. I opted to do nothing about this; graphite will successfully,
but the default
server
section has to be removed fromnginx.conf
. This needsto be noted somewhere in the README file.
gunicorn-debian
script, which callsgunicorn_django
, which is deprecated.python-cairocffi
now, as it's what is recommended/required by graphite. However, Ubuntu 15.10'spython-cairocffi
package is broken (it tries to importantsomelib.so
when the actual file issomelib.so.1
), so I've kept all debian variants usingpython-cairo
.systemctl restart gunicorn
after installThe following OS's were tested successfully:
The following OS's did not work:
The pip provider is broken (see PUP-4502