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

ome_identity_pool requires iscsi iqn_prefix while GUI does not #329

Closed
MallocArray opened this issue Oct 21, 2021 · 0 comments · Fixed by #366
Closed

ome_identity_pool requires iscsi iqn_prefix while GUI does not #329

MallocArray opened this issue Oct 21, 2021 · 0 comments · Fixed by #366
Labels
type/feature-request New feature or request
Milestone

Comments

@MallocArray
Copy link

Summary

Using the ome_identity_pool to create iSCSI settings, the iqn_prefix of initiator_config is required to be completed or the task fails. Same with the initiator_ip_pool_settings.

In the OME-M GUI (1.30.00) , the only required fields are "Starting virtual MAC Address" and "Number of iSCSI MAC addresses", while the other two options can remain unchecked (Configure iSCSI Initiator and Enable iSCSI Intitator IP Pool)

The functionality should match between GUI and Ansible, allowing for only the MAC Addreses to be filled in, but not require the Initiator or IP pool

Note: If the existing Identity Pool in OME has iSCSI configured without these options, this Ansible module will fail to update Ethernet settings, even if iSCSI is not defined in the playbook at all.

Component Name

ome_identity_pool

Ansible Version
ansible 2.10.11
iDRAC or OpenManage Enterprise version

OME-M version 1.30.00

Steps to Reproduce

With same settings currently configure in an Identity Pool, the playbook should execute without error, including only iSCSI Mac address and number of addresses.

Playbook used
# Vars file
identity_pools:
  - pool_name: POC Identity Pool Ansible Test
    pool_description:
    ethernet_settings:
      starting_mac_address: "12:20:11:05:01:00"
      identity_count: 640
    fcoe_settings:
      starting_mac_address: "1A:20:11:05:03:00"
      identity_count: 640
    iscsi_settings:
      starting_mac_address: "16:20:11:05:02:00"
      identity_count: 640
      # initiator_config:
      #   iqn_prefix: "iqn.myprefix."
      # initiator_ip_pool_settings:
      #   ip_range: "10.33.0.1-10.33.0.255"
      #   subnet_mask: "255.255.255.0"
      #   gateway: "192.168.4.1"
      #   primary_dns_server : "10.8.8.8"
      #   secondary_dns_server : "8.8.8.8"
    fc_settings:
      starting_address: "1E:20:11:05:04:00"
      identity_count: 640

# Task in existing playbook
    - name: Identity Pool
      dellemc.openmanage.ome_identity_pool:
        hostname: "{{ inventory_hostname }}"
        username: "{{ dell_ome_username }}"
        password: "{{ dell_ome_password }}"
        state: present
        pool_name: "{{ item.pool_name }}"
        pool_description: "{{ item.pool_description }}"
        ethernet_settings: "{{ item.ethernet_settings }}"
        fcoe_settings: "{{ item.fcoe_settings }}"
        iscsi_settings: "{{ item.iscsi_settings }}"
        fc_settings: "{{ item.fc_settings }}"
      loop: "{{ identity_pools }}"
      loop_control:
        label: "{{ item.pool_name }}"
      when:
        - identity_pools is defined
      tags:
        - identity
Expected Results

Expected to run successfully as there is already an Identity Pool in existence with these exact settings

Actual Results
# With only starting MAC address and number of addresses, indicating the IQN prefix is required, although it is not required in the GUI
TASK [Identity Pool] ********************************************************************************************************************************************************************************
failed: [dellpod23.smrcy.com] (item=POC Identity Pool Ansible Test) => {"ansible_facts": {"discovered_interpreter_python": "/usr/libexec/platform-python"}, "ansible_loop_var": "item", "changed": false, "item": {"ethernet_settings": {"identity_count": 640, "starting_mac_address": "12:20:11:05:01:00"}, "fc_settings": {"identity_count": 640, "starting_address": "1E:20:11:05:04:00"}, "fcoe_settings": {"identity_count": 640, "starting_mac_address": "1A:20:11:05:03:00"}, "iscsi_settings": {"identity_count": 640, "starting_mac_address": "16:20:11:05:02:00"}, "pool_description": null, "pool_name": "POC Identity Pool Ansible Test"}, "msg": "IQN prefix is required to create iSCSI settings."}

# With Starting MAC Address, number of addresses, and IQN prefix completed, but no IP Pool which is not required in the GUI
TASK [Identity Pool] ********************************************************************************************************************************************************************************
failed: [dellpod23.smrcy.com] (item=POC Identity Pool Ansible Test) => {"ansible_facts": {"discovered_interpreter_python": "/usr/libexec/platform-python"}, "ansible_loop_var": "item", "changed": false, "item": {"ethernet_settings": {"identity_count": 640, "starting_mac_address": "12:20:11:05:01:00"}, "fc_settings": {"identity_count": 640, "starting_address": "1E:20:11:05:04:00"}, "fcoe_settings": {"identity_count": 640, "starting_mac_address": "1A:20:11:05:03:00"}, "iscsi_settings": {"identity_count": 640, "initiator_config": {"iqn_prefix": "iqn.myprefix."}, "starting_mac_address": "16:20:11:05:02:00"}, "pool_description": null, "pool_name": "POC Identity Pool Ansible Test"}, "msg": "Both ip range and subnet mask in required to create an identity pool using iSCSI settings."}

Community Note

  • Please vote on this issue by adding a 👍 reaction
    to the original issue to help the community and maintainers prioritize this request
  • Please do not leave "+1" or other comments that do not add relevant new information or questions,
    they generate extra noise for issue followers and do not help prioritize the request
  • If you are interested in working on this issue or have submitted a pull request, please leave a comment
@anupamaloke anupamaloke added the type/feature-request New feature or request label Oct 26, 2021
@anupamaloke anupamaloke added this to the v5.1.0 milestone Feb 3, 2022
@anupamaloke anupamaloke linked a pull request Mar 1, 2022 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/feature-request New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants