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

Update, Get Inventory, Fetch or Modify the Power States of the CrayXD and it's components #7822

Closed
wants to merge 85 commits into from

Conversation

Srujana-2000
Copy link

SUMMARY

New module request
Adding a new module_utils file for the purpose of Updating, Getting Info, fetching the Power_Status.
And the addition of three module files separately for Updating, fetching Inventory Info and Getting or modifying the power status.

ISSUE TYPE
  • New Module/Plugin Pull Request
COMPONENT NAME

module_utils\hpc_system_firmware_utils.py
module\hpc_update_system_firmware.py
module\hpc_get_system_fw_inv.py
module\hpc_get_power_state.py

ADDITIONAL INFORMATION

@ansibullbot ansibullbot added module module module_utils module_utils needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR new_contributor Help guide this first time contributor plugins plugin (any type) labels Jan 10, 2024
@Srujana-2000 Srujana-2000 changed the title Update, Get Inventory, Fetch or Modify the CrayXD and it's components Update, Get Inventory, Fetch or Modify the Power States of the CrayXD and it's components Jan 10, 2024
@felixfontein felixfontein added check-before-release PR will be looked at again shortly before release and merged if possible. backport-8 Automatically create a backport for the stable-8 branch labels Jan 10, 2024
@felixfontein
Copy link
Collaborator

Thanks for your contribution. Please note https://github.com/ansible-collections/community.general/blob/main/CONTRIBUTING.md#creating-new-modules-or-plugins.

Also please note that it might be more useful to create a new collection instead of adding a set of modules for a new system to community.general, especially if these modules are supported by a company and not by the community and do not depend on anything else in community.general.

@Srujana-2000
Copy link
Author

We are working on the same line as ILO(ilo_redfish_utils.py) that's present in community.general. We have dependencies on community.general so we don't want to move outside

@Srujana-2000
Copy link
Author

We look forward to your guidance on how to proceed with this contribution.

@ansibullbot

This comment was marked as outdated.

@ansibullbot
Copy link
Collaborator

@Srujana-2000 this PR contains the following merge commits:

Please rebase your branch to remove these commits.

click here for bot help

@ansibullbot ansibullbot added the merge_commit This PR contains at least one merge commit. Please resolve! label Jan 18, 2024
@ansibullbot ansibullbot removed the ci_verified Push fixes to PR branch to re-run CI label Jan 31, 2024
@ansibullbot
Copy link
Collaborator

@Srujana-2000 this PR contains the following merge commits:

Please rebase your branch to remove these commits.

click here for bot help

@ansibullbot
Copy link
Collaborator

@Srujana-2000 this PR contains the following merge commits:

Please rebase your branch to remove these commits.

click here for bot help

@felixfontein felixfontein removed the check-before-release PR will be looked at again shortly before release and merged if possible. label Feb 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-8 Automatically create a backport for the stable-8 branch merge_commit This PR contains at least one merge commit. Please resolve! module_utils module_utils module module needs_rebase https://docs.ansible.com/ansible/devel/dev_guide/developing_rebasing.html needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR new_contributor Help guide this first time contributor plugins plugin (any type) WIP Work in progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants