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

ucs_vlans add support for vlan groups #16

Open
Akasurde opened this issue Aug 4, 2020 · 1 comment
Open

ucs_vlans add support for vlan groups #16

Akasurde opened this issue Aug 4, 2020 · 1 comment

Comments

@Akasurde
Copy link

Akasurde commented Aug 4, 2020

From @denngie on Jul 19, 2018 09:37

SUMMARY

Would like to see support for vlan group as a parameter in the ucs_vlans module. I think the python code in this example could be useful

ISSUE TYPE
  • Feature Idea
COMPONENT NAME

ucs_vlans

ANSIBLE VERSION
ansible 2.6.1
  config file = None
  configured module search path = [u'/home/e600790/.ansible/plugins/modules', u'/usr/share/ansible/plugins/modules']
  ansible python module location = /usr/local/lib/python2.7/dist-packages/ansible
  executable location = /usr/local/bin/ansible
  python version = 2.7.13 (default, Nov 24 2017, 17:33:09) [GCC 6.3.0 20170516]
CONFIGURATION

N/A

OS / ENVIRONMENT

Debian 9.5

STEPS TO REPRODUCE
- name: Configure VLAN
  ucs_vlans:
    hostname: 172.16.143.150
    username: admin
    password: password
    name: vlan2
    id: '2'
    native: 'yes'
    group: Prod
EXPECTED RESULTS

VLAN is created and added to the VLAN Group Prod

ACTUAL RESULTS

N/A

Copied from original issue: ansible/ansible#43006

@bakrir
Copy link

bakrir commented Apr 28, 2021

Just piling on here. We probably should also have the vnic_template accept vlan_groups as a paramter, for example, instead of specifying each VLAN on a vNIC template like so:

- name: Configure vNIC template
  cisco.ucs.ucs_vnic_template:
    hostname: 172.16.143.150
    username: admin
    password: password
    name: vNIC-A
    fabric: A
    vlans_list:
    - name: default
      native: 'yes'

Maybe, have a vlan_group parameter:

- name: Configure vNIC template
  cisco.ucs.ucs_vnic_template:
    hostname: 172.16.143.150
    username: admin
    password: password
    name: vNIC-A
    fabric: A
    vlans_list:
    - name: default
      native: 'yes'
    vlans_groups:
    - name: prod

Obviously vlan_groups would need to be created prior to this change kicing in (similar to VLANs).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants