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
Replace wsgi package conflict with config file #1523
Conversation
1b785c5
to
faaeaf8
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is just out-of-curiosity review. IPA devel should check from functional POV.
ipaplatform/base/tasks.py
Outdated
def configure_httpd_wsgi_conf(self): | ||
"""Configure WSGI for correct Python version""" | ||
# only used on Fedora, NOP on other platforms | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
raise NotImplementedError()
missing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nope, I did this on purpose. The function is not implemented on any platform except Fedora.
ipaplatform/redhat/tasks.py
Outdated
if sys.version_info > (3, 0): | ||
wsgi_module = constants.MOD_WSGI_PYTHON3 | ||
else: | ||
wsgi_module = constants.MOD_WSGI_PYTHON3 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should not it be constants.MOD_WSGI_PYTHON2
here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah, got catch :)
Instead of a package conflict, freeIPA now uses an Apache config file to enforce the correct wsgi module. The workaround only applies to Fedora since it is the only platform that permits parallel installation of Python 2 and Python 3 mod_wsgi modules. RHEL 7 has only Python 2 and Debian doesn't permit installation of both variants. See: https://pagure.io/freeipa/issue/7161 Fixes: https://pagure.io/freeipa/issue/7394 Signed-off-by: Christian Heimes <cheimes@redhat.com>
faaeaf8
to
748ca34
Compare
master:
|
Instead of a package conflict, freeIPA now uses an Apache config file to
enforce the correct wsgi module. The workaround only applies to Fedora
since it is the only platform that permits parallel installation of
Python 2 and Python 3 mod_wsgi modules. RHEL 7 has only Python 2 and
Debian doesn't permit installation of both variants.
Fixes: https://pagure.io/freeipa/issue/7394
Signed-off-by: Christian Heimes cheimes@redhat.com