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

aci_epg_to_domain: parameter VLAN mode for VMM Domain (DCNE-225) #702

Closed
alexeim1973 opened this issue Nov 1, 2024 · 3 comments · Fixed by #703
Closed

aci_epg_to_domain: parameter VLAN mode for VMM Domain (DCNE-225) #702

alexeim1973 opened this issue Nov 1, 2024 · 3 comments · Fixed by #703
Labels
documentation Improvements or additions to documentation jira-sync Sync this issue to Jira

Comments

@alexeim1973
Copy link

ISSUE TYPE
enhancement or bug report

COMPONENT NAME
aci_epg_to_domain
aci_epg_to_domain_ansible_doc
aci_epg_to_domain_apic

ANSIBLE VERSION
v10

SUMMARY
When creating a VMM Domain Association for a EPG, it would be good to have the possibility to configure the VLAN mode as "static" and "dynamic" with default being "dynamic".

Currently this doesn't seem to be possible.

For the module, the name of this parameter could be something like "vlan_mode" and the value is a str.

The policy is configured in ACI via Tenant > Application Profile > EPG > Add VMM domain > VLAN mode [dynamic | static] - see the attached screenshot with APIC GUI.

There is a parameter "encap_mode" in this module with str values of "auto" "vlan" "vxlan" and is used in conjunction with parameter "encap" and in "encap" description it is stated that it can be used with static "encaps_mode". So probably it is a bug report and not an enhancement request. See the atached screenshot from Ansible documentation.

APIC version and APIC Platform

  • V 5.2.8e on-prem.

Collection versions

  • cisco.aci 2.10.1

Output/ Error message

failed: [10.46.31.51] (item={'epg': 'EPG-FACTORY-176', 'custom_epg_name': 'EPG-FACTORY-176-STATIC', 'encap_mode': 'static', 'encap': 176, 'bd': 'BD-FACTORY-176', 'pref_grp': True, 'dm': 'ACI-VMM-DVS7', 'dm_type': 'vmm', 'vm_provider': 'vmware'}) => {"ansible_loop_var": "item", "changed": false, "item": {"bd": "BD-FACTORY-176", "custom_epg_name": "EPG-FACTORY-176-STATIC", "dm": "ACI-VMM-DVS7", "dm_type": "vmm", "encap": 176, "encap_mode": "static", "epg": "EPG-FACTORY-176", "pref_grp": true, "vm_provider": "vmware"}, "msg": "value of encap_mode must be one of: auto, vlan, vxlan, got: static"}

@alexeim1973 alexeim1973 added the bug Something isn't working label Nov 1, 2024
@akinross akinross removed the bug Something isn't working label Nov 1, 2024
@akinross
Copy link
Collaborator

akinross commented Nov 1, 2024

Hi @alexeim1973,

The value encap_mode is not representing the vlan mode as you point towards in UI. The vlan mode in UI is only a toggle button to be able to set the port encap in UI. With the ansible module you only need to provide the encap if you want the port to be displayed as static in UI. I have removed the bug label for now, please let me know if this answers your question.

@akinross akinross added the jira-sync Sync this issue to Jira label Nov 1, 2024
@github-actions github-actions bot changed the title aci_epg_to_domain: parameter VLAN mode for VMM Domain aci_epg_to_domain: parameter VLAN mode for VMM Domain (DCNE-225) Nov 1, 2024
@alexeim1973
Copy link
Author

Hi @alexeim1973,

The value encap_mode is not representing the vlan mode as you point towards in UI. The vlan mode in UI is only a toggle button to be able to set the port encap in UI. With the ansible module you only need to provide the encap if you want the port to be displayed as static in UI. I have removed the bug label for now, please let me know if this answers your question.

Hello and thanks for looking into this.
My understanding is that VLAN mode static will be used automatically once the encaps parameter is used. Is it correct?

As for the encaps_mode parameter, it is referred to in the encaps documentation as having value "static" , see the screedshot. The encaps documentation should probably be updated.

It should also be updated with the notion that VLAN mode GUI element static will be triggered automatically in case of static VLAN encapsulation in use.

Hope it makes sense.

@akinross
Copy link
Collaborator

Hi @alexeim1973,

See above attached PR with the suggested changes, hope this clarifies it for you. Let me know if this is unclear then I am open to suggestions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation jira-sync Sync this issue to Jira
Projects
None yet
2 participants