core icon indicating copy to clipboard operation
core copied to clipboard

HA сonfiguration check doesn't correspond to HA error check in log

Open Spirituss opened this issue 4 years ago • 25 comments

The problem

In some typical cases, such as when HA configuration has syntactic error or automation with no data parameter, standard configuration check tool doesn't show any error, while HA log shows it and stops the integrations with such errors. This is inconsistent behavour of the system. The expected behaviour is to shows all config check errors in HA config check tool (configurations - server controls - check configuration).

What is version of Home Assistant Core has the issue?

core-2021.5.1

What was the last working version of Home Assistant Core?

none

What type of installation are you running?

Home Assistant Container

Integration causing the issue

System tool

Link to integration documentation on our website

https://www.home-assistant.io/getting-started/configuration/

Example YAML snippet

automation:
  - alias: check
    trigger:
      platform: sun
    action:
      service: system_log.write
      data:
        message: "sunset"

Anything in the logs that might be useful for us?

2021-05-31 20:07:38 ERROR (MainThread) [homeassistant.config] Invalid config for [automation]: required key not provided @ data['event']. Got None. (See ?, line ?).

Additional information

No response

Spirituss avatar May 31 '21 17:05 Spirituss

There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved with the latest updates. Please make sure to update to the latest Home Assistant version and check if that solves the issue. Let us know if that works for you by adding a comment 👍 This issue has now been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.

github-actions[bot] avatar Sep 10 '21 09:09 github-actions[bot]

It is still the serious headache, because HA doesn't provide correct compilation information for some cases nevertheless it can.

Spirituss avatar Sep 10 '21 09:09 Spirituss

There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved with the latest updates. Please make sure to update to the latest Home Assistant version and check if that solves the issue. Let us know if that works for you by adding a comment 👍 This issue has now been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.

github-actions[bot] avatar Dec 12 '21 20:12 github-actions[bot]

Still is the case. I'm surprised that such a core problem has not been solved yet.

Spirituss avatar Dec 13 '21 04:12 Spirituss

Any update on this very annoying issue?

Spirituss avatar Jan 19 '22 10:01 Spirituss

There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved with the latest updates. Please make sure to update to the latest Home Assistant version and check if that solves the issue. Let us know if that works for you by adding a comment 👍 This issue has now been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.

github-actions[bot] avatar Apr 28 '22 11:04 github-actions[bot]

Nothing was changed. Still very illogical behaviour.

Spirituss avatar Apr 28 '22 15:04 Spirituss

There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved with the latest updates. Please make sure to update to the latest Home Assistant version and check if that solves the issue. Let us know if that works for you by adding a comment 👍 This issue has now been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.

github-actions[bot] avatar Aug 03 '22 16:08 github-actions[bot]

@Spirituss it seems that #77258 addresses this issue, should we link this issue to the PR?

AmadeusW avatar Nov 10 '22 18:11 AmadeusW

There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved with the latest updates. Please make sure to update to the latest Home Assistant version and check if that solves the issue. Let us know if that works for you by adding a comment 👍 This issue has now been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.

The problem is still relevant

Spirituss avatar Apr 17 '23 07:04 Spirituss

There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved with the latest updates. Please make sure to update to the latest Home Assistant version and check if that solves the issue. Let us know if that works for you by adding a comment 👍 This issue has now been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.

Bump

Spirituss avatar Jul 16 '23 20:07 Spirituss

There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved with the latest updates. Please make sure to update to the latest Home Assistant version and check if that solves the issue. Let us know if that works for you by adding a comment 👍 This issue has now been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.

@Spirituss it seems that #77258 addresses this issue, should we link this issue to the PR? @AmadeusW I suppose this pr could solve the problem. So please link it.

Spirituss avatar Oct 15 '23 11:10 Spirituss

There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved with the latest updates. Please make sure to update to the latest Home Assistant version and check if that solves the issue. Let us know if that works for you by adding a comment 👍 This issue has now been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.

@AmadeusW The problem still exists.

Spirituss avatar Jan 13 '24 13:01 Spirituss

There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved with the latest updates. Please make sure to update to the latest Home Assistant version and check if that solves the issue. Let us know if that works for you by adding a comment 👍 This issue has now been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.

bump

Spirituss avatar Apr 12 '24 19:04 Spirituss

There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved with the latest updates. Please make sure to update to the latest Home Assistant version and check if that solves the issue. Let us know if that works for you by adding a comment 👍 This issue has now been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.

bump

Spirituss avatar Jul 15 '24 11:07 Spirituss

@AmadeusW @ Here is another case of such a problem:

The error from the log:

2024-10-07 21:25:42.732 ERROR (MainThread) [homeassistant.components.automation] Automation with alias 'cover_pack_open_window' could not be validated and has been disabled: required key not provided @ data['trigger'][0]['platform']. Got None

But the yaml config check return no errors.

Spirituss avatar Oct 08 '24 10:10 Spirituss

There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved with the latest updates. Please make sure to update to the latest Home Assistant version and check if that solves the issue. Let us know if that works for you by adding a comment 👍 This issue has now been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.

Still has not been solved. I'm sad.

Spirituss avatar Jan 23 '25 17:01 Spirituss

I'm not a regular dev on this project. I was only curious how config validation works, and I stumbled upon your issue report. Each component is responsible for its own configuration validation. This is for both the YAML check config, as well as startup configurations. They seem to usually be two different things. Although I don't have a full grasp of it yet, it seems that YAML configuration is being phased out somewhat. What I really wanted to say to you is that the original issue reported on this for the "automation" component may very well be resolved, and the nature of your issue report is too generic for anyone to work on. You might have more luck opening a report for each component that gives you this issue, then the team assigned to that component can look into it.

fosterchuck avatar Mar 25 '25 22:03 fosterchuck

Oh hi there 👋

Going through our backlogs noticed this one. When this issue was reported, it would have been valid. Nowadays, this is no longer the case.

Configuration checking only looks for issues that may cause start-up issues. Issues with automations, for example, are provided by the automation integration itself at runtime. A single non-working integration doesn't stop the rest of Home Assistant and the automation to work.

This behavior has been refined over the past 2 years and is currently pretty much final.

This is also described in our UI nowadays:

Image

I'm going to close this issue for that reason.

../Frenck

frenck avatar Apr 15 '25 20:04 frenck