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

Traefik 3 migration problems #215

Closed
49 of 62 tasks
maximuskowalski opened this issue Dec 28, 2022 · 36 comments
Closed
49 of 62 tasks

Traefik 3 migration problems #215

maximuskowalski opened this issue Dec 28, 2022 · 36 comments
Labels
bug Something isn't working help wanted Extra attention is needed

Comments

@maximuskowalski
Copy link
Collaborator

maximuskowalski commented Dec 28, 2022

Going to try to keep moving rather than get bogged down on issues so these are the apps not working currently.

  • bookstack - DB role
  • booksonic
  • calibre
  • calibre-web
  • delugevpn, needs proper test too.
  • gitea - another DB role
  • guacamole - 9720877
  • invoiceninja - DB
  • homeassistant - gateway timeout, no docs, might just be a URL thing
  • joplin - Bad Gateway - DB related
  • koel - DB
  • minecraft related roles - I don't know how to test other than the install. Roles are :- sandbox-mcrouter, sandbox-minecraft, sandbox-minecraft-bedrock
  • miniflux - 404 [ [FATAL] Unable to connect to the database: dial tcp: lookup miniflux-postgres: Try again ]
  • moviematch - probably just needs real plex instance [ error: Uncaught (in promise) TypeError: Cannot read property 'map' of undefined const availableLibraryNames = sections.MediaContainer.Directory.map( ]
  • mqtt - installs fine but not tested
  • nabarr - installs fine, untested
  • nextcloud - installs fine - may need to convert labels, regex middleware
  • node-red - 404 probably needs dehyphening - fixed 0cf3105
  • omegabrr - 404 - may just need config, no idea what it does.
  • paperless-ngx - has hyphen [ Error: Error -3 connecting to paperless_redis:6379. Temporary failure in name resolution..]
  • photoprism - might need testing on real server - see my error here
  • qbittorrentvpn - needs proper testing with a VPN account
  • rdtclient - needs proper testing with a real debrid account
  • rocketchat - no wiki so unsure if other steps reqd - timeout connecting to mongodb ( I think )
  • sabthrottle - installs but not able to test properly
  • stash - installs OK but have an error after running initial first run wizard - TypeError: Failed to fetch dynamically imported module: https://stash._mydomain_._com_/assets/Settings.b9790d5d.js See Traefik 3 migration problems #215 (comment) - d3e6da7
  • syncthing - installs fine, syncing things untested.
  • not sure how to deal with olivetin_traefik_middleware_api: "{{ traefik_global_middleware }}"
  • traefik_robotstxt - labels, regex middleware
  • transmissionvpn - needs testing with a VPN account
  • # default to blank
    whisparr_api_key: # should this get an ""
  • Ombi-X - should be migrated to an instance app instead of an arrX app
  • requestrrX - should be migrated to an instance app instead of an arrX app
  • rfloodX - should be migrated to an instance app instead of an arrX app
  • tandoor - fixed 3514327
  • tdarr
  • teamspeak (no web ui)
  • telegraf (no web ui)
  • thelounge
  • tika (no web ui)
  • tqm (no web ui)
  • trackarr
  • trilium
  • tubearchivist
  • tvheadend
  • unifi
  • unmanic
  • unpackerr (no web ui)
  • uptime-kuma
  • varken (no web ui)
  • vaultwarden
  • vnstat
  • whisparr
  • wikijs
  • wireguard
  • wizarr
  • wordpress
  • wrapperr
  • xbackbone - fixed a044f2a
  • xteve - I believe there is no ui, but app output is there (I presume the guide XML) so it seems fine
  • yacht
  • znc
@maximuskowalski maximuskowalski added bug Something isn't working help wanted Extra attention is needed labels Dec 28, 2022
@saltydk
Copy link
Member

saltydk commented Dec 28, 2022

popeen/Booksonic-Air#23 may be the cause, you could try and approach LSIO about it?

@saltydk
Copy link
Member

saltydk commented Dec 28, 2022

booksonic_docker_envs_default:
  PUID: "{{ uid }}"
  PGID: "{{ gid }}"
  TZ: "{{ tz }}"
  JAVA_OPTS: "-Dserver.use-forward-headers=true"

seems to resolve the issue you wrote on discord but to me it seems like it has other issues as well.

@maximuskowalski
Copy link
Collaborator Author

JAVA_OPTS: "-Dserver.use-forward-headers=true"

That was what fixed airsonic, I did try running with that but I don't think dockerfile is set up to take the ENV. Will be aiming to get as much as possible working ASAP for now.

@saltydk
Copy link
Member

saltydk commented Dec 28, 2022

It fixes the http vs https issue, since they use the same framework if I had to take a guess.

@maximuskowalski
Copy link
Collaborator Author

Wait.. I have been looking in the wrong spot then. :|
https://gitlab.com/Linuxserver.io/docker-booksonic-air/-/blob/master/Dockerfile

@saltydk
Copy link
Member

saltydk commented Dec 28, 2022

Dockerfile doesn't have anything to do with runtime stuff though. That is the entrypoint and S6 services in this case.

@maximuskowalski
Copy link
Collaborator Author

That is what I meant by "looking in the wrong spot" not the github vs gitlab.

@maximuskowalski
Copy link
Collaborator Author

Have merged my last lot of changes, TESTED UP TO L's, the Ms - Ss still to check.
Added to first post things that need to be followed up.

@owine
Copy link
Collaborator

owine commented Jan 3, 2023

not sure how to deal with olivetin_traefik_middleware_api: "{{ traefik_global_middleware }}"

I have no idea why this is there and it can simply be nuked.

@owine
Copy link
Collaborator

owine commented Jan 3, 2023

Committed changes that should get the db roles working again. Needed to replace var used for mysql root password since it changed with the new image.

2263c1a

@owine
Copy link
Collaborator

owine commented Jan 3, 2023

Committed changes that should get the db roles working again. Needed to replace var used for mysql root password since it changed with the new image.

2263c1a

This will cause CI to fail from this point forward as the var changes correspond to changes on the Saltbox traefik3 branch.

@RaneyDazed
Copy link
Contributor

makemkv seems to work fine, medusa is giving me an update error, not sure if its expected?

2023-01-03 17:03:40 WARNING  Thread_0 :: [] Unable to contact github, can't check for update: 
ConnectionError(MaxRetryError("HTTPSConnectionPool(host='api.github.com', port=443): Max retries exceeded with URL:
 /orgs/pymedusa (Caused by NewConnectionError('<urllib3.connection.VerifiedHTTPSConnection object at
 0x7f8b1cf50c40>: Failed to establish a new connection: [Errno -3] Try again'))"))

@maximuskowalski
Copy link
Collaborator Author

No problems with medusa for me and I was able to check for update.

@saltydk
Copy link
Member

saltydk commented Jan 4, 2023

miniflux is broken because these are empty:

miniflux_postgres_username:
miniflux_postgres_password:

@saltydk
Copy link
Member

saltydk commented Jan 4, 2023

I suspect the multi-instance work made the omit break the intended behavior.

@saltydk
Copy link
Member

saltydk commented Jan 4, 2023

0fb8aba should fix miniflux.

@saltydk
Copy link
Member

saltydk commented Jan 4, 2023

I'm not getting gateway timeout on homeassistant. May want to recheck that.

@saltydk
Copy link
Member

saltydk commented Jan 4, 2023

Committed changes that should get the db roles working again. Needed to replace var used for mysql root password since it changed with the new image.
2263c1a

This will cause CI to fail from this point forward as the var changes correspond to changes on the Saltbox traefik3 branch.

Should be fine after I made a temporary edit to checkout the traefik3 saltbox branch in the CI.

@maximuskowalski
Copy link
Collaborator Author

I'm not getting gateway timeout on homeassistant. May want to recheck that.

So yours worked fine and I can tick it off? I have a crappy VPS it sometimes doesn't work so good.

@maximuskowalski
Copy link
Collaborator Author

Photoprism: fails on my vps - no /dev/dri/

2023-01-05 11:52:51,247 p=1549934 u=root n=ansible | TASK [photoprism : Resources | Tasks | Docker | Create Docker Container | Create Docker Container] **********************************************************
2023-01-05 11:52:51,247 p=1549934 u=root n=ansible | Thursday 05 January 2023  11:52:51 +1100 (0:00:00.124)       0:01:40.372 ****** 
2023-01-05 11:52:58,637 p=1549934 u=root n=ansible | fatal: [localhost]: FAILED! => {"attempts": 1, "changed": false, "msg": "Error starting container 58e6fd4b95f3fd82a78238e7a8f6eaa211beaf70bc3cce1aab34b7c3b570e374: 500 Server Error for http+docker://localhost/v1.41/containers/58e6fd4b95f3fd82a78238e7a8f6eaa211beaf70bc3cce1aab34b7c3b570e374/start: Internal Server Error (\"error gathering device information while adding custom device \"/dev/dri\": no such file or directory\")"}

@RaneyDazed
Copy link
Contributor

I think I made that pr, should I remove that? or I guess should it be removed? Not sure how we're doing this.

@maximuskowalski
Copy link
Collaborator Author

What PR?

@RaneyDazed
Copy link
Contributor

for photoprism. I think I added /dev/dri when testing in SB mod. Must have forgotten to remove?

@maximuskowalski
Copy link
Collaborator Author

Oh, I am not sure - plex and other apps use it if it exists, so perhaps it might need to be checked for existence first. Once I get through all the role tests I'll come back to trying to fix things unless they have already been fixed.

If you want to try to fix it you are more than welcome to, just remember to do it on the T3 branch and test it on the T3 branch.

@owine
Copy link
Collaborator

owine commented Jan 6, 2023

Remove it as a default (set to blank) and do this in the role

- name: Docker Devices Task
  ansible.builtin.include_tasks: "{{ resources_tasks_path }}/docker/set_docker_devices_variable.yml"
  when: gpu.intel or gpu.nvidia

@RaneyDazed
Copy link
Contributor

Ok, give me a sec to get vscode up : )

@owine
Copy link
Collaborator

owine commented Jan 16, 2023

whisparr_api_key:

This is how the arrs are in the main repo so I think this is fine

@owine
Copy link
Collaborator

owine commented Jan 16, 2023

Stash works, issue is related to https://github.com/stashapp/stash/wiki/Authentication-Required-When-Accessing-Stash-From-the-Internet - since we deploy Authelia by default we should set the config bit appropriately

Not related to t3 migration so I am marking complete

d3e6da7

@owine
Copy link
Collaborator

owine commented Jan 16, 2023

Testing the following roles:

  • tandoor
  • tdarr
  • teamspeak (no web ui)
  • telegraf (no web ui)
  • thelounge
  • tika (no web ui)
  • tqm (no web ui)
  • trackarr
  • trilium
  • tubearchivist
  • tvheadend
  • unifi
  • unmanic
  • unpackerr (no web ui)
  • uptime-kuma
  • varken (no web ui)
  • vaultwarden
  • vnstat
  • whisparr
  • wikijs
  • wireguard
  • wizarr
  • wordpress
  • wrapperr
  • xbackbone - fixed a044f2a
  • xteve - I believe there is no ui, but app output is there (I presume the guide XML) so it seems fine
  • yacht
  • znc

@keldian
Copy link
Contributor

keldian commented Feb 12, 2023

omegabrr - 404 - may just need config, no idea what it does.

At the moment the root URL doesn't point to anything. There's no frontend—it's strictly for API requests via https://omegabrr.domain.tld/api/webhook/trigger. I suppose we could have Traefik map the endpoint to the root URL but meh.

@saltydk
Copy link
Member

saltydk commented Feb 21, 2023

What is the current status, anything that needs my intervention or?

@maximuskowalski
Copy link
Collaborator Author

Trying to keep first post updated - unchecked boxes need some looking at but I believe we have made a full first pass over everything. Some regex things that I am not sure about from that post.

nextcloud - installs fine - may need to convert labels, regex middleware
traefik_robotstxt - labels, regex middleware

@saltydk
Copy link
Member

saltydk commented Feb 28, 2023

Maybe edit the main post to have the full status as we go along as this all seems very confusing.

@saltydk
Copy link
Member

saltydk commented Feb 28, 2023

I don't see any problems with the Nextcloud regex middleware as I don't think that went through any changes.

@saltydk
Copy link
Member

saltydk commented Feb 28, 2023

traefik_robotstxt: Should be fixed now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

5 participants