Redfish: Updated message handling for update operations to skip message objects with missing MessageId properties

[mraineri]

SUMMARY

Updated message handling for update operations to skip message objects with missing MessageId properties. While this is invalid, it's easy to handle and just move on to other messages to surface to the user.

Fix #7465

ISSUE TYPE

• Bugfix Pull Request

COMPONENT NAME

redfish_utils, redfish_command

ADDITIONAL INFORMATION

Output after change.

ok: [localhost] => {
    "redfish_results": {
        "ansible_facts": {
            "discovered_interpreter_python": "/usr/bin/python3"
        },
        "changed": true,
        "failed": false,
        "msg": "Action was successful",
        "return_values": {
            "update_status": {
                "handle": <REDACTED>,
                "messages": [],
                "resets_requested": [],
                "ret": true,
                "status": "New"
            }
        },
        "session": {}
    }
}

[ansibullbot]

cc @Bhavya06 @jyundt @rajeevkallur @renxulei @tomasg2012 @xmadsen
click here for bot help

[felixfontein]

recheck

[felixfontein]

Looks good to me. I'll merge this beginning of the upcoming week if nobody objects.

[patchback]

Backport to stable-7: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-7/ab0b85d7d2196e36c00cf4316c55582bad3b53f3/pr-7471

Backported as #7491

🤖 @patchback
I'm built with octomachinery and
my source is open — https://github.com/sanitizers/patchback-github-app.

[patchback]

Backport to stable-8: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-8/ab0b85d7d2196e36c00cf4316c55582bad3b53f3/pr-7471

Backported as #7492

🤖 @patchback
I'm built with octomachinery and
my source is open — https://github.com/sanitizers/patchback-github-app.

[felixfontein]

@mraineri thanks a lot for this fix!