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

Upgrade base image to phusion/baseimage:noble-1.0.0 #413

Closed

Conversation

ajhodgson
Copy link
Contributor

  • Upgrades image base to baseimage built with Ubuntu 24.04 LTS (Noble Numbat)
  • Default Python version is now 3.12

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.

@ajhodgson ajhodgson force-pushed the ajhodgson/upgrade-to-noble branch from bd56cf7 to f18337a Compare August 24, 2024 18:17
##

access_log /var/log/nginx/access.log;
error_log /var/log/nginx/error.log;

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?

Copy link
Contributor Author

@ajhodgson ajhodgson Aug 26, 2024

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.

@ajhodgson ajhodgson force-pushed the ajhodgson/upgrade-to-noble branch from fc4a44c to 32f28b3 Compare September 3, 2024 14:14
@CamJN
Copy link
Member

CamJN commented Oct 14, 2024

github was being dumb about merging this so I did it on the cli

@CamJN CamJN closed this Oct 14, 2024
@ajhodgson ajhodgson deleted the ajhodgson/upgrade-to-noble branch October 15, 2024 01:16
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 this pull request may close these issues.

3 participants