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

Updating to Alpine v3.16 & PHP8 #45

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

seancrites
Copy link
Contributor

@seancrites seancrites commented Jul 3, 2022

Updated image to use Alpine v3.16 & PHP8.

Interesting outcome is that the v3.16/php8 image is 12MB smaller than the previous v3.12/php7:

nagios-alpine312  latest       283177380089   4 minutes ago    216MB
nagios-alpine316  latest       a48db1ef3aaf   17 minutes ago   204MB

@manios
Copy link
Owner

manios commented Jul 7, 2022

Hi @seancrites ,

Unfortunately upgrading to Alpine 3.16 works only for amd64 architecture. All other architectures have a problem with time as you can see in the picture:

image

For more information you can read this: #35 (comment)

The problem appears since Alpine 3.13. The musl-1.2 upgrade changed the definition of time_t to 64-bits on all arches. This affects armhf, armv7 and x86. See the musl time64 release notes and the wiki for more information.

At the moment NagiosEnterprises/nagioscore has not changed anything and I do not know how to migrate the Nagios code. I did some attempts by replacing

- include <sys/time.h>
+ include <compat/time32.h>

but in vain.

Any help from somebody with specific C++ knowledge on the matter would be appreciated.

Thanks,
Christos

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