linuxserver / docker-nextcloud

GNU General Public License v3.0
678 stars 128 forks source link

[BUG] No update notifications for installed apps #356

Closed GuiPoM closed 9 months ago

GuiPoM commented 10 months ago

Is there an existing issue for this?

Current Behavior

Nextcloud is not informing admin users of new apps version. Also no mail received. It might be linked to the removal of system update notification.

I am not able to easily reproduce this issue but Recognize app is currently 4.3.0, new version is 4.3.2, and nextcloud is not notifying about the update. maj

Expected Behavior

I was used to get notifications (both app, webapp or mail) when a new app version is released

Steps To Reproduce

Nothing special, but a new app version must be available for one installed app

Environment

- OS: OMV 6.7.0-1 (Shaitan)
- How docker service was installed: omv docker compose official plugin

CPU architecture

x86-64

Docker creation

version: "3.8"
services:

  nextcloud:
    image: ghcr.io/linuxserver/nextcloud
    container_name: nextcloud
    environment:
      PUID: 998
      PGID: 100
      TZ: Europe/Paris
    labels:
      diun.enable: true
    volumes:
      - /srv/XXX/nextcloud/config:/config
      - /srv/XXX/appdata/nextcloud/data:/data
      - XXX
    depends_on:
      - mariadb
    restart: unless-stopped
    networks:
      - nextcloud
      - nextcloud_db

Container logs

[migrations] started
[migrations] 01-nginx-site-confs-default: skipped
[migrations] 02-default-location: skipped
[migrations] done
usermod: no changes
───────────────────────────────────────
      ██╗     ███████╗██╗ ██████╗ 
      ██║     ██╔════╝██║██╔═══██╗
      ██║     ███████╗██║██║   ██║
      ██║     ╚════██║██║██║   ██║
      ███████╗███████║██║╚██████╔╝
      ╚══════╝╚══════╝╚═╝ ╚═════╝ 
   Brought to you by linuxserver.io
───────────────────────────────────────
To support LSIO projects visit:
https://www.linuxserver.io/donate/
───────────────────────────────────────
GID/UID
───────────────────────────────────────
User UID:    998
User GID:    100
───────────────────────────────────────
using keys found in /config/keys
[custom-init] No custom files found, skipping...
[ls.io-init] done.
github-actions[bot] commented 10 months ago

Thanks for opening your first issue here! Be sure to follow the relevant issue templates, or risk having this issue marked as invalid.

nemchik commented 10 months ago

Can you try restarting your container and see if it updates the apps?

That wouldn't be the final recommended solution, but I'm curious if that does it. I was under the impression (and looked at the NC code, but maybe missed something) that the updatenotification app only notifies for updates to NC itself. If this is incorrect, I'd like to find a better solution, and it's easy to add back into our image.

GuiPoM commented 10 months ago

Hello. Thank you for your answer. Sadly I updated this app, but next time there is another app update I will check that the notification is not available, and check if the container restart does anything.

szaimen commented 10 months ago

The updatenotification app is indeed also responsible for notifying about app updates.

GuiPoM commented 10 months ago

I have one app to update, Maps 1.1.0 to 1.1.1. I do not get the notification and even a container restart does not help here.

nemchik commented 10 months ago

It can be easily added back into a future release. I'll have a look at doing this soon.