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

hponcfg - revamped the module using ModuleHelper #3840

Merged
merged 5 commits into from
Dec 11, 2021
Merged

hponcfg - revamped the module using ModuleHelper #3840

merged 5 commits into from
Dec 11, 2021

Conversation

russoz
Copy link
Collaborator

@russoz russoz commented Dec 2, 2021

SUMMARY

Revamped the module using ModuleHelper. By default that adds the improvement on calling run_command with a list instead of a string.

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

plugins/modules/remote_management/hpilo/hponcfg.py

@ansibullbot
Copy link
Collaborator

cc @haad
click here for bot help

@ansibullbot ansibullbot added feature This issue/PR relates to a feature request module module plugins plugin (any type) remote_management labels Dec 2, 2021
@ansibullbot

This comment has been minimized.

@ansibullbot ansibullbot added ci_verified Push fixes to PR branch to re-run CI needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR labels Dec 2, 2021
@ansibullbot ansibullbot removed the ci_verified Push fixes to PR branch to re-run CI label Dec 2, 2021
@ansibullbot

This comment has been minimized.

@ansibullbot ansibullbot added the ci_verified Push fixes to PR branch to re-run CI label Dec 2, 2021
@ansibullbot ansibullbot removed the ci_verified Push fixes to PR branch to re-run CI label Dec 2, 2021
@ansibullbot

This comment has been minimized.

@ansibullbot ansibullbot added the ci_verified Push fixes to PR branch to re-run CI label Dec 2, 2021
@ansibullbot ansibullbot removed ci_verified Push fixes to PR branch to re-run CI needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR labels Dec 2, 2021
@felixfontein felixfontein added backport-4 check-before-release PR will be looked at again shortly before release and merged if possible. labels Dec 3, 2021
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.

Looks good to me.

@felixfontein
Copy link
Collaborator

Will merge in ~a week if nobody complains.

@felixfontein felixfontein removed the check-before-release PR will be looked at again shortly before release and merged if possible. label Dec 11, 2021
@felixfontein felixfontein merged commit 7cbe1bc into ansible-collections:main Dec 11, 2021
@patchback
Copy link

patchback bot commented Dec 11, 2021

Backport to stable-4: 💚 backport PR created

✅ Backport PR branch: patchback/backports/stable-4/7cbe1bcf63dfeb5dd72d5ee8cc63ea9908e90777/pr-3840

Backported as #3891

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

@felixfontein
Copy link
Collaborator

@russoz thanks for this one as well!

patchback bot pushed a commit that referenced this pull request Dec 11, 2021
* hponcfg - revamped the module using ModuleHelper

* added changelog fragment

* fixed imports

* Update plugins/modules/remote_management/hpilo/hponcfg.py

* fixed

(cherry picked from commit 7cbe1bc)
@russoz russoz deleted the hponcfg-improvements branch December 11, 2021 20:25
felixfontein pushed a commit that referenced this pull request Dec 11, 2021
* hponcfg - revamped the module using ModuleHelper

* added changelog fragment

* fixed imports

* Update plugins/modules/remote_management/hpilo/hponcfg.py

* fixed

(cherry picked from commit 7cbe1bc)

Co-authored-by: Alexei Znamensky <103110+russoz@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature This issue/PR relates to a feature request module module plugins plugin (any type) remote_management
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants