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

multiple: missing mac_overrides of mikrotik sxtsq devices #900

Closed
34 tasks done
Noki opened this issue Jul 18, 2024 · 5 comments · Fixed by #1007
Closed
34 tasks done

multiple: missing mac_overrides of mikrotik sxtsq devices #900

Noki opened this issue Jul 18, 2024 · 5 comments · Fixed by #1007
Assignees

Comments

@Noki
Copy link
Member

Noki commented Jul 18, 2024

We have multiple bbb-configs locations that miss macaddress overrides for mikrotik_sxtsq devices. This should be fixed.

The mac_address of the device can be retrieved using cat /sys/firmware/mikrotik/hard_config/mac_base

Please mark the location with your name if you will fix it.

@FFHener
Copy link
Contributor

FFHener commented Jul 19, 2024

Where are mac overrides missing at fesev?

@FFHener FFHener transferred this issue from freifunk-berlin/meta Jul 19, 2024
@Noki
Copy link
Member Author

Noki commented Jul 19, 2024

Where are mac overrides missing at fesev?

Looks like I did a mistake there...

@FFHener
Copy link
Contributor

FFHener commented Jul 19, 2024

Looks like I did a mistake there...

Ok, just wanted to make shure i didnt miss anything while fixing the stuff. Thank you for researching this list

@FFHener FFHener removed their assignment Jul 22, 2024
@noxilixon noxilixon removed their assignment Jul 24, 2024
@FFHener FFHener self-assigned this Jul 30, 2024
@ZuquetuPlexis
Copy link
Contributor

ZuquetuPlexis commented Sep 21, 2024

PR: #973

  • temp fix

  • hacrafu-fiocchi-core

  • hacrafu-fiocchi-core

  • hacrafu-fiocchi-core

  • override temp with real mac address

@Noki
Copy link
Member Author

Noki commented Sep 22, 2024

I adjusted the check to only run on changed files.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants