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

"Home Assistant Repairs" detecting notify in Automation's as an "Error" #115566

Open
Coolie1101 opened this issue Apr 14, 2024 · 14 comments
Open

Comments

@Coolie1101
Copy link

Coolie1101 commented Apr 14, 2024

The problem

I keep getting the following repair error for multiple automations after automations are reloaded, please note, I have been using this notifier for a long time with no issues, it is currently working and sending notifications.

Uptime Kuma Notification uses an unknown service
The automation "Uptime Kuma Notification" (automation.uptime_kuma_notification) has an action that calls an unknown service: notify.telegram.

This error prevents the automation from running correctly. Maybe this service is no longer available, or perhaps a typo caused it.

To fix this error, [edit the automation](http://hassos:8123/config/automation/edit/1651758116418) and remove the action that calls this service.

Click on SUBMIT below to confirm you have fixed this automation.

What version of Home Assistant Core has the issue?

2024.4.1

What was the last working version of Home Assistant Core?

2024.3.3

What type of installation are you running?

Home Assistant OS

Integration causing the issue

Home Assistant Repair

Link to integration documentation on our website

https://www.home-assistant.io/integrations/repairs/

Diagnostics information

No response

Example YAML snippet

telegram_bot:
  - platform: polling
    api_key: !secret telegram_api_key
    allowed_chat_ids:
      - !secret telegram_chat_id_me
    parse_mode: html

notify:
  - platform: telegram
    chat_id: !secret telegram_chat_id_me
    name: Telegram

action:
  - service: notify.telegram
    data:
      message: >-
        {{ trigger.to_state.attributes.friendly_name }} is now {{ 'Online' if
        trigger.to_state.state == 'up' }} {{ 'Offline' if trigger.to_state.state
        == 'down' }} {{ 'Paused' if trigger.to_state.state == 'pause' }} {{
        'Unknown' if trigger.to_state.state == 'unknown' }}

Anything in the logs that might be useful for us?

No entries in log pertaining to this error.

Additional information

Most recent notificatio:

USI GATEWAY is now Online

@home-assistant
Copy link

Hey there @home-assistant/core, mind taking a look at this issue as it has been labeled with an integration (repairs) you are listed as a code owner for? Thanks!

Code owner commands

Code owners of repairs can trigger bot actions by commenting:

  • @home-assistant close Closes the issue.
  • @home-assistant rename Awesome new title Renames the issue.
  • @home-assistant reopen Reopen the issue.
  • @home-assistant unassign repairs Removes the current integration label and assignees on the issue, add the integration domain after the command.
  • @home-assistant add-label needs-more-information Add a label (needs-more-information, problem in dependency, problem in custom component) to the issue.
  • @home-assistant remove-label needs-more-information Remove a label (needs-more-information, problem in dependency, problem in custom component) on the issue.

(message by CodeOwnersMention)


repairs documentation
repairs source
(message by IssueLinks)

@erkr
Copy link

erkr commented Apr 14, 2024

Just a dumb question wrt the YAML snippet; is that all defined in an automation?
The first two sections should be defined outside the automations in the configuration.yaml:


telegram_bot:
  - platform: polling
    api_key: !secret telegram_api_key
    allowed_chat_ids:
      - !secret telegram_chat_id_me
    parse_mode: html

notify:
  - platform: telegram
    chat_id: !secret telegram_chat_id_me
    name: Telegram

@Coolie1101
Copy link
Author

Coolie1101 commented Apr 14, 2024

No, they are not all defined in the automation, the Telegram Integration and the Notify component is defined in configuration.yaml

FYI, the automations work as expected, and has been for years.

@erkr
Copy link

erkr commented Apr 14, 2024

Thanks for clarifying. I hope someone can confirm your issue.
I defined a number of notifiers myself (platforms: groups, rest, file and pushsafer), and use them in my automations.

None of them are causing repair errors. You could temporary try to use another platform then telegram in the notify. Maybe it is telegram related.

@Coolie1101
Copy link
Author

Maybe it is telegram related.

Maybe, but there isn't any related entries in the log, and why would it only show up now?

@erkr
Copy link

erkr commented Apr 14, 2024

There where several repair improvement over the latest releases. That can cause regression. Just try if it is telegram specific. The more specific you are, the bigger the chance someone looks into it.
The is nothing wrong with notification used in automations

@Coolie1101
Copy link
Author

I have multiple notifiers using Telegram, none of which is producing errors, except notify.telegram, I currently don't have any other notify integrations.

notify:
  - platform: telegram
    chat_id: !secret telegram_chat_id_me
    name: Telegram

  - platform: telegram
    chat_id: !secret telegram_chat_id_her
    name: Trini11

  - platform: telegram
    chat_id: !secret telegram_chat_id_niala
    name: Niala

  - platform: telegram
    chat_id: !secret telegram_chat_id_hassbianbot
    name: HassbianBot

  - platform: telegram
    chat_id: !secret telegram_chat_id_group
    name: Group

@erkr
Copy link

erkr commented Apr 14, 2024

Maybe something goes wrong in checking repairs using the same name as the platform.
But that is then telegram specific, as my pushsafer notifier also has Pushsafer as name without issues

@Coolie1101
Copy link
Author

Should I then report this as an issue under the Telegram Integration?

@erkr
Copy link

erkr commented Apr 14, 2024

I think that is a better guess indeed

@Coolie1101
Copy link
Author

Can we move this or I have to create new?

@erkr
Copy link

erkr commented Apr 14, 2024

I am just a user like you, but try if you can change the integration and link.
Otherwise creating a new one that points to this one will be fine

@Coolie1101
Copy link
Author

Seems like alot more is broken after updating to 2024.4.1, I just noticed a couple template errors in the logs relevant to #114907

@erkr
Copy link

erkr commented Apr 15, 2024

Not necessarily broken. Latest HA starts up faster. As a result more templates initially can have uninitialised sensors. I had to make some templates more robust to get rid of template errors during startup myself.

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