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

vmware_guest_disk: add backing_uuid return #348

Conversation

git-cgallagher
Copy link
Contributor

@git-cgallagher git-cgallagher commented Aug 15, 2020

SUMMARY

Adds backing_uuid value return for vmware_guest_disk module

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

vmware_guest_disk

ADDITIONAL INFORMATION

Negates the need to use vmware_guest_disk_info to return backing_uuid of a newly created disk.

"vm_disk_info": {
        "changed": true,
        "disk_changes": {
            "0": "Disk created."
        },
        "disk_data": {
            "0": {
                "backing_datastore": "WorkloadDatastore",
                "backing_disk_mode": "persistent",
                "backing_eagerlyscrub": false,
                "backing_filename": "[WorkloadDatastore] 68ec2a5f-dd1b-184f-487e-0e267151c681/US05SDB01A.vmdk",
                "backing_sharing": "sharingNone",
                "backing_thinprovisioned": true,
                "backing_uuid": "6000C29a-b1b5-1bf8-40d7-a2e2c43fb182",
                "backing_writethrough": false,
                "capacity_in_bytes": 69793218560,
                "capacity_in_kb": 68157440,
                "controller_key": 1000,
                "iolimit_limit": -1,
                "iolimit_shares_level": "normal",
                "iolimit_shares_limit": 1000,
                "key": 2000,
                "label": "Hard disk 1",
                "shares_level": "normal",
                "shares_limit": 1000,
                "summary": "68,157,440 KB",
                "unit_number": 0
            }

@ansibullbot
Copy link

@ansibullbot ansibullbot added affects_2.10 community_review feature This issue/PR relates to a feature request module module needs_ci This PR requires CI testing to be performed. Please close and re-open this PR to trigger CI needs_triage Needs a first human triage before being processed. new_contributor Help guide this first time contributor stale_ci CI is older than 7 days, rerun before merging labels Aug 19, 2020
@Akasurde Akasurde removed the needs_triage Needs a first human triage before being processed. label Aug 20, 2020
@Akasurde
Copy link
Member

recheck

@goneri goneri self-assigned this Aug 21, 2020
@goneri goneri added the gate label Aug 21, 2020
@ansible-zuul ansible-zuul bot merged commit 7e0f26f into ansible-collections:main Aug 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
affects_2.10 community_review feature This issue/PR relates to a feature request module module needs_ci This PR requires CI testing to be performed. Please close and re-open this PR to trigger CI new_contributor Help guide this first time contributor stale_ci CI is older than 7 days, rerun before merging
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants