-
Notifications
You must be signed in to change notification settings - Fork 201
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
Refactor(cv_deploy): Optimize creation of configlet containers #3950
Refactor(cv_deploy): Optimize creation of configlet containers #3950
Conversation
e13e7b8
to
4cb28a6
Compare
LOGGER.info("deploy_configs_to_cv: %s existing device containers under AVD root container.", len(existing_device_containers)) | ||
# Create dict keyed by container id with value of tuple containing key container parameters. Used later to detect changes. | ||
existing_device_containers_by_id = { | ||
cv_container.key.configlet_assignment_id: (cv_container.display_name, cv_container.description, cv_container.query, cv_container.configlet_ids) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add a suggestion with the fix, or just commit the fix directly here.
ansible_collections/arista/avd/plugins/plugin_utils/cv_client/workflows/deploy_configs_to_cv.py
Outdated
Show resolved
Hide resolved
Quality Gate passedIssues Measures |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested manually against 2024.1.2 - LGTM!
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
…workflows/deploy_configs_to_cv.py Co-authored-by: Alexey Gorbunov <[email protected]>
for more information, see https://pre-commit.ci
4ffeb78
to
0ed1ec6
Compare
Conflicts have been resolved. A maintainer will review the pull request shortly. |
b45841e
to
0ed1ec6
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM - tested with cv_deploy and cv_workflow molecule scenarios
Change Summary
Optimize creation of configlet containers, but only creating/updating missing containers.
Component(s) name
arista.avd.cv_deploy
Proposed changes
How to test
Checklist
User Checklist
Repository Checklist