search

ansible-collections / community.routeros

Ansible modules for managing MikroTik RouterOS instances.
https://galaxy.ansible.com/ui/repo/published/community/routeros/
GNU General Public License v3.0
95 stars 45 forks source link

Make wireless parameter "running" read-only #233

Closed hansmi closed 10 months ago

hansmi commented 10 months ago
SUMMARY

Commit e4a21311 added API data for the "interface wireless" path. It contained the read-only "running" parameter. Writing to the parameter fails:

Error while modifying for name="…" (ID *3): unknown parameter running
ISSUE TYPE
codecov[bot] commented 10 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (1beddb5) 83.15% compared to head (e11a4b0) 83.15%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #233 +/- ## ======================================= Coverage 83.15% 83.15% ======================================= Files 32 32 Lines 4031 4031 Branches 867 867 ======================================= Hits 3352 3352 Misses 494 494 Partials 185 185 ``` | [Flag](https://app.codecov.io/gh/ansible-collections/community.routeros/pull/233/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=ansible-collections) | Coverage Δ | | |---|---|---| | [integration](https://app.codecov.io/gh/ansible-collections/community.routeros/pull/233/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=ansible-collections) | `66.86% <ø> (ø)` | | | [sanity](https://app.codecov.io/gh/ansible-collections/community.routeros/pull/233/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=ansible-collections) | `22.12% <ø> (ø)` | | | [units](https://app.codecov.io/gh/ansible-collections/community.routeros/pull/233/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=ansible-collections) | `83.10% <ø> (ø)` | | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=ansible-collections#carryforward-flags-in-the-pull-request-comment) to find out more. | [Files](https://app.codecov.io/gh/ansible-collections/community.routeros/pull/233?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=ansible-collections) | Coverage Δ | | |---|---|---| | [plugins/module\_utils/\_api\_data.py](https://app.codecov.io/gh/ansible-collections/community.routeros/pull/233?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=ansible-collections#diff-cGx1Z2lucy9tb2R1bGVfdXRpbHMvX2FwaV9kYXRhLnB5) | `69.38% <ø> (ø)` | |

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

hansmi commented 10 months ago

Thanks for fixing this! Can you please add a changelog fragment?

Done. I had it ready, just forgot to add it to the commit. :-(

felixfontein commented 10 months ago

Thanks @hansmi!