[release-4.21] Handle missing RelatedProperties in TransferProtocolType detection#152
Conversation
Some BMCs (e.g. Cisco C845A M8 with Redfish Base.1.18.1) return ActionParameterMissing without RelatedProperties, only providing the missing parameter name in MessageArgs. The is_transfer_protocol_required method now checks MessageArgs as a fallback, enabling the automatic retry with TransferProtocolType. Change-Id: I1dc23853099533817b74a8a47767ff52dc6e99e2 Signed-off-by: Jacob Anders <jacob-anders-dev@proton.me> Assisted-By: Claude Code Sonnet 4.6 (cherry picked from commit c161b3a)
|
/retest |
|
@jacob-anders: This pull request references Jira Issue OCPBUGS-82294, which is invalid:
Comment The bug has been updated to refer to the pull request using the external bug tracker. DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository. |
|
@jacob-anders: all tests passed! Full PR test history. Your PR dashboard. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |
|
/lgtm |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: iurygregory, jacob-anders The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
@jacob-anders: No Jira issue is referenced in the title of this pull request. DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository. |
|
/jira refresh |
|
@jacob-anders: No Jira issue is referenced in the title of this pull request. DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository. |
6f4c2dc
into
openshift:release-4.21
Some BMCs (e.g. Cisco C845A M8 with Redfish Base.1.18.1) return ActionParameterMissing without RelatedProperties, only providing the missing parameter name in MessageArgs. The is_transfer_protocol_required method now checks MessageArgs as a fallback, enabling the automatic retry with TransferProtocolType.
Change-Id: I1dc23853099533817b74a8a47767ff52dc6e99e2
Assisted-By: Claude Code Sonnet 4.6
(cherry picked from commit c161b3a)