nielsfaber / alarmo

Easy to use alarm system integration for Home Assistant
1.33k stars 121 forks source link

Actions doesn't work when triggered by "master" #938

Closed burbon-pl closed 4 months ago

burbon-pl commented 4 months ago

Alarmo Version

v1.10.1

HA Version

2024.5.2

Bug description

Action works when attached to alarm trigger in any particular area, but stops working when attached to master "area" alarm trigger.

Steps to reproduce

-Add two areas -add PIR sensor to one of them -set action to do whatever action you want (triggering wifi relay in my case) when alarm triggered by "master" area.

Relevant log output

No response

nielsfaber commented 4 months ago

I don't agree with this bug, for me (and many others) it works fine.

Here's my configuration: image

From the debug log I can see that the automation is indeed exectued when the alarm is triggered:

2024-06-02 08:44:21.312 DEBUG (MainThread) [custom_components.alarmo.automations] state of alarm_control_panel.master is updated from armed_home to triggered 2024-06-02 08:44:21.312 DEBUG (MainThread) [custom_components.alarmo.automations] Executing automation 1717309887

burbon-pl commented 3 months ago

"It woks for me" doesn't prove anything. My configuration looks exactly the same (apart of that "script" in Task of course - i have "relay on" function there). If I set area to "master", when PIR detects movement the alarm zone where the PIR belongs triggers, the master also - but the relay doesn't fire up. If I change area from "master" to one of the subzones - relay reacts properly. I've just made another test: with area set to "master", but the event set to "Alarm no longer triggered", the relay works after alarm ends and alarm arms back (ca. 1 minute from alarm triggering).

burbon-pl commented 3 months ago

It also works with "Alarm is armed", "Alarm is disarmed" and "Failed to arm" events. Just the most important "Alarm is triggered" doesn't work. Alarmo v1.10.2 HA 2024.6.0

nielsfaber commented 3 months ago

With the information you provided I was unable to reproduce the issue. So I don’t really know how to help you. Perhaps you can make a screen video demonstrating what you do and how the system responds?

burbon-pl commented 3 months ago

Zrzut ekranu z 2024-06-06 21-01-28 Zrzut ekranu z 2024-06-06 21-01-59 Zrzut ekranu z 2024-06-06 21-02-09 There are the screenshots. Every other combination works except of area "master" and event "Alarm is triggered".

nielsfaber commented 3 months ago

I just created this action to duplicate yours: image

Next, with the siren off, I arm the master alarm. image

Next I trigger a sensor linked to an area, such that the master alarm is triggered. The siren was turned on: image

Same as previous, it works OK for me. I don't see any issue, so I don't know how to help you.

burbon-pl commented 3 months ago

Zrzut ekranu z 2024-06-06 21-10-50 Here the log from two subsequent tests: -first - the alarm fails to arm because of one sensor missing. Event set to "failed to arm", area "master". Relay fires up. -second - the alarm is triggered by one of the sensors. Event set to "alarm is triggered", area "master". Action doesn't work.

burbon-pl commented 3 months ago

OK, I have no idea what is wrong. Thanks anyway.