Skip to content
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

Update Nextcloud to 28 #2641

Closed
9 tasks done
Tracked by #2724
Pilzinsel64 opened this issue Jan 10, 2024 · 6 comments · Fixed by #2752
Closed
9 tasks done
Tracked by #2724

Update Nextcloud to 28 #2641

Pilzinsel64 opened this issue Jan 10, 2024 · 6 comments · Fixed by #2752

Comments

@Pilzinsel64
Copy link
Member

Pilzinsel64 commented Jan 10, 2024

Pending todos

Pending PRs

Pending upstream issues

@Pilzinsel64
Copy link
Member Author

@kyrofa @pachulo I would like to assign the tasks about creation of branch 27 and merging of #2665 to you, if you agree and have that time. I was thinking about creating the 27 branch myself to be able to merge and test #2668 but I have no idea what needs to be done at launchpad regarding the 27 branche and even don't have any abilities to do that. So I probably should leave that to you before I break anything. ;)

@kyrofa
Copy link
Member

kyrofa commented Feb 4, 2024

Done.

@Pilzinsel64
Copy link
Member Author

Pilzinsel64 commented Feb 4, 2024

Alright, after a quick test on my prod I found out that the log-flooting bug still exists. However, this does NOT happen as long as you just install the raw snap package without any additional or recommended apps.

See nextcloud/server#42671 (comment)

I recommend to not move this to stable until we (or Nextcloud) found a solution as many server owner do probably additional apps.

In my case there are 8 apps that cause that error. But for sure, this is a problem in the core server repo and NOT of the apps.

Everything else does work fine as far as I can tell after 30 minutes testing. The general performance (e.g. loading a page) seems to be even noticable better then with 27.1.6.

If there is something else, I will let you know here.

@Pilzinsel64

This comment was marked as outdated.

@Pilzinsel64
Copy link
Member Author

Info: We did everything that were required todo. The only thing left, before a release, is test v28.0.6 via latest/beta for a few days and merge #2738.

@scubamuc
Copy link
Member

fresh install tests completed without issues

sudo snap install nextcloud --channel=latest/beta

grafik

grafik

two warnings in logs

grafik

debugging script logs

{"reqId":"r2QUqeOYvK3LDOz7UQzX","level":2,"time":"2024-05-25T20:47:02+00:00","remoteAddr":"2.206.244.214","user":"xxxx","app":"no app in context","method":"GET","url":"/ocs/v2.php/apps/spreed/api/v4/room","message":"Request does not pass strict cookie check","userAgent":"Mozilla/5.0 (X11; Linux x86_64; rv:126.0) Gecko/20100101 Firefox/126.0","version":"28.0.6.1","data":[]}
{"reqId":"z5EEbo7g4rACr4ARf6ZC","level":2,"time":"2024-05-25T20:49:51+00:00","remoteAddr":"","user":"--","app":"mail","method":"","url":"--","message":"itinerary file doesn't exist","userAgent":"--","version":"28.0.6.1","data":{"app":"mail"}}

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants