linuxserver / docker-nzbget

GNU General Public License v3.0
149 stars 83 forks source link

six modules removed during cleanup #115

Closed keppo070 closed 3 years ago

keppo070 commented 3 years ago

linuxserver.io


Expected Behavior

Apprise would work from NZBGET. Notify script visible in settings NZBGET

Current Behavior

Module six is removed https://github.com/linuxserver/docker-nzbget/blob/master/Dockerfile#L96

Steps to Reproduce

  1. docker exec -it nzbget /bin/bash
  2. python Notify.py

Environment

OS: Debian 10.6 CPU architecture: x86_64 How docker service was installed: official docker repo

Command used to create docker container (run/create/compose/screenshot)

version: "2"
services:
  nzbget:
    image: linuxserver/nzbget
    container_name: nzbget
    environment:
      - PUID=1000
      - PGID=1000
      - TZ=Europe/Amsterdam
      - DOCKER_MODS=linuxserver/mods:universal-apprise
    volumes:
      - /home/<userid>/docker-config/nzbget/config:/config
      - /mnt/storage/tempdata/complete:/downloads
    ports:
      - 6789:6789
    restart: unless-stopped

Docker logs

[s6-init] making user provided files available at /var/run/s6/etc...exited 0.
[s6-init] ensuring user provided files have correct perms...exited 0.
[fix-attrs.d] applying ownership & permissions fixes...
[fix-attrs.d] done.
[cont-init.d] executing container initialization scripts...
[cont-init.d] 01-envfile: executing...
[cont-init.d] 01-envfile: exited 0.
[cont-init.d] 10-adduser: executing...

-------------------------------------
          _         ()
         | |  ___   _    __
         | | / __| | |  /  \
         | | \__ \ | | | () |
         |_| |___/ |_|  \__/

Brought to you by linuxserver.io
-------------------------------------

To support the app dev(s) visit:
NZBGet: https://nzbget.net/donate

To support LSIO projects visit:
https://www.linuxserver.io/donate/
-------------------------------------
GID/UID
-------------------------------------

User uid:    1000
User gid:    1000
-------------------------------------

[cont-init.d] 10-adduser: exited 0.
[cont-init.d] 30-config: executing...
[cont-init.d] 30-config: exited 0.
[cont-init.d] 99-custom-files: executing...
[custom-init] no custom files found exiting...
[cont-init.d] 99-custom-files: exited 0.
[cont-init.d] done.
[services.d] starting services
[services.d] done.
[INFO] nzbget 21.0 server-mode
[INFO] Reloading...
[INFO] nzbget 21.0 server-mode
[INFO] Executing script Notify.py with command TestServers
[INFO] Notify: Traceback (most recent call last):
[INFO] Notify:   File "/downloads/scripts/Notify.py", line 169, in <module>
[INFO] Notify:     import six
[INFO] Notify: ModuleNotFoundError: No module named 'six'
[ERROR] Script Notify.py with command TestServers failed (terminated with unknown status)
aptalca commented 3 years ago

@keppo070 Can you give this test build a try? lsiodev/nzbget:amd64-v21.0-pkg-4980b6a2-dev-23a0f3368fe7e172621917e9cef77d5c4b3bbaf6 Just replace linuxserver/nzbget with the above and recreate the container Thanks

keppo070 commented 3 years ago

Works for me!

But I had to copy the Notify.py script to the downloads/script folder to make it work. So it seems apprise is working but to use it from nzbget you have to copy the files. Can you include them in the image? That would make it out-of-the-box.

maybe this is not in scope of linuxserver... you decide

aptalca commented 3 years ago

Thanks for confirming.

We're only responsible for apprise working. That script is a third party script.