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

Update Passenger to a newer release #272

Closed
CSC-swesters opened this issue Apr 25, 2024 · 4 comments · Fixed by #274
Closed

Update Passenger to a newer release #272

CSC-swesters opened this issue Apr 25, 2024 · 4 comments · Fixed by #274

Comments

@CSC-swesters
Copy link

We have a Python 3.9 Passenger app in our OOD deployment, which is causing Passenger to print deprecation warnings like this:

App 475 output: /opt/rh/ondemand/root/usr/share/passenger/helper-scripts/wsgi-loader.py:26: DeprecationWarning: the imp module is deprecated in favour of importlib; see the module's documentation for alternative uses
App 475 output:   import sys, os, re, imp, threading, signal, traceback, socket, select, struct, logging, errno

This is apparently fixed in a later Passenger release, here: phusion/passenger@a19753c

Would it be possible to bump the version of Passenger that is packaged for OOD? The newest at the time of writing seems to be 6.0.20 (see their Releases page).

In addition to this specific python 3.9 fix, Passenger has bumped a lot of their own dependencies as well, so it might be good to keep up with them just in general.

Thanks in advance!

@treydock
Copy link
Collaborator

The only passenger related dependencies managed by OnDemand are passenger itself and NGINX. We build a Passenger RPM and a NGINX RPM. Some related dependencies like Ruby are more tied to OnDemand and currently we depend on what the OS supports so we are limited in what updates can be performed.

@CSC-swesters
Copy link
Author

I see, so the Passenger RPM that OnDemand ships is presumably quite standalone, without any downstream dependencies, thanks to the way you are packaging it?

I notice that you've opened a new PR attempting to upgrade. Thanks for the quick action!

@treydock
Copy link
Collaborator

I see, so the Passenger RPM that OnDemand ships is presumably quite standalone, without any downstream dependencies, thanks to the way you are packaging it?

Most of the dependencies for Passenger are bundled into the Passenger RPM. I believe there are a few external dependencies like Ruby and Apache which come from the OS. The only dependencies called out with versions in the RPM spec we adopted from upstream is NGINX and boost, both of which are getting updated in #274.

@CSC-swesters
Copy link
Author

Thank you for merging the fixes!


Passenger has bumped a lot of their own dependencies as well, so it might be good to keep up with them just in general.

Most of the dependencies for Passenger are bundled into the Passenger RPM.

I think we've arrived at what I tried to say originally.The dependencies that are bundled into the Passenger RPM are also shipped by OOD (since you choose to package it yourselves), and that makes it necessary to keep an eye on them. They are on OOD's software bill of materials (SBOM), if you will.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants