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

OpenWRT: command "arp" has failed #3597

Closed
timkgh opened this issue Sep 15, 2021 · 3 comments
Closed

OpenWRT: command "arp" has failed #3597

timkgh opened this issue Sep 15, 2021 · 3 comments
Assignees
Milestone

Comments

@timkgh
Copy link

timkgh commented Sep 15, 2021

2021/09/15 01:06:11.931024 24345#18 [debug] command "arp" has failed: "exec: \"arp\": executable file not found in $PATH" code:-1

On OpenWRT arp seems to be defined as a shell function arp() { cat /proc/net/arp; }

Should AGH switch to using ip neigh if ip is installed and fall back to arp, if it exists as an executable or just look in /proc/net/arp?

@timkgh timkgh changed the title OpenWRT: OpenWRT: command "arp" has failed Sep 15, 2021
@ainar-g
Copy link
Contributor

ainar-g commented Sep 15, 2021

Hello and thanks for the suggestion. I actually see a few issues in our current implementation:

  1. AGH doesn't use alternative ways of getting the ARP info from the kernel, including /proc/net/arp and ip neigh.

  2. AGH doesn't stop the attempts to get the client info after several failures. One could assume that this is a feature, since the arp binary can technically still appear later, but I personally think that this is a rather weak excuse.

  3. This functionality should really be moved to package aghnet and not just stay in package home.

I think we can refactor it all together in the v0.108 cycle.

@ainar-g ainar-g added this to the v0.108.0 milestone Sep 15, 2021
adguard pushed a commit that referenced this issue Mar 15, 2022
Merge in DNS/adguard-home from 3597-wrt-netlink to master

Updates #3597.

Squashed commit of the following:

commit 1709582
Merge: 0507b6e e7b3c99
Author: Eugene Burkov <E.Burkov@AdGuard.COM>
Date:   Tue Mar 15 20:25:18 2022 +0300

    Merge branch 'master' into 3597-wrt-netlink

commit 0507b6e
Author: Eugene Burkov <E.Burkov@AdGuard.COM>
Date:   Tue Mar 15 20:21:29 2022 +0300

    all: imp code

commit 71f9758
Author: Eugene Burkov <E.Burkov@AdGuard.COM>
Date:   Wed Mar 9 18:03:48 2022 +0500

    aghnet: imp naming

commit c949e76
Author: Eugene Burkov <E.Burkov@AdGuard.COM>
Date:   Wed Mar 9 17:26:30 2022 +0500

    all: imp code, docs

commit cf605dd
Author: Eugene Burkov <E.Burkov@AdGuard.COM>
Date:   Tue Mar 8 15:33:52 2022 +0500

    all: imp code, docs

commit 2960c65
Author: Eugene Burkov <E.Burkov@AdGuard.COM>
Date:   Sun Mar 6 21:34:58 2022 +0500

    all: imp code & docs, fix tests

commit 29c049f
Author: Eugene Burkov <E.Burkov@AdGuard.COM>
Date:   Wed Mar 2 20:45:34 2022 +0300

    all: add arpdb
@EugeneOne1
Copy link
Member

EugeneOne1 commented Mar 15, 2022

@timkgh, hello again. We've improved ARP-related logic a bit. It's available in the latest edge build and it should use some different sources of data including parsing the /proc/net/arp file. It should also notify about occured errors only once at the start. Could you please check if it fixes the issue for you?

adguard pushed a commit that referenced this issue Mar 22, 2022
Merge in DNS/adguard-home from 3142-fix-clients to master

Updates #3142.
Updates #3597.

Squashed commit of the following:

commit 4dcabed
Author: Eugene Burkov <E.Burkov@AdGuard.COM>
Date:   Tue Mar 22 15:13:15 2022 +0300

    all: imp log of changes

commit 481088d
Author: Eugene Burkov <E.Burkov@AdGuard.COM>
Date:   Tue Mar 22 14:36:44 2022 +0300

    all: swap arp and rdns priority
