-
Notifications
You must be signed in to change notification settings - Fork 407
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
Upgrade base image to phusion/baseimage:noble-1.0.0 #413
Conversation
bd56cf7
to
f18337a
Compare
## | ||
|
||
access_log /var/log/nginx/access.log; | ||
error_log /var/log/nginx/error.log; |
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.
is it falling back to STDOUT by default ? why this change?
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.
It still sets error_log /var/log/nginx/error.log; - it just does it at the root scope now.
You can look at phusion/passenger#2541 for why.
There is no change in behaviour here.
Make Python 3.12 the default python version
during image build.
fc4a44c
to
32f28b3
Compare
github was being dumb about merging this so I did it on the cli |
I fixed the minor issues I found and noted significant user-affecting changes in CHANGELOG.md.
I have built all variants of the image and verified that the correct interpreter versions are installed in each.
I tested 2 Rails apps and 2 python apps on versions of this container (amd64 only).
Review and further testing recommended.