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 ability to configure the bandwidth test server #474

Closed
mirceanton opened this issue May 28, 2024 · 3 comments · Fixed by #475
Closed

Add ability to configure the bandwidth test server #474

mirceanton opened this issue May 28, 2024 · 3 comments · Fixed by #475
Assignees
Labels
enhancement New feature or request released

Comments

@mirceanton
Copy link
Contributor

Is your feature request related to a problem? Please describe.

I would like the ability to configure the bandwidth test server on my Mikrotik RB5009.

Describe the solution you'd like

Similar to how we can configure the MAC server:

resource "routeros_tool_mac_server" "test" {
  allowed_interface_list = "LAN"
}

I would like to be able to configure the bandwidth test server:

resource "routeros_tool_bandwidth_test_server" "test" {
  enabled = true
  authenticate = false
  max_sessions = 100
  allocate_udp_ports_from = 2000
}

Additional context

Here are the commands to do this via the CLI:

[admin@MikroTik-RB5009] > /tool/bandwidth-server/set authenticate=no
[admin@MikroTik-RB5009] > /tool/bandwidth-server/set enabled=yes    
[admin@MikroTik-RB5009] > /tool/bandwidth-server/set authenticate=no
[admin@MikroTik-RB5009] > /tool/bandwidth-server/set max-sessions=100
[admin@MikroTik-RB5009] > /tool/bandwidth-server/set allocate-udp-ports-from=2000
@mirceanton mirceanton added the enhancement New feature or request label May 28, 2024
@mirceanton
Copy link
Contributor Author

As described in this doc page, it is recommended to disable it, which is exactly what I am trying to do:
image

@mirceanton
Copy link
Contributor Author

More details can be found here: https://help.mikrotik.com/docs/display/ROS/Bandwidth+Test

@vaerh vaerh self-assigned this May 29, 2024
vaerh added a commit that referenced this issue May 29, 2024
Closes #474 

* add bandwitdh server resource

* add tests for the bandwitdh server resource

* specify the bandwitdh server resource in provider

* add examples for the bandwidth server resource

* fix typo

* fix function name

* generate docs for bandwidth server resource

* refactor: Small fixes to the added resource

* refactor: Add a helper for KeyEnabled

* fix: Resolve merge conflict

---------

Co-authored-by: Vaerh <vaerh@tutanota.com>
@vaerh
Copy link
Collaborator

vaerh commented May 29, 2024

🎉 This issue has been resolved in version 1.54.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@vaerh vaerh added the released label May 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request released
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants