Comments (2)
let entrypoint do its magic
The problem starts here. I just spend a day dissecting the logic of the docker image, sorry for being harsh, but it's ridiculous. Why does it copy over perfectly fine source code at /usr/src/nextcloud
?
My scenario was trying to get OVERWRITEPROTOCOL
to work which eventually I found was in /usr/src/nextcloud/config/...
however, that directory is only rsynced at first boot. So changes made to that directory are never migrated to the /var/www/html directory during upgrades.
My installation being somewhat older never got reverse-proxy.config.php
and all other fancy stuff.
Would the above issue of not being up-to-date with (at least) /usr/src/nextcloud/config be solved with your proposal?
from helm.
The current setup effectively requires several writable and persistent directories, both for writing configs via entrypoints as well as for managing nextcloud itself.
Yeah, that's currently just how nextcloud works.
Furthermore, by using the entrypoint dockerism, secret values are unnecessarily exposed to env (apps exposing env via debug endpoints is a favorite ;D ).
You'd need to take this up with the nextcloud/docker repo, and possibly nextcloud/server if it's an issue of env vars being exposed via debug endpoints. When you open that issue, please include steps to reproduce, including the debug endpoints you saw the security issue for.
Furthermore, apps can be installed from within the application.
That's probably not going to change, as that's just a feature of nextcloud.
unless we add another persistent, writable mount to the app directory, adding new apps requires new images (which is kinda the point of this exercise, but there is a need for clear communication to the user)
Adding a new app does not currently require a new image. I currently backup all of my nextcloud-files PVC and it backs up all the apps. You're still free to build an image with the apps you want, but then you'd have to rebuild the image everytime one of the apps upgrades, and that seems a bit unnecessary, but technically doable.
This issue is a bit all over the place. Is there something specific you'd like to see done? If so, please create an issue for the specific thing you'd like to do. For instance, if you would like to update "notes.txt mentioning limitations when readOnlyRootFilesystem and no persistence is set", it would be better to create an issue for just that. This chart is community maintained, and large issues mentioning multiple specific requests are a bit harder for maintainers to follow up on. You can also submit PRs directly to this repo for work you'd like to see done.
from helm.
Related Issues (20)
- Android | DAVx5 autoconfig does not work | PROPFIND |405 Not Allowed
- Redis config customizations superseded by builtin environment variables HOT 5
- Can't start Nextcloud because the version of the data (0.0.0.0) is higher than the docker image version () and downgrading is not supported. Are you sure you have pulled the newest image version? HOT 1
- Helm chart 4.6.8 does not work when using an external database for NextCloud. "key" is not defined. HOT 1
- Cannot disable packaged PostgreSQL HOT 2
- Feature: Use CronJob resources instead of a container with crond HOT 9
- PHP Memory Limit should be configured properly by default HOT 8
- Enabling `.persistence.enabled` leads to `/var/www/html/lib` and probably other files not being created HOT 5
- chown permission in nfs pvc HOT 2
- Error: Accessing site insecurely via HTTP | Config Question HOT 3
- Add ExtraVolumes for Cronjob Container HOT 2
- Feature: Add Valkey as an optional subchart to be an alternative to redis HOT 1
- Feature: Add test for enabling redis to ci
- Feature: Add test for metrics in CI
- Feature: Add OnlyOffice as a subchart
- Feature: Add Collabora as a subchart
- New S3 Configuration breaks prior S3 configuration and is missing uploadPartSize HOT 7
- Feature: Add test for adding extra configs in CI
- S3: `failed to create resource: [...] spec.template.spec.containers[1].env[22].valueFrom.secretKeyRef.key: Required value]` HOT 5
- Changing nextcloud.host vars results in crashloopbackoff HOT 1
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from helm.