adguard pushed a commit that referenced this issue Mar 22, 2022
Merge in DNS/adguard-home from fix-arp-names to master

Updates #3597.

Squashed commit of the following:

commit b4737a3
Author: Eugene Burkov <E.Burkov@AdGuard.COM>
Date:   Tue Mar 22 16:55:52 2022 +0300

    home: fix adding client
@EugeneOne1
Copy link
Member

@timkgh, hello. I'll close the issue for now. Feel free to file new ones if you'll face any bugs with our solution.

@ainar-g ainar-g modified the milestones: v0.108.0, v0.107.7 Jun 2, 2022
adguard pushed a commit that referenced this issue Jun 2, 2022
Merge in DNS/adguard-home from 3142-fix-clients to master

Updates #3142.
Updates #3597.

Squashed commit of the following:

commit 4dcabed
Author: Eugene Burkov <E.Burkov@AdGuard.COM>
Date:   Tue Mar 22 15:13:15 2022 +0300

    all: imp log of changes

commit 481088d
Author: Eugene Burkov <E.Burkov@AdGuard.COM>
Date:   Tue Mar 22 14:36:44 2022 +0300

    all: swap arp and rdns priority
heyxkhoa pushed a commit to heyxkhoa/AdGuardHome that referenced this issue Mar 20, 2023
Merge in DNS/adguard-home from 3597-wrt-netlink to master

Updates AdguardTeam#3597.

Squashed commit of the following:

commit 1709582
Merge: 0507b6e e7b3c99
Author: Eugene Burkov <E.Burkov@AdGuard.COM>
Date:   Tue Mar 15 20:25:18 2022 +0300

    Merge branch 'master' into 3597-wrt-netlink

commit 0507b6e
Author: Eugene Burkov <E.Burkov@AdGuard.COM>
Date:   Tue Mar 15 20:21:29 2022 +0300

    all: imp code

commit 71f9758
Author: Eugene Burkov <E.Burkov@AdGuard.COM>
Date:   Wed Mar 9 18:03:48 2022 +0500

    aghnet: imp naming

commit c949e76
Author: Eugene Burkov <E.Burkov@AdGuard.COM>
Date:   Wed Mar 9 17:26:30 2022 +0500

    all: imp code, docs

commit cf605dd
Author: Eugene Burkov <E.Burkov@AdGuard.COM>
Date:   Tue Mar 8 15:33:52 2022 +0500

    all: imp code, docs

commit 2960c65
Author: Eugene Burkov <E.Burkov@AdGuard.COM>
Date:   Sun Mar 6 21:34:58 2022 +0500

    all: imp code & docs, fix tests

commit 29c049f
Author: Eugene Burkov <E.Burkov@AdGuard.COM>
Date:   Wed Mar 2 20:45:34 2022 +0300

    all: add arpdb
heyxkhoa pushed a commit to heyxkhoa/AdGuardHome that referenced this issue Mar 20, 2023
Merge in DNS/adguard-home from 3142-fix-clients to master

Updates AdguardTeam#3142.
Updates AdguardTeam#3597.

Squashed commit of the following:

commit 4dcabed
Author: Eugene Burkov <E.Burkov@AdGuard.COM>
Date:   Tue Mar 22 15:13:15 2022 +0300

    all: imp log of changes

commit 481088d
Author: Eugene Burkov <E.Burkov@AdGuard.COM>
Date:   Tue Mar 22 14:36:44 2022 +0300

    all: swap arp and rdns priority
heyxkhoa pushed a commit to heyxkhoa/AdGuardHome that referenced this issue Mar 20, 2023
Merge in DNS/adguard-home from fix-arp-names to master

Updates AdguardTeam#3597.

Squashed commit of the following:

commit b4737a3
Author: Eugene Burkov <E.Burkov@AdGuard.COM>
Date:   Tue Mar 22 16:55:52 2022 +0300

    home: fix adding client
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

3 participants