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

Cannot arm concord232 alarm #5049

Closed
CTLS opened this issue Dec 23, 2016 · 8 comments · Fixed by #7789
Closed

Cannot arm concord232 alarm #5049

CTLS opened this issue Dec 23, 2016 · 8 comments · Fixed by #7789

Comments

@CTLS
Copy link
Contributor

CTLS commented Dec 23, 2016

Make sure you are running the latest version of Home Assistant before reporting an issue.

You should only file an issue if you found a bug. Feature and enhancement requests should go in the Feature Requests section of our community forum:

Home Assistant release (hass --version):
0.35.2

Python release (python3 --version):
Python 3.4.2

Component/platform:
Concord232 Alarm Control Panel

Description of problem:
Cannot arm alarm from the included arm home or arm away. disarm works fine though. Home assistant is using an incorrect arm type as seen in the log below

I believe this is the exact same issue as seen in the NX584 component issue #2194

Expected:

Problem-relevant configuration.yaml entries and steps to reproduce:

Traceback (if applicable):

Traceback (most recent call last):
  File "/usr/lib/python3.4/concurrent/futures/thread.py", line 54, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/srv/homeassistant/lib/python3.4/site-packages/homeassistant/core.py", line 1045, in execute_service
    service_handler.func(service_call)
  File "/srv/homeassistant/lib/python3.4/site-packages/homeassistant/components/alarm_control_panel/__init__.py", line 61, in alarm_service_handler
    getattr(alarm, method)(code)
  File "/srv/homeassistant/lib/python3.4/site-packages/homeassistant/components/alarm_control_panel/concord232.py", line 124, in alarm_arm_home
    self._alarm.arm('home')
  File "/home/homeassistant/.homeassistant/deps/concord232/client.py", line 28, in arm
    raise Exception('Invalid arm type')
Exception: Invalid arm type

Additional info:

@JasonCarter80

@CTLS
Copy link
Contributor Author

CTLS commented Dec 23, 2016

I have made a custom component by duplicating the concord232 component and changing the home arming from 'home' to 'stay' which fixed that, but I couldn't get the exit/away/auto to work. The concord232 server which this runs on seems to have an issue and with the little documentation I found I cant figure out if the alarm panel needs settings changed or if this is a bug in the component/ server

@JasonCarter80
Copy link
Contributor

JasonCarter80 commented Dec 23, 2016 via email

@CTLS
Copy link
Contributor Author

CTLS commented Dec 24, 2016

another issue I have now found with this, and this one seems more dangerous, is that if I do not enter a code on the call service it can be disarmed. I am not sure if this is normal with this alarm panel but it seems like something that shouldn't and I am not sure if the server or something somehow saved my code and is using it whenever I call disarm or what.

@balloobbot
Copy link

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 👍

@marbaugh
Copy link

If you go to "/srv/homeassistant/lib/python3.4/site-packages/homeassistant/components/alarm_control_panel/concord232.py" line 124 and edit the line from self._alarm.arm('home') to self._alarm.arm('stay') everything works great.

@MartinHjelmare
Copy link
Member

@marbaugh Please provide a pull request with the fix.

@CTLS
Copy link
Contributor Author

CTLS commented Mar 24, 2017

The issue still exists. I havent had a need to get anything other than stay mode to work, but the fix mentioned by @marbaugh is what I had been doing and it works.

@CTLS
Copy link
Contributor Author

CTLS commented May 27, 2017

@marbaugh have you figured out how to get the away mode to work. I submitted a PR to fix home/stay mode but haven't figured out the issue with away mode.

@home-assistant home-assistant locked and limited conversation to collaborators Sep 4, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants