redfish_command - failure on user account deletion

[spike77453]

SUMMARY

Running the example playbook from https://docs.ansible.com/ansible/latest/modules/redfish_command_module.html#examples to disable and delete a certain user works as expected, but results in an error message

The specified value is not allowed to be configured if the user name \nor password is blank.'

Note that the user to be deleted (in the example below the user with id 5) is different from the user specified in "username".
The user (in the example below the user with id 5) is deleted as expected despite the error message.

ISSUE TYPE

• Bug Report

COMPONENT NAME

redfish_command

ANSIBLE VERSION

ansible 2.10.0.dev0
  config file = /home/ansible/ansible_local/ansible.cfg
  configured module search path = [u'/home/ansible/.ansible/plugins/modules', u'/usr/share/ansible/plugins/modules']
  ansible python module location = /usr/lib/python2.7/site-packages/ansible
  executable location = /usr/bin/ansible
  python version = 2.7.5 (default, Aug  7 2019, 00:51:29) [GCC 4.8.5 20150623 (Red Hat 4.8.5-39)]

CONFIGURATION

OS / ENVIRONMENT

CentOS Linux release 7.6.1810 (Core)
Dell R640, iDRAC Firmware 3.32.32.32

STEPS TO REPRODUCE

- name: Disable and delete user
  hosts: dellR640
  gather_facts: False
  tasks:
    - name: Disable and delete user with id 5
      local_action:
        module: redfish_command
        category: Accounts
        command: ["DisableUser", "DeleteUser"]
        baseuri: "{{ baseuri }}"
        username: "{{ username }}"
        password: "{{ password }}"
        id: "5"

EXPECTED RESULTS

ACTUAL RESULTS

fatal: [dellR640 -> localhost]: FAILED! => {
    "changed": false, 
    "invocation": {
        "module_args": {
            "account_properties": {}, 
            "baseuri": "dellR640idrac", 
            "boot_next": null, 
            "bootdevice": null, 
            "category": "Accounts", 
            "command": [
                "DisableUser", 
                "DeleteUser"
            ], 
            "id": "5", 
            "new_password": null, 
            "new_username": null, 
            "password": "VALUE_SPECIFIED_IN_NO_LOG_PARAMETER", 
            "resource_id": null, 
            "roleid": null, 
            "timeout": 10, 
            "uefi_target": null, 
            "update_username": null, 
            "username": "customadminuser"
        }
    }, 
    "msg": "HTTP Error 400 on PATCH request to 'https://dellR640idrac/redfish/v1/Managers/iDRAC.Embedded.1/Accounts/5', extended message: 'The specified value is not allowed to be configured if the user name \\nor password is blank.'"
}

[ansibot]

Files identified in the description:

• lib/ansible/modules/remote_management/redfish/redfish_command.py

If these files are inaccurate, please update the component name section of the description or use the !component bot command.

click here for bot help

[ansibot]

cc @billdodd @jose-delarosa @mraineri @tomasg2012
click here for bot help

[billdodd]

@spike77453 I haven't been able to reproduce this error on the Dell R630 I have access to. And I do not have access to an R640.

I do have a suspicion of what may be causing it and a potential fix idea. If I create a PR with a potential fix, would you be willing and able to test it?

[spike77453]

@billdodd Sure thing. I'm happy to test any potential fix. Just let me know.

[billdodd]

Sure thing. I'm happy to test any potential fix. Just let me know.

Thanks! PR is ready to test: #64797

[spike77453]

#64797 works for me. Thanks a bunch!

[billdodd]

@spike77453 Great! Thanks for testing.

Would you mind re-opening the issue? That way the core team will see that my PR is to fix an open issue and will be sure to merge it. Otherwise, there might be some confusion if the issue is already closed. :-) Once the PR is merged, your issue will be automatically closed.

[spike77453]

Would you mind re-opening the issue? That way the core team will see that my PR is to fix an open issue and will be sure to merge it.

Sorry, wasn't aware that it's still needed.

[billdodd]

No worries. Thanks again.