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

Dockerfile: apt-get upgrade -y to get the latest packages #129

Merged
merged 1 commit into from
Nov 12, 2024

Conversation

Al2Klimov
Copy link
Member

@Al2Klimov Al2Klimov commented Nov 12, 2024

regardless of the image being up to date or not.

regardless of the image being up to date or not.
@Al2Klimov
Copy link
Member Author

I'd like you to merge this, given that there's a complete v2.12.2 milestone already.

Copy link
Contributor

@julianbrost julianbrost left a comment

Choose a reason for hiding this comment

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

As I also said in Icinga/docker-icinga2#133 (review):

Note that this isn't supposed to say that we shouldn't use up-to-date base images, but that there's a possibility that the latest base image contains packages for which a newer update is already available in the Debian package repos.

@julianbrost
Copy link
Contributor

@Al2Klimov requested a review from @julianbrost 1 hour ago
@Al2Klimov assigned @julianbrost 9 minutes ago

Such impatience!

@Al2Klimov Al2Klimov merged commit 796b6c7 into master Nov 12, 2024
2 checks passed
@Al2Klimov Al2Klimov deleted the Al2Klimov-patch-2 branch November 12, 2024 10:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants