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
Facts dev upstream #57082
Facts dev upstream #57082
Conversation
@ndswartz I am not seeing any testing for this at all, could we get some coverage on this? |
Added the following facts: - storage system segment size - cache block size capabilities - workload tags - storage array hosts - host groups - list of mapped volumes for each initiator
9cb46d6
to
ccbba16
Compare
The test
|
ccbba16
to
7cd12c4
Compare
The test
|
7cd12c4
to
c4b0871
Compare
The test
|
The test
|
c4b0871
to
e8db837
Compare
The test
|
The test
|
e8db837
to
7c7e9fa
Compare
@thedoubl3j Added unit test coverage |
thank you @ndswartz reviewing now |
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, thanks for adding some coverage.
SUMMARY
Added the following facts:
ISSUE TYPE
COMPONENT NAME
netapp_e_facts
ADDITIONAL INFORMATION