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

[enhancement]: improve unsatisfied dependency error message #2762

Open
1 task done
mcm1957 opened this issue May 23, 2024 · 1 comment
Open
1 task done

[enhancement]: improve unsatisfied dependency error message #2762

mcm1957 opened this issue May 23, 2024 · 1 comment

Comments

@mcm1957
Copy link
Contributor

mcm1957 commented May 23, 2024

No existing issues.

  • There is no existing issue for my request.

Description

At a multihost system the following occured

2024-05-23 15:48:00.630 - error: host.iobroker startInstance system.adapter.tankerkoenig.0 Adapter dependency not fulfilled on any host: required adapter "admin" has wrong version. Installed "6.13.15", required ">=6.13.16"!
bkunder@iobroker:~$ iobroker update
Used repository: stable
Adapter    "admin"        : 6.13.16  , installed 6.13.16

The adapter required admin 6.13.16. But admin is a global dependency and at the slave an older admin was installed.

The problem for be improved is that the displayed error message does not clearly state why the test fails. A normal user would not guess the the slve is failing the check.

Why?

Users should be able to solve a problem by undersatnding error messages

How?

Some Ideas:

  • Log details about the host or hosts failing the check.
  • Or at least adapt the error text by using "Adapter dependencies nor fulfiller on ALL host"
@mcm1957 mcm1957 changed the title [enhancement]: improve unsatisfied dependency message [enhancement]: improve unsatisfied dependency error message May 23, 2024
@foxriver76
Copy link
Collaborator

foxriver76 commented May 24, 2024

I don't get the scenario completely, for a global dependency the message seems to be good if no admin satisfied the condition. Or what do you mean exactly. I mean for a local dep it makes sense to mention the host on which it was not found but for global this is fine.

I mean it already states Adapter dependency not fulfilled on any host: replacing it by your recommendation gives the feeling it would need to be present on every host..

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants