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

Add managed disks list to dynamic inventory hostvars #687

Merged
merged 2 commits into from
Nov 24, 2021

Conversation

nbr23
Copy link
Contributor

@nbr23 nbr23 commented Nov 14, 2021

SUMMARY

Address #678 by adding managed disks list to dynamic inventory facts

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME
  • azure_rm (dynamic inventory)
ADDITIONAL INFORMATION

Add a new data_disks field in the inventory hostvars, listing the attached disks with their LUN, name, and azure ID.
This also adds the disk ID for the OS disk.

          data_disks:
          - id: /subscriptions/XXXXXX/resourceGroups/myRG-eastus2/providers/Microsoft.Compute/disks/nbr23vm-data1
            lun: 0
            name: nbr23vm-data1

@Fred-sun Fred-sun added inventory plugin/Inventory/azure_rm.py related issues medium_priority Medium priority work in In trying to solve, or in working with contributors ready_for_review The PR has been modified and can be reviewed and merged and removed work in In trying to solve, or in working with contributors labels Nov 22, 2021
@xuzhang3 xuzhang3 merged commit 79e56bb into ansible-collections:dev Nov 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
inventory plugin/Inventory/azure_rm.py related issues medium_priority Medium priority ready_for_review The PR has been modified and can be reviewed and merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants