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

allowed_vlans on meraki_ms_switchport should be string not list #345

Open
craigers521 opened this issue Aug 15, 2022 · 0 comments
Open

allowed_vlans on meraki_ms_switchport should be string not list #345

craigers521 opened this issue Aug 15, 2022 · 0 comments

Comments

@craigers521
Copy link

craigers521 commented Aug 15, 2022

##REQUEST##

The default data type for the API parameter is a string, i believe this should also be true for the ansible module. It looks like you are taking a list of ints and converting them to a string anyway. I believe this is breaking idempotency, and in my case its also forcing me to use more complex logic when converting my host_vars from using the switchport module into creating a body for the action_batches payload. Also causing some weirdness when using a range of vlans.

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

No branches or pull requests

1 participant