alexbelgium / hassio-addons

My homeassistant addons
MIT License
1.56k stars 223 forks source link

✨ [REQUEST] wger - custom env variables #591

Closed christophermichaelshaw closed 1 year ago

christophermichaelshaw commented 1 year ago

Which addon?

Wger

Is your feature request related to a problem? Please describe.

I would like to configure the env options (smtp, etc). Would it be possible to add custom env variables for the addon/docker image?

**If a new addon, have you checked on Google that such as addon doesn't already exists? N/A

Describe the solution you'd like

Add additional configuration options (custom env variables) to the addon in the UI.

Describe alternatives you've considered

Attempted to edit env variables and rebuild image.

Additional context

N/A

alexbelgium commented 1 year ago

Hi, adding env variable in the file /config/addons_config/wger/config.yaml should load them at app start.

Where would be the full list of env variables to reference in the documentation? Thanks

Edit : corrected the link

christophermichaelshaw commented 1 year ago

Here are the specific settings I am trying to load: https://wger.readthedocs.io/en/latest/production.html

I tried /confit/addons_config/wger/config.yaml

Are you sure it’s ../whee/.. ? I’m giving a try now.

On Wed, Dec 7, 2022 at 21:08 Alexandre @.***> wrote:

Hi, adding env variable in the file /config/addons_config/whee/config.yaml should load them at app start.

Where would be the full list of env variables to reference in the documentation? Thanks

— Reply to this email directly, view it on GitHub https://github.com/alexbelgium/hassio-addons/issues/591#issuecomment-1342050704, or unsubscribe https://github.com/notifications/unsubscribe-auth/APSZUKA2XB5T5PFMNIH3Z2DWMFUL3ANCNFSM6AAAAAASXNNOEM . You are receiving this because you authored the thread.Message ID: @.***>

--

alexbelgium commented 1 year ago

Ahah, it's autocorrect indeed :) thanks

christophermichaelshaw commented 1 year ago

No, THANK YOU!!

I'm receiving the following error when I attempt to start the addon, with and without the config.yaml file.

[FATAL tini (7)] exec /home/wger/entrypoint.sh failed: Exec format error

I'm guessing there might be an error in the entrypoint.sh...

On Wed, Dec 7, 2022 at 10:26 PM Alexandre @.***> wrote:

Ahah, it's autocorrect indeed :) thanks

— Reply to this email directly, view it on GitHub https://github.com/alexbelgium/hassio-addons/issues/591#issuecomment-1342131103, or unsubscribe https://github.com/notifications/unsubscribe-auth/APSZUKAPNB7TRYOUP4KQE2LWMF5P7ANCNFSM6AAAAAASXNNOEM . You are receiving this because you authored the thread.Message ID: @.***>

--

Christopher Shaw @.*** 425.435.8440

alexbelgium commented 1 year ago

This error just appeared now? Thanks

deku-m commented 1 year ago

Error still appears in 1.9.3:

[FATAL tini (7)] exec /home/wger/entrypoint.sh failed: Exec format error

christophermichaelshaw commented 1 year ago

Confirmed — it is still happening in 1.9.3

On Thu, Dec 8, 2022 at 01:51 deku-m @.***> wrote:

Error still appears in 1.9.3:

[FATAL tini (7)] exec /home/wger/entrypoint.sh failed: Exec format error

— Reply to this email directly, view it on GitHub https://github.com/alexbelgium/hassio-addons/issues/591#issuecomment-1342399293, or unsubscribe https://github.com/notifications/unsubscribe-auth/APSZUKHKAT6B7HO2P7TOZ5DWMGVSZANCNFSM6AAAAAASXNNOEM . You are receiving this because you authored the thread.Message ID: @.***>

--

alexbelgium commented 1 year ago

Looks like a mistake in their latest released versions, they don't publish in latest anymore for amd64 and aarch64... I'll look for older versions

alexbelgium commented 1 year ago

I've tagged an older version. Guess their latest image mixes amd64 and aarch64

christophermichaelshaw commented 1 year ago

@alexbelgium Thanks so much for your quick response!

I am able to start the addon but there are a few remaining issues: 1) The addon appears to reset the admin user account password to default at start:

Installed 1 object(s) from 1 fixture(s)
*** Password for user admin was reset to 'adminadmin'
Installed 3 object(s) from 1 fixture(s)
yarn install v1.22.19

2) The addon does not respect the port set in the addon config, e.g. 9928:80/tcp. Lots of addons attempt to use 8000/tcp as default so reimplementing the proxy would be helpful.

3) I receive some errors that seem to be permissions issues (see attached files), even after complete remove. wger_errors_20221208.rtf.zip

4) It appears they are still publishing for amd64.... https://hub.docker.com/r/wger/base/tags

alexbelgium commented 1 year ago
alexbelgium commented 1 year ago

Actually your log explains it : there is an issues with your database, I'll check

christophermichaelshaw commented 1 year ago

Thanks! I appreciate it very much!

After uninstall > Reboot > Purge > Reinstall I can get it booted but am running into issues with images and other content not loading.

I've configured NPM/nginx correctly, and even removed proxy services from the equation and still have permission errors.

Let me know if you need any additional logs or anything like that.

alexbelgium commented 1 year ago

Could you please paste your addon startup log? To see if scripts are correctly installed. Looking at your log, it seems there is no permission to the /data folder used to store data, which induces all those issues. Thanks!

github-actions[bot] commented 1 year ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.