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

Error updating app #197

Closed
kieeps opened this issue Feb 28, 2021 · 26 comments
Closed

Error updating app #197

kieeps opened this issue Feb 28, 2021 · 26 comments
Assignees
Labels
bug Something isn't working wontfix This will not be worked on

Comments

@kieeps
Copy link

kieeps commented Feb 28, 2021

Describe the bug
Noticed an upgrade was available for my installed apps, installed 1.3.0, available version 1.3.4.
But every time i try to update any of the apps i get an error saying the hostPath is wrong.

Sonarr error:
Error: [EINVAL] config.hostPath: attribute required [EINVAL] downloads.hostPath: attribute required [EINVAL] media.hostPath: attribute required

Radarr error:
Error: [EINVAL] config.hostPath: attribute required [EINVAL] downloads.hostPath: attribute required [EINVAL] media.hostPath: attribute required

Bazarr error:
Error: [EINVAL] config.hostPath: attribute required [EINVAL] downloads.hostPath: attribute required [EINVAL] media.hostPath: attribute required

Jackett error:
Error: [EINVAL] config.hostPath: attribute required

So basically anything with an hostpath it seems.

To Reproduce
Steps to reproduce the behavior:

  1. Go to 'Apps'
  2. Click on 'Installed Applications'
  3. Click on the 3 dots
  4. Click on 'Upgrade'
  5. Get error

Expected behavior
I expected the app to update to 1.3.4

Screenshots
If applicable, add screenshots to help explain your problem.

Additional context
The system is currently running TrueNAS-SCALE-21.02-MASTER-20210205-112916

@kieeps kieeps added the bug Something isn't working label Feb 28, 2021
@Ornias1993
Copy link
Member

Can you click on "edit" and add a screenshot?
Did you have ix_volumes or hostPath references set-up for these apps initially?

@Ornias1993 Ornias1993 self-assigned this Feb 28, 2021
@Ornias1993
Copy link
Member

I've looked at the history and I don't think there where any relevant changes between 1.3.0 and today at least.

It might even fix it to hit "edit" double check the settings and save them.

@kieeps
Copy link
Author

kieeps commented Feb 28, 2021

Can you click on "edit" and add a screenshot?
Did you have ix_volumes or hostPath references set-up for these apps initially?

yea i'w had hostPath set-up since the start (installed it as 1.3.0)
Edit and Save did not help though :/
383-1024

@Ornias1993
Copy link
Member

I would highly suggest not using hostPath for /config
because IX-Volumes (the non-hostpath Solution) automatically snapshots on update for those and rollback when you roll back the app ;-)

(actually I'm inclined to remove the option even for /config volumes, because it's quite fundemental for the Apps environment to NOT put those on a hostPath...)

But... that does not solve your issues at all.

@kieeps
Copy link
Author

kieeps commented Feb 28, 2021

yea i used hostPath for config only 'cus i moved from a docker-compose setup to this new fancy system :) for the sake of testing before going live i just copied it and ran it :)

is ix-volumes different from docker-like volumes? is it possible to copy current /config content in to a ix-container?

@Ornias1993
Copy link
Member

I reproduced the issue, i'll look into it shortly.
I'm not sure where the fault lays to be honest so it might take some time!

@Ornias1993
Copy link
Member

@tprelog I might require some assistance on this one.

Just so you know:
As it's 1.3.0 and also in 1.4.0, it's not related to the reshuffle I did. (thats a good thing)

@Ornias1993
Copy link
Member

ix-volumes different from docker-like volumes?

Read the wiki ;-)
I short: No, it's just a hostPath under the hood. But it's automatically managed by TrueNAS to ensure it snapsshots and rolls back with updates. which is quite relevant with config files specifically.

You can copy into /mnt/tank/ix-applications/releases/releasename/volumes/ix_volumes

@kieeps
Copy link
Author

kieeps commented Feb 28, 2021

ix-volumes different from docker-like volumes?

Read the wiki ;-)
I short: No, it's just a hostPath under the hood. But it's automatically managed by TrueNAS to ensure it snapsshots and rolls back with updates. which is quite relevant with config files specifically.

