helm icon indicating copy to clipboard operation
helm copied to clipboard

[Feature] High Performance File Backend

Open Blackclaws opened this issue 4 years ago • 11 comments

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.

Blackclaws avatar Apr 19 '21 10:04 Blackclaws

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. As of now it seems in my opinion that there is too much manual steps to cover...

menardorama avatar May 13 '21 19:05 menardorama

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.

sbor23 avatar May 25 '21 16:05 sbor23

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

Blackclaws avatar May 25 '21 16:05 Blackclaws

Yes you're right that seems possible

menardorama avatar May 31 '21 22:05 menardorama

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.

chrisingenhaag avatar Jun 01 '21 08:06 chrisingenhaag

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.

Blackclaws avatar Jun 01 '21 11:06 Blackclaws

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 :)

jessebot avatar Jan 26 '23 17:01 jessebot

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.

provokateurin avatar May 13 '23 11:05 provokateurin

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 🤦

I think that would include the HPBs for Files and Talk, notify_push and the Talk recording server (and maybe more).

HPB is high performance backend, right? As in the backends listed in this blog post?

how do you feel about a nextcloud.objectStore section like mentioned in https://github.com/nextcloud/helm/issues/477#issue-2000830792?

Then we could have like:

nextcloud:
  objectStore:
    s3:
      enabled: true
    swift:
      enabled: false

jessebot avatar Dec 01 '23 14:12 jessebot

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.

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 …

Nils98Ar avatar Dec 14 '23 10:12 Nils98Ar

This would be cool. It seems to me, that for the following things are required:

  • nginx should be able to upgrade to websocket connections https://github.com/nextcloud/notify_push?tab=readme-ov-file#nginx
  • No port must be exposed (nginx acts as proxy)
  • Server must be started (located at /var/www/html/custom_apps/notify_push/bin/x86_64/notify_push after installed as app)
  • OCC command to setup notifications must be started

Can we preconfigure that into the chart so that it will be the default?

AndreKoepke avatar Apr 02 '24 05:04 AndreKoepke