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

migrate cluster dpm module from community.vmware #71

Conversation

mikemorency
Copy link
Collaborator

SUMMARY

This is migrating the community.vmware.vmware_cluster_dpm module here.

The inputs have changed in name, but not functionality. I updated the names based on VMwares documentation for DPM. The old names were based on the actual cluster configuration keys, but didnt make much sense when you read the documentation or looked in the UI

The old module had no outputs. The new module outputs details about the cluster reconfiguration task if changes were made.

ISSUE TYPE
  • New Module Pull Request
COMPONENT NAME

cluster_dpm

Copy link

codecov bot commented Sep 28, 2024

Codecov Report

Attention: Patch coverage is 40.00000% with 36 lines in your changes missing coverage. Please review.

Project coverage is 26.74%. Comparing base (013afa4) to head (dbeff87).
Report is 15 commits behind head on main.

Files with missing lines Patch % Lines
plugins/modules/cluster_dpm.py 40.00% 36 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #71      +/-   ##
==========================================
+ Coverage   26.09%   26.74%   +0.65%     
==========================================
  Files          19       21       +2     
  Lines        1667     1802     +135     
  Branches      331      346      +15     
==========================================
+ Hits          435      482      +47     
- Misses       1232     1320      +88     
Flag Coverage Δ
sanity 26.74% <40.00%> (+0.65%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@mikemorency mikemorency force-pushed the mm-feature/migrate-cluster-dpm branch from a96fc82 to dbeff87 Compare October 1, 2024 13:16
@mikemorency mikemorency merged commit 69443e8 into ansible-collections:main Oct 1, 2024
13 checks passed
@mikemorency mikemorency deleted the mm-feature/migrate-cluster-dpm branch October 1, 2024 13:28
softwarefactory-project-zuul bot pushed a commit to ansible-collections/community.vmware that referenced this pull request Oct 24, 2024
SUMMARY
Deprecate community.vmware.vmware_cluster_dpm in favor of vmware.vmware.cluster_dpm.
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
vmware_cluster_dpm
ADDITIONAL INFORMATION
ansible-collections/vmware.vmware#71
softwarefactory-project-zuul bot pushed a commit to ansible-collections/community.vmware that referenced this pull request Oct 25, 2024
Backport of #2217
SUMMARY
Deprecate community.vmware.vmware_cluster_dpm in favor of vmware.vmware.cluster_dpm.
ISSUE TYPE

Feature Pull Request

COMPONENT NAME
vmware_cluster_dpm
ADDITIONAL INFORMATION
ansible-collections/vmware.vmware#71
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants