mirror of
https://github.com/koel/koel.git
synced 2026-04-25 08:46:00 +03:00
[GH-ISSUE #1286] Postgres error while adding files on fresh install #736
Labels
No labels
Authentication
Dependencies
Documentation
Feature Request
Flac
Help Wanted
Installation/Setup
Integration
Mobile
PR Welcome
Pending Release
Performance
Playlist
S3
Search
Sync
[Pri] Low
[Pri] Normal
[Status] Keep Open
[Status] Needs Author Reply
[Status] Needs Review
[Status] Stale
[Status] Will Implement
[Type] Blessed
[Type] Bug
[Type] Duplicate
[Type] Enhancement
[Type] Help Request
[Type] Question
[Type] Task
pull-request
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/koel-koel#736
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @vitoyucepi on GitHub (Mar 4, 2021).
Original GitHub issue: https://github.com/koel/koel/issues/1286
Describe the bug
I'm creating my own container image with koel and pgsql.
You can find my build script and images in this gitlab repo.
On a clean install of postgresql, I cannot add new files until I make three requests that fail.
Error log
As you can see in the log the first 3 queries were failed with database errors.
The fourth query is ok.
This bug is also reproducible with https://github.com/Hyzual/docker-koel but with pgsql instead of mysql.
To reproduce
Steps to reproduce the behavior:
git clone https://gitlab.com/vito-containers/koel.gitpdo_mysqlwithpdo_pgsqlin file/containers/5.0/php-apache/Dockerfile./build.sh 5.0 php-apacheordocker build -t "registry.gitlab.com/vito-containers/koel/php-apache:5.0" ./containers/5.0/php-apache/docker-compose upordocker-compose up -dto start in background.http://localhost:8080and login with loginadmin@localhostand passwordadmin.databasecontainer log and500errors inkoelcontainer log.Expected behavior
Clean installation with pgsql should work ok, but it seems the problem with scheme.
Environment
Additional context
I've tested this bug with
postgresandmysqlcontainer images.There is no error in mysql and mariadb.
It seems the problem only present for the first 3 queries to new instance.
I've started using postgres a while ago because of memory and storage usage
Also there's no need to create your own images to override default values to
utf8mb4here and there.@vitoyucepi commented on GitHub (Mar 5, 2021):
Maybe related to laravel/framework#4897
@phanan commented on GitHub (Mar 25, 2021):
This might not be the answer you're looking for but if MySQL works, can you use it? I don't use Postgres myself and the whole DB layer is done by Laravel's Eloquent, so there's very little I can help here.
@vitoyucepi commented on GitHub (Mar 25, 2021):
Now I'm using koel with mariadb. It seems to work fine.
But I've found many other issues, like infinite loop for console commands and race condition for library scanning.
Didn't create issues about them.
As I already said but some time ago I think that postgresql was too complicated for me.
Now I'm thinking that postgresql is far more user-friendly technology because of docker image.
There's no need to configure anything. There's no need to create a custom container image because of utf8 by default.
Also there should be a warning in the docs, that postgresql is not officially supported by koel because of this bug.
@vitoyucepi commented on GitHub (Mar 26, 2021):
@phanan so what is the status of this issue.
Is it closed because won't fix?
Or maybe it should be opened with help wanted tag?
@Hyzual commented on GitHub (May 20, 2021):
I reproduced this exact issue while testing https://github.com/koel/docker/pull/78
@phanan commented on GitHub (May 21, 2021):
Hmm, looks like I'll have to install Postres and give it a try.
@phanan commented on GitHub (May 21, 2021):
@vitoyucepi Sorry I just realized I haven't answered your question. Yeah I was inclined to mark this as won't fix, as I thought it was an edge-cased issue. But now that it doesn't look like one, I've reopened the issue and put a "Help Wanted" label.