immich-app / immich

High performance self-hosted photo and video management solution.
https://immich.app
GNU Affero General Public License v3.0
52.42k stars 2.78k forks source link

Can't delete user #11433

Open ScuttleSE opened 3 months ago

ScuttleSE commented 3 months ago

The bug

I create a user

I delete the user immediately following the instructions here: https://immich.app/docs/administration/user-management#immediately-remove-user

Users is still in the list of users, now in red and unable to delete, edit or do anything with.

The OS that Immich Server is running on

Ubuntu 20

Version of Immich Server

1.110.0

Version of Immich Mobile App

n/a

Platform with the issue

Your docker-compose.yml content

#
# WARNING: Make sure to use the docker-compose.yml of the current release:
#
# https://github.com/immich-app/immich/releases/latest/download/docker-compose.yml
#
# The compose file on main may not be compatible with the latest release.
#

name: immich

services:
  immich-server:
    container_name: immich_server
    image: ghcr.io/immich-app/immich-server:${IMMICH_VERSION:-release}
    # extends:
    #   file: hwaccel.transcoding.yml
    #   service: cpu # set to one of [nvenc, quicksync, rkmpp, vaapi, vaapi-wsl] for accelerated transcoding
    volumes:
      - ${UPLOAD_LOCATION}:/usr/src/app/upload
      - /etc/localtime:/etc/localtime:ro
    env_file:
      - .env
    ports:
      - 2283:3001
    depends_on:
      - redis
      - database
    restart: always

  immich-machine-learning:
    container_name: immich_machine_learning
    # For hardware acceleration, add one of -[armnn, cuda, openvino] to the image tag.
    # Example tag: ${IMMICH_VERSION:-release}-cuda
    image: ghcr.io/immich-app/immich-machine-learning:${IMMICH_VERSION:-release}
    # extends: # uncomment this section for hardware acceleration - see https://immich.app/docs/features/ml-hardware-acceleration
    #   file: hwaccel.ml.yml
    #   service: cpu # set to one of [armnn, cuda, openvino, openvino-wsl] for accelerated inference - use the `-wsl` version for WSL2 where applicable
    volumes:
      - model-cache:/cache
    env_file:
      - .env
    restart: always

  redis:
    container_name: immich_redis
    image: docker.io/redis:6.2-alpine@sha256:328fe6a5822256d065debb36617a8169dbfbd77b797c525288e465f56c1d392b
    healthcheck:
      test: redis-cli ping || exit 1
    restart: always

  database:
    container_name: immich_postgres
    image: docker.io/tensorchord/pgvecto-rs:pg14-v0.2.0@sha256:90724186f0a3517cf6914295b5ab410db9ce23190a2d9d0b9dd6463e3fa298f0
    environment:
      POSTGRES_PASSWORD: ${DB_PASSWORD}
      POSTGRES_USER: ${DB_USERNAME}
      POSTGRES_DB: ${DB_DATABASE_NAME}
      POSTGRES_INITDB_ARGS: '--data-checksums'
    volumes:
      - ${DB_DATA_LOCATION}:/var/lib/postgresql/data
    healthcheck:
      test: pg_isready --dbname='${DB_DATABASE_NAME}' --username='${DB_USERNAME}' || exit 1; Chksum="$$(psql --dbname='${DB_DATABASE_NAME}' --username='${DB_USERNAME}' --tuples-only --no-align --command='SELECT COALESCE(SUM(checksum_failures), 0) FROM pg_stat_database')"; echo "checksum failure count is $$Chksum"; [ "$$Chksum" = '0' ] || exit 1
      interval: 5m
      start_interval: 30s
      start_period: 5m
    command: ["postgres", "-c" ,"shared_preload_libraries=vectors.so", "-c", 'search_path="$$user", public, vectors', "-c", "logging_collector=on", "-c", "max_wal_size=2GB", "-c", "shared_buffers=512MB", "-c", "wal_compression=on"]
    restart: always

volumes:
  model-cache:

Your .env content

# You can find documentation for all the supported env variables at https://immich.app/docs/install/environment-variables

# The location where your uploaded files are stored
UPLOAD_LOCATION=./library
# The location where your database files are stored
DB_DATA_LOCATION=./postgres

# To set a timezone, uncomment the next line and change Etc/UTC to a TZ identifier from this list: https://en.wikipedia.org/wiki/List_of_tz_database_time_zones#List
# TZ=Etc/UTC

# The Immich version to use. You can pin this to a specific version like "v1.71.0"
IMMICH_VERSION=release

# Connection secret for postgres. You should change it to a random password
DB_PASSWORD=postgres

