get_MS_names_by_lpar_name not enough values to unpack (expected 2, got 1) about ansible-power-hmc HOT 3 OPEN

I added

raise Error(ms) before line 981 to see what ms is

GXX-XXXX-XXX-SNXXXX,Operating

so the split should work. Not understanding why this does not work

from ansible-power-hmc.

Ok looks like the profiles i tried to list wat lpm-ed but never booted on a managed systems. this only retruns the state and not the name of the managed system.
I fixed it by excluding the "unknowns" for the inventory

I also changed get_MS_names_by_lpar_name

def get_MS_names_by_lpar_name(hmc_obj, lpar_name):
    mss = hmc_obj.list_all_managed_system_details("name,state")
    ms_list = []
    for ms in mss:
        if len(ms.split(',')) < 2:
            if ms == "Off":
                continue
            else:
                raise Exception("Only one Argument+" + ms)
        ms_name, state = ms.split(',')
        if state == 'Operating':
            lpar_names = hmc_obj.list_all_lpars_details(ms_name, "name")
            if lpar_name in lpar_names:
                ms_list.append(ms_name)
    return ms_list

If it only gets "Off" that it continues to the next loop.

from ansible-power-hmc.

This is an issue, we will fix it.

from ansible-power-hmc.