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

Release 28.0.5snap1 #2731

Closed
wants to merge 19 commits into from

Conversation

Pilzinsel64
Copy link
Member

stondino00 and others added 19 commits February 4, 2024 16:42
…xtcloud-28-0-2

Upgrade Nextcloud to 28.0.2
Fix some typos, info, and make the section for configuring notify_push as brief as possible, as the proper instructions are on the wiki.
Also lift the phpredis extension to version 6.0.2

Fixes nextcloud-snap#2647
…dme-notify_push

Fix some README.md problems
…-nextcloud2804

Upgrade Nextcloud to 28.0.4
…-upgrade-apache-2459

Upgrade Apache to 2.4.59
…-update-nextcloud-to-2805

Update Nextcloud to 28.0.5
@scubamuc
Copy link
Member

Fresh install tests completed with issues

grafik

grafik

grafik

  1. Maintenance window: sudo nextcloud.occ config:system:set maintenance_window_start --value="1"
  2. Missing indices: sudo nextcloud.occ db:add-missing-indices

logs look fine

@Pilzinsel64
Copy link
Member Author

Pilzinsel64 commented Apr 30, 2024

  1. To be ignored. See Update Nextcloud to 28 #2641 (comment)
  2. Works fine for me. Tested myself by installing latest/stable and then refreshing to latest/beta/pr-2731. See nextcloud: update to 27.1.9 #2728 (comment)

@scubamuc
Copy link
Member

retested fresh install sudo snap install nextcloud --channel=latest/beta/pr-2731

wait 15 Minutes...

issues persist, see messages;

grafik

@Pilzinsel64
Copy link
Member Author

Pilzinsel64 commented Apr 30, 2024

Oh dear, I didn't installed recommended apps. It's the mail app. That's why. I could reproduce that behavior by just installing the mail app.

When you install mail without ever having it installed before, you get a bunch of missing indices. This seems to be a bug(?) by the Mail app. The most users will not regonize that as bug as it is common that indices are missing after an update (or for snap users: missing indices always get fixed on update). The only way you get this is installing the Mail app without having it installed every before.

This PR probably introduces the behavior: nextcloud/mail#9295
Looks like it's time for me to create a new upstream issue. :/

But as mail isn't installed by default, this shouldn't be a blocker for us, I guess.

@scubamuc
Copy link
Member

scubamuc commented May 5, 2024

@Pilzinsel64,

But as mail isn't installed by default, this shouldn't be a blocker for us...

my tests are done as an absolute noob would install the snap. the process hasn't changed. that would definitely include installing "recommended apps". so the "mail app" will most probably be installed when the snap is initialised. there is a wiki entry describing how to add indices... so that's covered.

I'm not sure if this is a PR-Testing issue because according to this;

grafik

your hint "should be done by fixer" is absolutely right. how will we know? @kyrofa could you shed some light please?

@Pilzinsel64
Copy link
Member Author

Closing as we wait for #2738 and #2659

@Pilzinsel64 Pilzinsel64 deleted the release/28.0.5 branch May 16, 2024 06:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update Nextcloud to 28
4 participants