New S3 Configuration breaks prior S3 configuration and is missing uploadPartSize about helm HOT 7 OPEN

@jessebot thanks for the fast reaction, I can confirm that the workaround (nextcloud.defaultConfigs.s3.config.php=false) works for me :) I've been able to upgrade and everything seems to work fine.

from helm.

@provokateurin thanks for the quick response, I can try to open a PR but I'm no expert in helm charts and I can't test the change locally at the moment.

from helm.

🙇 sorry about that, @acidicX!

I actually submitted nextcloud/docker#2271 upstream for these changes, but I'll submit one here as well now!

from helm.

Sorry for this breaking change :/ I think your config should continue to work fine by just renaming the config file.
Could you open a PR for the uploadPartSize so that it is present and allows you to switch to the new configs?

from helm.

In the meantime, you should be able to disable the default config being deployed by setting nextcloud.defaultConfigs.s3.config.php=false, or via values.yaml:

nextcloud:
  defaultConfigs:
    s3.config.php: false

Can you confirm if this works? Because if that doesn't work either, we should also try to fix that. Sorry again for the inconvenience regardless 🙏

from helm.

Sorry for the rapid updates, but if #614 gets merged, it will close this Issue automatically. That doesn't mean we can't still help you if other things are still broken here though, so please just let us know if that's the case and we can reopen this issue for you if it gets closed 🙏

Update: We're going to wait till nextcloud/docker reviews/merges our upstream PR, nextcloud/docker#2271 before merging #614

from helm.

Hello @jessebot , @acidicX !
I got the same trouble with already defined mapping key for s3.config.php file.
I used:

nextcloud:
  defaultConfigs:
    s3.config.php: false

and it does not work for me. I try for 5.4.0 and 5.5.2 chart versions.

I was able to fix it with renaming my s3 config file to:

configs:
        s33.config.php:

Unfortunately it steel needs to be fix. :(

Essential part of my helmrelease file:

    defaultConfigs:
      s3.config.php: false
      swift.config.php: false
    objectStore:
      s3:
        enabled: true
        ssl: true
        port: ***
        region: ***
        usePathStyle: true
        autoCreate: true
        existingSecret: nextcloud-values
        secretKeys:
          host: hostname
          accessKey: key
          secretKey: secret
          bucket: bucket

from helm.