# The values below this line do not need to be changed
###################################################################################
DB_USERNAME=postgres
DB_DATABASE_NAME=immich

Reproduction steps

1. Create user
2. Delete user "immediately"

Relevant log output

Nothing

Additional information

No response

jrasm91 commented 3 months ago

What are the server logs for this period of time? Or, when the delete job runs at night

ScuttleSE commented 3 months ago

These are the logs from this nigh:


[Nest] 7  - 07/29/2024, 10:26:06 PM   DEBUG [Microservices:VersionService] Running version check
[Nest] 7  - 07/29/2024, 11:26:06 PM   DEBUG [Microservices:VersionService] Running version check
[Nest] 7  - 07/30/2024, 12:00:00 AM   DEBUG [Microservices:LibraryService] Refreshing all external libraries: force=false
[Nest] 7  - 07/30/2024, 12:00:00 AM   DEBUG [Microservices:LibraryService] Refreshing all external libraries: force=false
[Nest] 7  - 07/30/2024, 12:00:00 AM   DEBUG [Microservices:LibraryService] Cleaning up any pending library deletions
[Nest] 7  - 07/30/2024, 12:00:00 AM   DEBUG [Microservices:LibraryService] Cleaning up any pending library deletions
[Nest] 7  - 07/30/2024, 12:00:00 AM   DEBUG [Microservices:PersonService] Deleted 0 people
[Nest] 7  - 07/30/2024, 12:00:00 AM VERBOSE [Microservices:SessionService] Deleted expired session token: /
[Nest] 7  - 07/30/2024, 12:00:00 AM VERBOSE [Microservices:SessionService] Deleted expired session token: Mac OS/Safari
[Nest] 7  - 07/30/2024, 12:00:00 AM VERBOSE [Microservices:SessionService] Deleted expired session token: iOS/Mobile Safari
[Nest] 7  - 07/30/2024, 12:00:00 AM VERBOSE [Microservices:SessionService] Deleted expired session token: Windows/Chrome
[Nest] 7  - 07/30/2024, 12:00:00 AM VERBOSE [Microservices:SessionService] Deleted expired session token: Linux/Chrome
[Nest] 7  - 07/30/2024, 12:00:00 AM VERBOSE [Microservices:SessionService] Deleted expired session token: Android/Chrome
[Nest] 7  - 07/30/2024, 12:00:00 AM VERBOSE [Microservices:SessionService] Deleted expired session token: iOS/Chrome
[Nest] 7  - 07/30/2024, 12:00:00 AM VERBOSE [Microservices:SessionService] Deleted expired session token: Linux/Firefox
[Nest] 7  - 07/30/2024, 12:00:00 AM VERBOSE [Microservices:SessionService] Deleted expired session token: Linux/Firefox
[Nest] 7  - 07/30/2024, 12:00:00 AM VERBOSE [Microservices:SessionService] Deleted expired session token: Linux/Firefox
[Nest] 7  - 07/30/2024, 12:00:00 AM VERBOSE [Microservices:SessionService] Deleted expired session token: Linux/Firefox
[Nest] 7  - 07/30/2024, 12:00:00 AM     LOG [Microservices:SessionService] Deleted 11 expired session tokens
[Nest] 7  - 07/30/2024, 12:00:00 AM VERBOSE [Microservices:JobRepository] Waiting for undefined queue to stop...
[Nest] 7  - 07/30/2024, 12:00:01 AM   DEBUG [Microservices:PersonService] Face 71ff7891-219c-4ec7-bc0d-f4076bdd6eaf only matched the face itself, skipping
[Nest] 7  - 07/30/2024, 12:26:06 AM   DEBUG [Microservices:VersionService] Running version check
[Nest] 7  - 07/30/2024, 1:26:06 AM   DEBUG [Microservices:VersionService] Running version check
[Nest] 7  - 07/30/2024, 2:26:06 AM   DEBUG [Microservices:VersionService] Running version check
[Nest] 7  - 07/30/2024, 3:26:06 AM   DEBUG [Microservices:VersionService] Running version check
[Nest] 7  - 07/30/2024, 4:26:06 AM   DEBUG [Microservices:VersionService] Running version check

And in my system I have five deleted non-deleted users.

osa911 commented 3 months ago

@ScuttleSE I had the same issue delete the user immediately feature works incorrectly sometimes. I guess if your user has a large amount of assets, it will fail. I read the code to find the issue, but it looks fine to me. To resolve this issue I manually changed the user's status from deleted to active in DB and triggered delete user again (but not immediately at this time) And in the morning it was fine.