-
Notifications
You must be signed in to change notification settings - Fork 24k
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
added aix_nimclient module #50760
added aix_nimclient module #50760
Conversation
@AugustusKling @ColOfAbRiX @EvanK @LinusU @Mogztter @MorrisA @abulimov @adejoux @ahtik @azaghal @bgurney-rh @dankeder @davidobrien1985 @davixx @dirtyharrycallahan @dougluce @dsummersl @flynn1973 @gforster @giovannisciortino @goozbach @groks @haad @hryamzik @indrajitr @jasperla @jbenden @jdauphant @jhoekx @jsumners @jtyr @kairoaraujo @kevensen @lberruti @mattjeffery @matze @mcv21 @mpdehaan @mulby @natefoo @nibalizer @obourdon @ovcharenko @pilou- @pmarkham @pyykkis @rhaido @risaacson @ryan_sb @saito-hideki @sebastiendarocha @sfromm @srvg @tacatac @tdtrask @tmshn @xen0l As a maintainer of a module in the same namespace this new module has been submitted to, your vote counts for shipits. Please review this module and add |
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.
Trying to make multiline edits with the Github suggestion mechanism was rather painful :-(
These are mostly cosmetic changes at this time. Will do second review of the code after this has been accepted.
Co-Authored-By: molekuul <[email protected]>
Co-Authored-By: molekuul <[email protected]>
Co-Authored-By: molekuul <[email protected]>
Co-Authored-By: molekuul <[email protected]>
Co-Authored-By: molekuul <[email protected]>
Co-Authored-By: molekuul <[email protected]>
Co-Authored-By: molekuul <[email protected]>
Co-Authored-By: molekuul <[email protected]>
Co-Authored-By: molekuul <[email protected]>
Co-Authored-By: molekuul <[email protected]>
Co-Authored-By: molekuul <[email protected]>
Co-Authored-By: molekuul <[email protected]>
Co-Authored-By: molekuul <[email protected]>
Co-Authored-By: molekuul <[email protected]>
This comment has been minimized.
This comment has been minimized.
Well that was painfull..... this tool isn't just not made for these kind of small adjustments. kind regards, Joris Weijters |
bot_skip |
I did look at all proposed changes, and included then into the module however the status is still Changes requested. What else has to be done to get this module in Ansible? Kind regards, Joris Weijters |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
NIM stands for Network Install Manager. It is meant to install AIX by the network. From this LPP-source you install AIX. Multiple LPP_sources can exist. How to patch a AIX server which is a nimclient:
pushing the patches, means a central management of the clients, and so access to the NIM-Server. In traditional aix system managemt, there is a central team managing the IBM Power hardware, VIO-Server and NIM-Server and nimclients. There is a little overlap with the installp module: The "nimclient" command can not remove or commit software. However to have this functionality, in the nimclient module this is done by the installp command. The aixoss packages are written for defining and managing the NIM-Server, and pushing patches to VIO-Servers. A VIO-Server is a special type of server, Virtual I/O Server. This server is used for connecting virtual devices to physical devices. If compared to VMWare, it's like the ESX layer, but it's not the same. Kind regards, Joris Weijters |
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, although this module needs to be reviewed and tested by AIX users.
(We could also benefit from having integration tests)
ready_for_review |
folks, please review to get it merged before collections makes things even more complicated. |
@molekuul Your branch does not contain a shippable.yml file. Please rebase your branch to trigger running of current tests. |
Hi @molekuul, Thank you very much for your interest in Ansible. This plugin/module is no longer maintained in this repository and has been migrated to https://github.com/ansible-collections/community.general |
SUMMARY
This new module is ment for AIX. It installs software at a nimclient using nimclient command and removes software using local commands. This module also updates ALL to latest version, and ia able to issues nimclient action on the nimclient like reset or deallocate
ISSUE TYPE
COMPONENT NAME
aix_nimclient
ADDITIONAL INFORMATION
Examples: