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

Integration doesn't work with RouterOS 7.15.3 #375

Open
fa35ife opened this issue Oct 2, 2024 · 17 comments
Open

Integration doesn't work with RouterOS 7.15.3 #375

fa35ife opened this issue Oct 2, 2024 · 17 comments
Labels
bug Something isn't working stale

Comments

@fa35ife
Copy link

fa35ife commented Oct 2, 2024

Describe the issue

integration doesn't work

How to reproduce the issue

Steps to reproduce the behavior:
Upgrade Router OS to 7.15.3 on device

Expected behavior

It should work like on Router OS <7.13

Screenshots

Screenshot from 2024-10-02 22-56-56
Screenshot from 2024-10-02 22-58-10

Software versions

  • Home Assistant version: HA 2024.9.3
  • Mikrotik Router integration version: v2.1.4
  • Mikrotik Hardware: L009UiGS-2HaxD
  • RouterOS version: v7.15.3

Diagnostics data

no

Traceback/Error logs

This error originated from a custom integration.

Logger: custom_components.mikrotik_router.mikrotikapi
Source: custom_components/mikrotik_router/mikrotikapi.py:95
integration: Mikrotik Router (documentation, issues)
First occurred: 1 October 2024 at 02:17:29 (34 occurrences)
Last logged: 22:41:24

Mikrotik 192.168.88.1 error while building list for path /caps-man/registration-table : no such command or directory (caps-man), no such command prefix

Additional context

@fa35ife fa35ife added the bug Something isn't working label Oct 2, 2024
@fa35ife fa35ife changed the title [Bug] Integration doesn't work with RouterOS 7.15.3 Oct 2, 2024
@djrawe
Copy link

djrawe commented Oct 4, 2024

Same problem with routerOS 7.16

@arfoll
Copy link

arfoll commented Oct 5, 2024

I have it working with 7.16 using tip of master, pretty sure 1bbedfa is fixing the issue you have with the wifi package changes. My router runs capsman (new one) and at least that works.

@pomah86
Copy link

pomah86 commented Oct 9, 2024

Is there anyone who can merge this and release a new version? Seems like the maintainer does not have time for this anymore

@djexit
Copy link

djexit commented Oct 10, 2024

The problem occurs with the wifi-qcom and wifi-qcom-ac packages because capsman is no longer available. The integration looks for the path "/caps-man/registration-table", which no longer exists. Now it should be "/interface/wifi/registration-table"

@pomah86
Copy link

pomah86 commented Oct 10, 2024

The problem occurs with the wifi-qcom and wifi-qcom-ac packages because capsman is no longer available. The integration looks for the path "/caps-man/registration-table", which no longer exists. Now it should be "/interface/wifi/registration-table"

I think that is established by now, the problem is that the maintainer does not have time to update this, and the forks that I have tried can not be used in HACS...

@7lano
Copy link

7lano commented Oct 18, 2024

same problem, same cause

2024-10-18 02:02:33.921 DEBUG (SyncWorker_2) [custom_components.mikrotik_router.mikrotikapi] API query: /caps-man/registration-table
2024-10-18 02:02:33.922 ERROR (SyncWorker_2) [custom_components.mikrotik_router.mikrotikapi] Mikrotik 10.10.12.34 error while building list for path /caps-man/registration-table : no such command or directory (caps-man), no such command prefix
2024-10-18 02:02:33.923 DEBUG (MainThread) [custom_components.mikrotik_router.coordinator] Finished fetching Mikrotik WiFi data in 1.164 seconds (success: False)

@enzon0
Copy link

enzon0 commented Oct 18, 2024

three changes to the file: \config\custom_components\mikrotik_router\coordinator.py
from "/caps-man/registration-table", to "/interface/wifi/registration-table"
from "/interface/{wifimodule}", to "/interface/wifi"
from "/interface/{wifimodule}/registration-table", to "/interface/wifi/registration-table"

I'm not sure if everything works, but the integration starts and shows the data I use.

@Extrapilot1
Copy link

Guys-
Not the same error, but for many, a problem. With the new MT software, they have removed from the CLI menu options that are empty. Example, AC2 seems to not have much in the way of system health in the sense some other hardware does- no fan, no power monitoring etc. So, you will not find a system/health option. A lot of the code in the Integration here assumes these paths must exist. But, it seems to function OK if you bypass the check(s) in question. Some have problems with CAPSMAN, but with the System/Health thing, which will catch a lot of people, its an edit to the file coordinator.py under the mikrotik_router custom component (depends on where that is installed on your setup. Its found starting at line 1347 for a get_system_health routine. In that, there is a check where if the firmware is <7 it does one thing, but for anything else, it does another thing. That secondary path is the problem. It is a check on line 1370, where there is an elif 0 < self.major_fw_version >=7: conditional. Copy that line and paste it just below, and comment it with a # sign, so you have the original to reference. And then just modify the check to something like elif 8< self.major_fw_version >=9:

That will bypass that second section of code which is looking as /system/health, and the system seems to be fine without it. Probably requires a HA restart to reload the config.

coordinator.zip

@DhamSky
Copy link

DhamSky commented Oct 23, 2024

Same problem with routerOS 7.15

@djrawe
Copy link

djrawe commented Oct 24, 2024

three changes to the file: \config\custom_components\mikrotik_router\coordinator.py from "/caps-man/registration-table", to "/interface/wifi/registration-table" from "/interface/{wifimodule}", to "/interface/wifi" from "/interface/{wifimodule}/registration-table", to "/interface/wifi/registration-table"

I'm not sure if everything works, but the integration starts and shows the data I use.

It’s working for me. Thank you very much.

@ilgarmehmetali
Copy link

FYI. self.major_fw_version only gets populated if user group has all three of "write, policy, reboot" policies.
If you dont give any of these policies integration works. Alas with limited functionality.

@lukyrys
Copy link

lukyrys commented Nov 4, 2024

+1

Copy link

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

@github-actions github-actions bot added the stale label Nov 19, 2024
@7lano
Copy link

7lano commented Nov 19, 2024

@tomaae wake up please

@github-actions github-actions bot removed the stale label Nov 20, 2024
Copy link

github-actions bot commented Dec 4, 2024

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

@github-actions github-actions bot added the stale label Dec 4, 2024
@h1ghrise
Copy link

h1ghrise commented Dec 5, 2024

not stale! Broken and not being maintained anymore..

@github-actions github-actions bot removed the stale label Dec 6, 2024
Copy link

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

@github-actions github-actions bot added the stale label Dec 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working stale
Projects
None yet
Development

No branches or pull requests