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

Alternative instructions for httpd Python 3 #285

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

pjonesIDBS
Copy link
Contributor

As per #280 here are some alternative instructions for CentOS 7 and RHEL 7 users to utlilse Python 3 with Apache (httpd).

Pull Request (PR) description

This Pull Request (PR) fixes the following issues

As per voxpupuli#280 here are some alternative instructions for CentOS 7 and RHEL 7 users to utlilse Python 3 with Apache (httpd).
Copy link
Member

@ghoneycutt ghoneycutt left a comment

Choose a reason for hiding this comment

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

Thanks for the docs! There is still value in the other way of setting it up, so would rather see this as an addition instead of a replacement.

@ghoneycutt ghoneycutt added the docs Improvements or additions to documentation label May 28, 2020
@pjonesIDBS
Copy link
Contributor Author

Hi, There is some value in the old docs, but they don't actually work as they are, which is part of the reason why I was removing them.

It's not enough to simply specify:
$mod_wsgi_package_name = 'rh-python36-mod_wsgi'

along with

class { 'apache':
  purge_configs => false,
  mpm_module    => 'prefork',
  default_vhost => false,
  default_mods  => false,
}

With this config and particularly purge_configs set to false Apache will be misconfigured at install and will fail to start. In my testing Puppet created additional module files but did not purge the standard module files. Lots of modules had duplicated content.

I also couldn't get Apache to symlink correctly to the module in /opt/rh/httpd24/root/usr/lib64/httpd/modules/ either. It errored every time.

The Apache modules in RHEL / CentOS Software Collections are all located under /opt and are generally meant to be installed alongside specific versions of httpd from the software collections repo, rather than the base version of Apache. I went so far as to test this route but it became painful as too many things had to change in my existing manifests. Service names and file locations all differ for the software collection versions of Apache. This is why I went to the IUS repo.

If you'd like me to add back some more details about the Software Collections option I can do so.

@vox-pupuli-tasks
Copy link

Dear @pjonesIDBS, thanks for the PR!

This is Vox Pupuli Tasks, your friendly Vox Pupuli Github Bot. I noticed that your pull request has CI failures. Can you please have a look at the failing CI jobs?
If you need any help, you can reach out to us on our IRC channel voxpupuli on Freenode or our Slack channel voxpupuli at slack.puppet.com.
You can find my sourcecode at voxpupuli/vox-pupuli-tasks

@vox-pupuli-tasks
Copy link

Dear @pjonesIDBS, thanks for the PR!

This is Vox Pupuli Tasks, your friendly Vox Pupuli GitHub Bot. I noticed that your pull request contains merge conflict. Can you please rebase?

You can find my sourcecode at voxpupuli/vox-pupuli-tasks

@bastelfreak
Copy link
Member

thanks for the PR! After a huge amount of work, our master branch is green again. To verify that this works, could you add it to our acceptance tests? At the moment we don't support CentOS 7, but if this works properly with the puppetboard/python module, we can readd it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Improvements or additions to documentation merge-conflicts needs-tests tests-fail
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants