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

Change CVP default configlet_prefix in AVD 5.0 #4693

Open
1 task done
AristaJon opened this issue Nov 6, 2024 · 0 comments
Open
1 task done

Change CVP default configlet_prefix in AVD 5.0 #4693

AristaJon opened this issue Nov 6, 2024 · 0 comments
Labels
type: enhancement New feature or request

Comments

@AristaJon
Copy link

Enhancement summary

Using the default variables, the arista.avd.eos_config_deploy_cvp role will create the following configlet name:

AVD-{{ fabric_name }}-_{{ hostname }}

  • notice the unnecessary dash followed by underscore.

Which component of AVD is impacted

eos_config_deploy_cvp

Use case example

We see the default config_prefix in ansible_collections/arista/avd/roles/eos_config_deploy_cvp/defaults/main/main.yml is:

configlets_prefix: 'AVD-{{ fabric_name }}-'

But in ansible_collections/arista/avd/roles/eos_config_deploy_cvp/templates/cvp-devices-v3.j2 we see this structure

configlets:
...
      - {{ configlets_prefix }}_{{ device }}
{%             endif %}

I suspect this is what is causing the concatenation.

Describe the solution you would like

Change the default value for configlets_prefix to be 'AVD_{{ fabric_name }}'

Describe alternatives you have considered

No response

Additional context

No response

Contributing Guide

  • I agree to follow this project's Code of Conduct
@AristaJon AristaJon added the type: enhancement New feature or request label Nov 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant