openwrt-luci-rpc icon indicating copy to clipboard operation
openwrt-luci-rpc copied to clipboard

Version check not working for Turris routers

Open dvenza opened this issue 3 years ago • 1 comments

  • openwrt-luci-rpc version: latest
  • Python version: all
  • Operating System: all

The version check performed in _determine_if_legacy_version(self) does not work for TurrisOS routers, mine for example reports:

# cat /etc/os-release
NAME="TurrisOS"
VERSION="5.2.5"
ID="turrisos"
ID_LIKE="lede openwrt"
PRETTY_NAME="TurrisOS 5.2.5"
VERSION_ID="5.2.5"
HOME_URL="https://www.turris.cz/"
BUG_URL="https://gitlab.nic.cz/groups/turris/-/issues/"
SUPPORT_URL="https://www.turris.cz/support/"
BUILD_ID="7378c94"
OPENWRT_BOARD="mvebu/cortexa9"
OPENWRT_ARCH="arm_cortex-a9_vfpv3-d16"
OPENWRT_TAINTS="busybox"
OPENWRT_DEVICE_MANUFACTURER="CZ.NIC"
OPENWRT_DEVICE_MANUFACTURER_URL="https://www.turris.cz/"
OPENWRT_DEVICE_PRODUCT="Turris Omnia"
OPENWRT_DEVICE_REVISION="v0"
OPENWRT_RELEASE="TurrisOS 5.2.5 7378c949270067d0a633cfd892b9dd6416581afa"

It runs a recent version of luci (rpc) and the check decides it is speaking to an old version instead.

This cause the home assistant integration to break, like in 50758, since it tries to use the old net.arptable method.

dvenza avatar Aug 23 '21 20:08 dvenza

ugly fix is appending -snapshot to VERSION_ID. But definitely not the solution. I think the version checko should use BUILD_ID revision (is still incremented even if git is used?)

vk496 avatar Nov 22 '21 11:11 vk496