-
Notifications
You must be signed in to change notification settings - Fork 269
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
[Feature] High Performance File Backend #109
Comments
Already have a working version but I can't find an elegant way to install and setup the app except using a one time job. |
Sorry for being off-topic. If you find a way it would be great to expose the feature of specifying a range of apps to install along side. This would help a lot getting towards a true stateless setup. |
Using a one time job is fine for setup things like this. A lot of other helm charts do that such as the gitlab chart |
Yes you're right that seems possible |
I think it would be more nice to have a docker image of the notify_push serverside component instead of "installing" it within Pod start. I would prevent this if I can. |
I guess having a separate docker image would also work, the concept of installing it into the server and simply mounting the same apps volume into notify push also works rather well however, that way you are also always in sync between the app and the service. |
I'm happy to review any pull request, but what exactly is the current plan here? We are on Nextcloud 25 now, so this issue may be stale, but I'm happy to discuss regardless :) |
I found this issue because I was checking if notify_push is supported. @jessebot I think we should try making more Nextcloud components ready to use. I think that would include the HPBs for Files and Talk, notify_push and the Talk recording server (and maybe more). I don't know how hard each of these are, but it would be nice to support them. |
Sorry for the extreme delay. Sometime in June I accidentally cleared all my github notifications and I'm still wading through old issues and PRs trying to figure out what I missed 🤦
HPB is high performance backend, right? As in the backends listed in this blog post? how do you feel about a Then we could have like: nextcloud:
objectStore:
s3:
enabled: true
swift:
enabled: false |
Maybe some should be separate charts? The templates from https://github.com/nextcloud/all-in-one/tree/main/nextcloud-aio-helm-chart/templates could probably be used as basis … |
This would be cool. It seems to me, that for the following things are required:
Can we preconfigure that into the chart so that it will be the default? |
Signed-off-by: e542445 <andre.koepke@sbb.ch>
Signed-off-by: Andre Köpke <github@akop.online>
Are you planning to include the High Performance File Backend aka Client Push in the Helm chart or would you accept a pull request that adds this? This might be related to #106 as Nextcloud 21 is a base requirement for that.
The text was updated successfully, but these errors were encountered: