Skip to content

Update base image in Dockerfile to latest version, add note on user - #220

Merged
umputun merged 1 commit into
umputun:masterfrom
paskal:paskal/dockerfile-improvements
Mar 16, 2025
Merged

umputun merged 1 commit into
umputun:masterfrom
paskal:paskal/dockerfile-improvements

Conversation

@paskal

@paskal paskal commented Mar 16, 2025

Copy link
Copy Markdown
Contributor

No description provided.

@paskal
paskal requested a review from umputun as a code owner March 16, 2025 17:49
Comment thread Dockerfile Outdated


FROM ghcr.io/umputun/baseimage/app:v1.12.0 as base
FROM ghcr.io/umputun/baseimage/app:latest as base

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I’d rather have the tagged version here instead of the latest. You know how sometimes incompatible changes sneak in and cause unexpected results? Well, for something as fundamental as this, it’s worth the extra effort to maintain compatibility.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In this case it makes no sense as we only copying certificate and timezone files from the base image, but I pinned it to v1.15.0 instead in the new version of the commit.

@umputun
umputun merged commit 1de0024 into umputun:master Mar 16, 2025
@paskal
paskal deleted the paskal/dockerfile-improvements branch March 16, 2025 18:05
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.

2 participants