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

Always use python3 wsgi apache module on Debian #343

Merged
merged 1 commit into from
Jan 22, 2022

Conversation

kenyon
Copy link
Member

@kenyon kenyon commented Nov 6, 2021

puppetboard only uses python3, so this commit configures apache::mod::wsgi to use the python3 version by default on Debian 10. Before this fix, following the README for using puppetboard::apache::conf, the python2 wsgi apache module would be installed. With this PR we provide a working puppetboard installation by default, with less user configuration needed.

Also use apache::custom_config instead of a plain file resource. Eventually this module should not have to know any apache implementation details, it would use puppetlabs-apache module resources instead. This helps move in that direction.

Also convert the apache conf and ldap templates to EPP.

Fixes voxpupuli/puppetboard/issues/527

@kenyon kenyon marked this pull request as draft November 6, 2021 07:05
@kenyon kenyon force-pushed the apache-custom-config branch 4 times, most recently from cd29dc0 to bbe1928 Compare November 6, 2021 08:04
@kenyon kenyon changed the title apache/conf: use python3 wsgi apache module on Debian always use python3 wsgi apache module on Debian Nov 6, 2021
@kenyon kenyon force-pushed the apache-custom-config branch 2 times, most recently from 5d0e8ff to d97cd57 Compare November 6, 2021 08:52
@kenyon kenyon marked this pull request as ready for review November 6, 2021 09:05
@kenyon kenyon force-pushed the apache-custom-config branch from d97cd57 to 7582c7d Compare November 6, 2021 09:13
@kenyon kenyon force-pushed the apache-custom-config branch 3 times, most recently from 163587f to db4c6e8 Compare January 22, 2022 22:07
@kenyon kenyon marked this pull request as draft January 22, 2022 22:07
@kenyon kenyon marked this pull request as ready for review January 22, 2022 22:33
@kenyon kenyon marked this pull request as draft January 22, 2022 22:39
puppetboard only uses python3, so this commit configures
apache::mod::wsgi to use the python3 version by default on Debian.
Before this fix, the python2 wsgi apache module would be
installed (unless you were on Debian 11, in which case
puppetlabs-apache configures apache::mod::wsgi to use the python3
version for you).

Also use apache::custom_config in puppetboard::apache::conf instead of
a plain file resource. Eventually this module should not have to know
any apache implementation details, it would use puppetlabs-apache
module resources instead. This helps move in that direction.

Also convert the apache conf and ldap templates to EPP.
@kenyon kenyon force-pushed the apache-custom-config branch from db4c6e8 to 789d6f8 Compare January 22, 2022 22:42
@kenyon
Copy link
Member Author

kenyon commented Jan 22, 2022

Added an ensure => absent for the original apache config filename to avoid breaking existing users. (Using apache::custom_conifg rather than placing the file ourselves changes the filename, but the content is the same, using the same template.)

@kenyon kenyon marked this pull request as ready for review January 22, 2022 23:04
@kenyon kenyon merged commit fb9fe6d into voxpupuli:master Jan 22, 2022
@kenyon kenyon added the enhancement New feature or request label Jan 22, 2022
@smortex smortex changed the title always use python3 wsgi apache module on Debian Always use python3 wsgi apache module on Debian Apr 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Internal Server Error since v1.1.0
2 participants