-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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 redfish processor architecture to Redfish CPU inventory #6864
Add redfish processor architecture to Redfish CPU inventory #6864
Conversation
Other than the missing changelog fragment, this looks good to me! |
Thanks for your contribution! You can find infos on changelog fragments here: https://docs.ansible.com/ansible/devel/community/collection_development_process.html#creating-a-changelog-fragment |
Thank you, I hope that d303457 will be ok. |
changelogs/fragments/6864-redfish-utils-fix-for-processorarchitecture-in-cpu-inventory.yaml
Outdated
Show resolved
Hide resolved
Co-authored-by: Felix Fontein <felix@fontein.de>
@felixfontein Thank you for your suggestion |
Backport to stable-7: 💔 cherry-picking failed — conflicts found❌ Failed to cleanly apply 4259792 on top of patchback/backports/stable-7/4259792751e72f5a622a4cfaf76d03381466a9e3/pr-6864 Backporting merged PR #6864 into stable-7
🤖 @patchback |
* Add Processor Architecture to CPU Inventory details * Add changelog fragment * Changelog fragment writing guide formatting Co-authored-by: Felix Fontein <felix@fontein.de> --------- Co-authored-by: Pierre-yves FONTANIERE <pyf@cc.in2p3.fr> Co-authored-by: Felix Fontein <felix@fontein.de>
Please open a PR against |
@felixfontein Sure, i will. |
@pyfontan no worries, it's also my fault - I totally overlooked it :D And thanks again for contributing! |
SUMMARY
This update adds ProcessorArchitecture to redfish CPU inventory
Fixes #6863
ISSUE TYPE
COMPONENT NAME
community.general.redfish_info
ADDITIONAL INFORMATION