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

Update mgmt api j2 to support acl in default vrf #529

Closed
onurgashi opened this issue Dec 21, 2020 · 0 comments · Fixed by #533
Closed

Update mgmt api j2 to support acl in default vrf #529

onurgashi opened this issue Dec 21, 2020 · 0 comments · Fixed by #533
Assignees
Labels
good first issue Good for newcomers role: eos_cli_config_gen issue related to eos_cli_config_gen role

Comments

@onurgashi
Copy link
Contributor

onurgashi commented Dec 21, 2020

Issue Type

  • Bug Report
  • Documentation Issue Report
  • EOS Configuration error

Summary

We don't factor default vrf because of the below when adding anything including ACLs.

{%     if management_api_http.enable_vrfs is defined and management_api_http.enable_vrfs is not none -%}
{%         for vrf in management_api_http.enable_vrfs | arista.avd.natural_sort if vrf is ne 'default' %}
   !
   vrf {{ vrf }}
      no shutdown

Role or Module Name

OS / Environment

EOS Version

Cloudvision version (optional)

OS running Ansible

Steps to reproduce

Expected results

$

Actual results

@onurgashi onurgashi added good first issue Good for newcomers role: eos_cli_config_gen issue related to eos_cli_config_gen role labels Dec 21, 2020
@onurgashi onurgashi changed the title Update mgmt api to support acl in default vrf Update mgmt api j2 to support acl in default vrf Dec 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers role: eos_cli_config_gen issue related to eos_cli_config_gen role
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant