-
Notifications
You must be signed in to change notification settings - Fork 23.7k
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
Populate product_name and system_vendor facts on Solaris #44114
Conversation
This comment has been minimized.
This comment has been minimized.
Tested with solaris 10 and solaris 11 under vmware.
after patch(es) applied:
|
@mavit can you rebase to latest ansible git version - https://docs.ansible.com/ansible/latest/dev_guide/developing_rebasing.html @alikins can you please apply this pull request?! Thanks! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, may need to be backported to v2.7 too.
@mavit Can you backport this to stable-2.7 ? https://docs.ansible.com/ansible/devel/community/development_process.html#backporting-merged-prs |
SUMMARY
The first line of
prtdiag
on Solaris mentions the manufacturer and model name of the hardware. Unfortunately, these two fields are not reliably delineated, so some guesswork is required.The existing code assumes that the model name consists of three words, with the middle word being "Enterprise". Whilst this is often true for recent Oracle hardware, it isn't true for a lot of other equipment that's still out there. By using a hard-coded list of known manufacturers, we can make a better attempt at separating the two parts.
ISSUE TYPE
COMPONENT NAME
Hardware facts.
ANSIBLE VERSION
ADDITIONAL INFORMATION
Before:
After: