aristanetworks / ansible-cvp

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

Fix(cv_tag_v3): Long workspace name to avoid conflict #679

Closed sugetha24 closed 11 months ago

sugetha24 commented 11 months ago

Change Summary

The cv_tag_v3 module creates workspaces in CloudVision under which to make the tag changes. Sometimes these workspaces are created with the same name as previously created instances, resulting in error. The error was observed in CVP with 700+ workspaces. Previously the WS name used 3 random alpha numberic string(42840 combinations). The new WS name uses 6 random alpha numberic string + current datetime

Related Issue(s)

Fixes #672

Component(s) name

arista.cvp.cv_tag_v3

Proposed changes

WS name to use 6 random alpha numberic string + current datetime

How to test

molecule converge -s cv_tag_v3

Checklist

User Checklist

Repository Checklist

chetryan commented 11 months ago

Tested again after the pylint fix and the solution still works as expected.

chetryan commented 11 months ago

Updated the format of the datetime string and tested on cloudvision

image