You can copy into /mnt/tank/ix-applications/releases/releasename/volumes/ix_volumes

obviously didn't read the wiki since that would have saved us from such a terrible question ;D yea i found them. would have been nice if i actually looked in "Storage" before asking :D i'll move the config over right away :)

@Ornias1993
Copy link
Member

Background:
It's related to this specific validation script:
https://github.com/truenas/middleware/blob/master/src/middlewared/middlewared/schema.py

@Ornias1993
Copy link
Member

Okey, some things to try after I managed to get it to update correctly:

  • for the moment disable hostPath on all but one of the fields (easier testing with less variables)

  • On the hostPath thats lift: First test changing the path back and forth with edit and try to upgrade again

  • If that doesn't work: create a path without CAPITALS or specialcharacters_

@Ornias1993
Copy link
Member

@tprelog I don't think your assistance is needed after all, it's most likely an upstream issue.

@Ornias1993
Copy link
Member

I got another error though, about the Path not existing.

@Ornias1993
Copy link
Member

ohh and please update your nightly install. A lot has changed in between now and 0205

@Ornias1993
Copy link
Member

If all else fails:

  • Disable hostPath on all of them
  • Upgrade
  • reenable hostPath on the two external storage locations (media and downloads)

@kieeps
Copy link
Author

kieeps commented Mar 1, 2021

I upgraded to the latest nightly before i went to work, connected to the truenas box from work to try and mess around with the hostPath, but now only 3 out of 4 start, the other 3 is "Deploying".
I'll look in to it when i get home, a bit to much to do on the phone :-)

@Ornias1993
Copy link
Member

I really suggest running the Alpha builds though, unless you want to actively develop Apps.
As TrueCharts is aimed at the alpha build.

@kieeps
Copy link
Author

kieeps commented Mar 1, 2021

I really suggest running the Alpha builds though, unless you want to actively develop Apps.
As TrueCharts is aimed at the alpha build.

I guess i could, but it sounds really boring though :-P i could easily fix the issue by just removing the app all together, reinstall them with the same hostPath and it will run, i just figured it was a bug i could help with :-) help as in "report it and hope for the best" that is :-D

@Ornias1993
Copy link
Member

If you want to help out, you could output the entire helm config (helm get all) for said containers to a txt file and email them to me directly. That way i can cross reference the output of different versions.

I've not been able to reproduce this specific error on latest builds of both TrueCharts and TrueNAS SCALE.
(my different error was fixable by my above advice)

IX didn't manage to either, so I suggest backing up and cleaning the slate, sorry.
I also can't find why this would trigger, taking the validation code into account.

@tprelog

This comment has been minimized.

@Ornias1993

This comment has been minimized.

@Ornias1993
Copy link
Member

Ornias1993 commented Mar 1, 2021

I've hided the whole storage talk complete and will not answer questions about it here. Before we know it this whole bug report is swarmed in off-topic shizzle.

Either make seperate issues or create a new one.
I would also suggest to always check recent merged PR's and (closed) issues before suggesting something that might or might not already be in-the-works

@Ornias1993
Copy link
Member

I'm going to close this as "wont fix" @kieeps as I don't think there is anything to do for us here.
It's either a strange fluke, related to an already depricated version or an upstream (SCALE) issue.

@kieeps
Copy link
Author

kieeps commented Mar 3, 2021

Yea i get it, sorry for not getting back to you about this but after update to latest nightly i cant even install new apps, i think it's something i messed up with the network section but I'll probably end up reinstalling and import the pool when i get time :-)
Went back to docker compose untill i get some spare time

@Ornias1993
Copy link
Member

@kieeps yes, with the ALPHA and BETA you really need to have the time to reinstall the Apps when they break.
Just like i'm also not providing migrations for next SCALE version, because I literally am refactoring all network related templates... (see the staging branch)

@truecharts-admin
Copy link
Collaborator

This issue is locked to prevent necro-posting on closed issues. Please create a new issue or contact staff on discord of the problem persists

@truecharts truecharts locked and limited conversation to collaborators Feb 10, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working wontfix This will not be worked on
Projects
None yet
Development

No branches or pull requests

4 participants