chore: replace nginx base image with busybox#366
Merged
JammingBen merged 1 commit intoopencloud-eu:mainfrom Feb 19, 2026
Merged
Conversation
dschmidt
force-pushed
the
chore/replace-nginx-with-busybox-base-image
branch
from
0a60133 to
e394b41
Compare
dschmidt
force-pushed
the
chore/replace-nginx-with-busybox-base-image
branch
from
e394b41 to
e7864a0
Compare
Contributor
Author
|
This changes the folder to which the apps are copied, so if anyone relies on their location in the nginx folder that needs to be adjusted now ... (or rather for every single app when it's next released ...) Not sure if it matters too much to anyone or if we should do a new release series so that this change can be done in one go for all apps ... thoughts? |
Contributor
Author
|
... or just keep the legacy path ... a bit ugly but doesnt break anything 🤷🏻 |
dschmidt
force-pushed
the
chore/replace-nginx-with-busybox-base-image
branch
from
e7864a0 to
00ff537
Compare
The nginx web server is no longer needed since OpenCloud serves web extensions directly. Replace the ~180MB nginx image with a ~4MB busybox image that only provides a minimal shell and basic utilities for file operations. - Switch base image from nginx to busybox (pinned by digest) - Move app files to /web/apps/ (matching WEB_ASSET_APPS_PATH convention) - Remove nginx-specific directives (CMD, STOPSIGNAL, html cleanup) - Include app_name in image title label
dschmidt
force-pushed
the
chore/replace-nginx-with-busybox-base-image
branch
from
00ff537 to
7aeec6a
Compare
Contributor
Author
|
Added a backward compatibility symlink 💪🏻 |
6 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
/web/apps/to match theWEB_ASSET_APPS_PATHconvention/usr/share/nginx/html→/web/appsfor backwards compatibility with existing deploymentsCMD,STOPSIGNAL, html dir cleanup)app_namein the OCIimage.titlelabelTest plan
/web/apps/<app_name>cp -r /usr/share/nginx/html/<app_name>still works via the symlink