helm icon indicating copy to clipboard operation
helm copied to clipboard

Next Major Release: v7

Open wrenix opened this issue 1 year ago • 6 comments

Let here collect all changes for the next major release.

Solve:

  • redis

  • postgresql

    • issue #641
    • additional PR #642
  • mariadb

    • issue: #658
  • nextcloud (v31 - needed by postgresql v17)

    • [x] version exists: tracked in https://github.com/nextcloud/server/milestone/280
    • [x] image exists: https://hub.docker.com/_/nextcloud/tags?name=31.0.0

    maybe also:

    • https://github.com/nextcloud/helm/pull/379

wrenix avatar Dec 19 '24 22:12 wrenix

hi @wrenix , v31.0.0 is available in docker hub

GreyXor avatar Feb 26 '25 09:02 GreyXor

So ready to merge ;)

wrenix avatar Feb 26 '25 09:02 wrenix

It would be great if someone could also test the updated subcharts. It would be bad to update them to a version that is broken for some reason.

provokateurin avatar Feb 26 '25 09:02 provokateurin

i run it with redis (without database / external) .

Therefore we have the Lint and Tests.

with my other patches:

  • #694
  • #581

see: https://github.com/wrenix/nextcloud-helm/commits/main/

wrenix avatar Feb 26 '25 18:02 wrenix

Just want to say that when I set image to 31 I couldn't install apps and got following error in browser console when trying:

Content-Security-Policy: The page’s settings blocked the loading of a resource (connect-src) at https://<redacted>/apps/files/ because it violates the following directive: “connect-src 'self'”

I used chart version 6.6.6. I don't know if that'll be same after this PR. I removed tag image.tag wiped everything and checked with old version (30 from chart) and I can install apps from GUI.

I use(d) external postgres and no memcache for now.

rafal11ck avatar Mar 01 '25 23:03 rafal11ck

I could verify that bug and created an Issue upstream: https://github.com/nextcloud/server/issues/51213

It looks like an error in http <-> https location handling, which couse the problem of not being self in Content-Security-Policy way ... - lets track it in the issue.

wrenix avatar Mar 03 '25 19:03 wrenix

LGTM

GreyXor avatar Mar 29 '25 08:03 GreyXor

Just want to say that when I set image to 31 I couldn't install apps and got following error in browser console when trying:

Content-Security-Policy: The page’s settings blocked the loading of a resource (connect-src) at https://<redacted>/apps/files/ because it violates the following directive: “connect-src 'self'”

I used chart version 6.6.6. I don't know if that'll be same after this PR. I removed tag image.tag wiped everything and checked with old version (30 from chart) and I can install apps from GUI.

I use(d) external postgres and no memcache for now.

This it can be ignored as there is declarative approach

nextcloud:
  hooks:
    before-starting: |-
      # install apps
      for appname in spreed contacts calendar user_oidc bookmarks deck
        do php occ app:install "$appname"
      done
      true

rafal11ck avatar Mar 29 '25 11:03 rafal11ck

@provokateurin hello, can we merge it and tag a new version ? Or maybe we miss something?

GreyXor avatar Apr 11 '25 09:04 GreyXor

Anything I can do as a user to assist in moving this forward? Maybe something to test?

The chart is still on 30.0.6 which means it lacks lots of fixes. Perhaps it's possible to move that to 30.0.10 if this is not ready?

dankar avatar Apr 19 '25 18:04 dankar

31.0.5 should habe the fix for https://github.com/nextcloud/server/issues/51213

I will test it, then the image is released.

Edit: Still no image on 2025-05-16: https://github.com/nextcloud/docker/actions/runs/15071619890 Edit: Now landet -> i rebase tomorrow (no time today)

wrenix avatar Apr 30 '25 12:04 wrenix

hello @wrenix thank you for your maintaining work on the helm chart :pray:. Just can we have an update on your test of 31.0.5 ? Did the new version didn't solve the plugin issue ?

JulesdeCube avatar May 15 '25 18:05 JulesdeCube

My test show it works. The server settings selfcheck is really slow.

Need a review. Ping @provokateurin

wrenix avatar May 20 '25 11:05 wrenix

Hello @provokateurin, sorry to ping you but when you think you could review this PR ?

I know that is repo is comunity driven but it's been 5 month since Nextcloud version 31 has been release and 2 week that this MR is ready.

JulesdeCube avatar Jun 03 '25 21:06 JulesdeCube

Sorry I will not continue working on this repo. I have not been using it myself for a while now and have no interest in maintaining it, as many people here (not anyone in this PR specifically) complain about everything that is not perfect, giving me no motivation at all to care about this anymore. I'm sorry that this PR is stuck, but please stop pinging me anywhere in this repo as I will not do anything about it. I hope you understand.

provokateurin avatar Jun 03 '25 21:06 provokateurin

Hello,

sorry again to ping you my goal wasn't at all to be toxic or to force you to review. I wasn't just sneaking some information on when this pr could be merge.

I totally understand your don't want to maintain this repo any more and that you can be burn out out with nextcloud's community toxicity.

Can you remove you as maintainer so this don't append again ? Also did you know witch maintainer is still active (so we can ask him a review) ?

JulesdeCube avatar Jun 03 '25 21:06 JulesdeCube

It is a great work from all the team. And knowing about the difficulties we really appreciate it. I understand also the frustrations the consumers (we) can bring sometimes. I also understand if some contributors want to stop, due to various reasons.

In the other hand would be nice if before stoping at least deliver this great upgrade. And let people who do not appreciate keep complaining.

Any reviewer which would help this to move forward, would be great.

varet80 avatar Jun 19 '25 03:06 varet80

@wrenix I am interested to become a contributor/author for the nextcloud chart, as my work is 90% on these. I would be happy also to help maintain this chart.

varet80 avatar Jun 19 '25 06:06 varet80

@wrenix - If nothing would be going on here we propably should fork the repo? - Or how can we find out active maintainers here.... just saw You already have a fork. Could You please publish Your helm charts?

em-devops-team avatar Jun 19 '25 09:06 em-devops-team

@em-devops-team I am going to fork it on my Charts and put it for release! I need a few hours. k8sonlab, will send the links.

varet80 avatar Jun 19 '25 09:06 varet80

Lets open an issue to discuss maintainer ships ... maybe @provokateurin will add some maintainers afterward (i do not know any other maintainer with more access / employee at nextcloud).

For everyone how like to use my personal main-branch, it is build AS oci thanks to #720. But maybe the Versionnumber is not stable i like to rebase in this project if something change: https://github.com/wrenix/nextcloud-helm/pkgs/container/nextcloud-helm%2Fnextcloud

wrenix avatar Jun 19 '25 10:06 wrenix

You can find people in https://cloud.nextcloud.com/call/xs25tz5y or https://cloud.nextcloud.com/call/oefq5xvp who will be able to help you assign new maintainers.

provokateurin avatar Jun 19 '25 10:06 provokateurin

lets goooo 🚀

ojsef39 avatar Jul 13 '25 08:07 ojsef39