Skip to content
This repository has been archived by the owner on Dec 1, 2023. It is now read-only.

Add module for Nutanix AHV Cluster Refresh #114

Open
wants to merge 8 commits into
base: devel
Choose a base branch
from

Conversation

eramnes
Copy link

@eramnes eramnes commented Feb 4, 2021

Description

Adds an Ansible module to initiate an AHV cluster metadata refresh

Related Issue

This project only accepts pull requests related to open issues.

  • If suggesting a new feature or change, please discuss it in an issue first.
  • If fixing a bug, there should be an issue describing it with steps to reproduce

#113

Motivation and Context

Allows AHV clusters to be refreshed from Ansible.

How Has This Been Tested?

Tested with Ansible 2.9.6, Rubrik 5.2.2 p2, and Nutanix AOS 5.18.1.2. Cluster metadata refresh appears to complete successfully.

Types of changes

What types of changes does your code introduce? Put an x in all the boxes that apply:

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

Go over all the following points, and put an x in all the boxes that apply. If you're unsure about any of these, don't hesitate to ask. We're here to help!

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTION document.
  • I have updated the CHANGELOG file accordingly for the version that this merge modifies.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

This module requires the most recent devel code for the Python SDK to function. Additionally, using the module with the code from the Galaxy release seems to require a change to the credential import support, shown on line 73. I've commented this out because the modules in the repository now don't use this format. I am not familiar with packaging for Galaxy so not sure where this change gets made in the process.

I'm afraid I'm kind of new at this, so if there is anything else you need from me please let me know. Thanks for your consideration!

EDIT: Forgot the unit test file, sorry.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant