-
-
Notifications
You must be signed in to change notification settings - Fork 4
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
Update MirrorBrain base image #244
Comments
I strongly suspect that we will have to implement #239 first, which will invalidate/fix this ticket as well. |
I just updated LOTS of stuff:
Most stuff is working fine on my local environment, I even found how to fix the flags + missing CSS on I will open corresponding PRs Thursday on the various repos. |
PRs are ready:
However, while preparing these PRs to make all this mergeable, I realize that I'm not really comfortable with our approach anymore. As mentioned in previous comment there are a lot of things which I have been modified forced to modify to update this base image, and I broke things while doing this (and there are probably things that I'm not aware of) ; see kiwix/mirrorbrain#1 for details. Just the fact that I had to fork means that chances to merge this one day are low due to the significant work needed to prepare PRs which could be reviewed (not speaking about the fact that there is probably no reviewers, at least for now). All that is combined with the fact that I realized during this base image update that:
To summarize, I think there is no momentum to have support on these changes anywhere in the community and if we continue on this path, we (I ?) are doomed to maintain this fork / project on our own. I really wonder if it makes sense to continue on the current path or if we should wait and move to another solution. Let's discuss this live tomorrow during our weekly meeting. |
MirrorBrain image is currently based on
httpd:2.4.43
which is more or less 3 years old.This means that its base OS (Debian buster) will soon (somewhere in 2024) be out of support.
We need to update to a more recent base image before that (or implement #239).
The text was updated successfully, but these errors were encountered: