Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Redfish Support multi-part HTTP Push Update #6471

Closed
1 task done
sseekamp opened this issue May 1, 2023 · 7 comments · Fixed by #6612
Closed
1 task done

Redfish Support multi-part HTTP Push Update #6471

sseekamp opened this issue May 1, 2023 · 7 comments · Fixed by #6612
Labels
feature This issue/PR relates to a feature request module module plugins plugin (any type)

Comments

@sseekamp
Copy link
Contributor

sseekamp commented May 1, 2023

Summary

At the moment we have GPU devices that only support HTTP push Redfish update capability. Would it be possible to open a discussion around implementing that functionality?

Per #3910 I see that it isn't supported and doesn't appear to be on a roadmap that I could find.

@mraineri - thank you in advance :)

Issue Type

Feature Idea

Component Name

redfish_command

Additional Information

Potentially implement it as:

- name: Update with http push
  community.general.redfish_command:
    category: Update
    command: PushUpdate
    baseuri: "{{ baseuri }}"
    username: "{{ username }}"
    password: "{{ password }}"
    update_image_file: file_to_push_to_target
    update_targets:
      - /redfish/v1/UpdateService/upload

Code of Conduct

  • I agree to follow the Ansible Code of Conduct
@ansibullbot
Copy link
Collaborator

Files identified in the description:

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

click here for bot help

@ansibullbot
Copy link
Collaborator

@ansibullbot ansibullbot added feature This issue/PR relates to a feature request module module plugins plugin (any type) labels May 1, 2023
@sseekamp sseekamp changed the title Redfish SimpleUpdate Support multi-part HTTP Push Redfish Support multi-part HTTP Push Update May 1, 2023
@mraineri
Copy link
Contributor

mraineri commented May 1, 2023

@sseekamp, before starting down this path, could you confirm that "/redfish/v1/UpdateService" contains the property "MultipartHttpPushUri" for your system?

@sseekamp
Copy link
Contributor Author

sseekamp commented May 1, 2023

  "Id": "UpdateService",
  "MaxImageSizeBytes": 205824000,
  "MultipartHttpPushUri": "/redfish/v1/UpdateService/upload",
  "Name": "Update Service",

@mraineri
Copy link
Contributor

mraineri commented May 1, 2023

Perfect, thanks!

@sseekamp
Copy link
Contributor Author

Revisiting - we have another vendor that is multipart upload and not simple update capable if that helps with effort to value calculations :)

@mraineri
Copy link
Contributor

Thanks for the ping; I lost this from my radar.

I don't think it should be hard to support; most of the building blocks are in place, and the multipart encoding can be leveraged from the requests toolkit module.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature This issue/PR relates to a feature request module module plugins plugin (any type)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants