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

Add support for Oem params #7330

Merged
merged 5 commits into from
Oct 2, 2023

Conversation

sseekamp
Copy link
Contributor

@sseekamp sseekamp commented Sep 29, 2023

Possible resolution to #7331

Add support for Oem parameters in the MultipartHTTPPushUpdate command of redfish_command

@ansibullbot
Copy link
Collaborator

@ansibullbot ansibullbot added module module module_utils module_utils plugins plugin (any type) labels Sep 29, 2023
@mraineri
Copy link
Contributor

Code looks good to me; thanks!

@felixfontein felixfontein added check-before-release PR will be looked at again shortly before release and merged if possible. backport-7 labels Sep 29, 2023
Copy link
Collaborator

@felixfontein felixfontein left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for your contribution! Can you please add a changelog fragment? Thanks.

plugins/modules/redfish_command.py Outdated Show resolved Hide resolved
Add version_added field
@felixfontein
Copy link
Collaborator

@mraineri if you're still happy I'll merge this at the beginning of the upcoming week.

@mraineri
Copy link
Contributor

Yup, the changes look good to me

@felixfontein felixfontein merged commit cd83b24 into ansible-collections:main Oct 2, 2023
144 checks passed
@felixfontein
Copy link
Collaborator

@sseekamp thanks for your contribution!
@mraineri thanks for reviewing!

@felixfontein felixfontein added backport-7 and removed check-before-release PR will be looked at again shortly before release and merged if possible. labels Oct 2, 2023
@patchback
Copy link

patchback bot commented Oct 2, 2023

Backport to stable-7: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-7/cd83b245bba372d3c9e471914235825b91e447d0/pr-7330

Backported as #7341

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

patchback bot pushed a commit that referenced this pull request Oct 2, 2023
* Add support for "Oem" parameters in the MultipartHTTPPushUpdate redfish_command

* Add doc entry

* Add changelog fragment
Add version_added field

* Re-word the changelog fragment

* After further testing this should not have a default definition as
it should not be in the payload if not defined

(cherry picked from commit cd83b24)
felixfontein pushed a commit that referenced this pull request Oct 2, 2023
)

Add support for Oem params (#7330)

* Add support for "Oem" parameters in the MultipartHTTPPushUpdate redfish_command

* Add doc entry

* Add changelog fragment
Add version_added field

* Re-word the changelog fragment

* After further testing this should not have a default definition as
it should not be in the payload if not defined

(cherry picked from commit cd83b24)

Co-authored-by: Scott Seekamp <[email protected]>
@sseekamp sseekamp deleted the sseekamp/oem-hmpp branch October 4, 2023 23:36
etrombly pushed a commit to etrombly/community.general that referenced this pull request Oct 25, 2023
* Add support for "Oem" parameters in the MultipartHTTPPushUpdate redfish_command

* Add doc entry

* Add changelog fragment
Add version_added field

* Re-word the changelog fragment

* After further testing this should not have a default definition as
it should not be in the payload if not defined
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
has_issue module_utils module_utils module module plugins plugin (any type)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants