dell / ansible-powerflex

Ansible Modules for Dell PowerFlex
Other
22 stars 15 forks source link

[BUG]: Add NONE check for secondary and tie-breaker MDM #27

Closed Bhavneet-Sharma closed 1 year ago

Bhavneet-Sharma commented 1 year ago

Describe the bug When the MDM cluster has one node, Add a missing None check for slave and tie-breaker key in the mdm_cluster module.

To Reproduce Steps to reproduce the behavior:

  1. for one node MDM cluster, while adding a secondary/slave node or,
  2. while adding a tie-breaker node to one node MDM cluster ... n. Getting this error:

check_ip_in_secondarys KeyError: slaves ", "module_stdout": "", "msg": "MODULE FAILURE See stdout/stderr for the exact error", "rc": 1}

powerflex/plugins/modules/mdm_cluster.py\", line 1091, in check_ip_in_standby KeyError: tieBreakers ", "module_stdout": "", "msg": "MODULE FAILURE See stdout/stderr for the exact error", "rc": 1}

Expected behavior The mdm_cluster module should able to add a secondary or tie-breaker node to one node MDM cluster without any error.

anupamaloke commented 1 year ago

Closed by #28