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

[BUG] No update notifications for installed apps #356

Closed
1 task done
GuiPoM opened this issue Aug 23, 2023 · 6 comments · Fixed by #358
Closed
1 task done

[BUG] No update notifications for installed apps #356

GuiPoM opened this issue Aug 23, 2023 · 6 comments · Fixed by #358

Comments

@GuiPoM
Copy link

GuiPoM commented Aug 23, 2023

Is there an existing issue for this?

  • I have searched the existing issues

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
Copy link

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

@nemchik
Copy link
Member

nemchik commented Aug 24, 2023

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
Copy link
Author

GuiPoM commented Aug 24, 2023

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
Copy link

szaimen commented Aug 27, 2023

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

@GuiPoM
Copy link
Author

GuiPoM commented Aug 30, 2023

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
Copy link
Member

nemchik commented Aug 30, 2023

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

nemchik added a commit that referenced this issue Aug 31, 2023
Closes #356

Signed-off-by: GitHub <noreply@github.com>
nemchik added a commit that referenced this issue Aug 31, 2023
Closes #356

Signed-off-by: GitHub <noreply@github.com>
@LinuxServer-CI LinuxServer-CI moved this from Issues to Done in Issue & PR Tracker Sep 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants