aristanetworks / ansible-cvp

Ansible modules for Arista CloudVision
http://cvp.avd.sh
Apache License 2.0
66 stars 61 forks source link

Fix(cv_device_v3): Implement device check mode #571

Closed colinmacgiolla closed 1 year ago

colinmacgiolla commented 1 year ago

Change Summary

So it looks like, for the most part the cv_device_v3 completely ignored if check mode was set or not, which is not great. The recent support for image bundles made this more painful, and triggered issue #551

Related Issue(s)

Fixes #551

Component(s) name

arista.cvp.cv_device_v3

Proposed changes

No changes to schema - we just block any API calls that would actually make changes on CVP with fake success. Anything going wrong before that point will fail as expected, so in theory this should be it.

How to test

Checklist

User Checklist

Repository Checklist