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

Add support for newer 'wifi' interface #253

Closed
radokristof opened this issue Jan 24, 2024 · 3 comments · Fixed by #266
Closed

Add support for newer 'wifi' interface #253

radokristof opened this issue Jan 24, 2024 · 3 comments · Fixed by #266
Labels
enhancement New feature or request

Comments

@radokristof
Copy link

If possible please also add support for the wifi interface type. Config is similar to wifiwave2, after 7.12 it replaces the wifiwave2 package actually.

Thank you

@felixfontein felixfontein added the enhancement New feature or request label Jan 24, 2024
@teadur
Copy link

teadur commented Jan 27, 2024

support for wifi package would be needed indeed, as of since routeros 7.13 you must be running wifi pkg if you have wifi ac/ax chips and currently we are unable to configure such a device with this role.

It seems that the maintainer is well aware of the fact:
https://github.com/search?q=repo%3Aansible-collections%2Fcommunity.routeros%207.13&type=code

I'm a bit surprised that there is nothing on the subject in the repo issues, if fixing it depends on something outside like routeros api implementation it would be nice to know, otherwise should be fairly easy to implement considering its more or less wifiwave2 renamed.

Hopefully this issue gets priority in #1 and @ytimenkov can help with the implementation/review as of he was the original implementor.

@ytimenkov
Copy link
Contributor

At that time ros 7.13 was beta and I didn't want to risk my ap 😁. I can try to upgrade and take a look when I get some free time.

@radokristof
Copy link
Author

AFAIK, commands are the same as for wifiwave2 maybe only the path names needs to be changed

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

Successfully merging a pull request may close this issue.

4 participants