-
Notifications
You must be signed in to change notification settings - Fork 23.7k
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
adding missing Azure facts modules #54838
Conversation
@zikalino this PR contains more than one new module. Please submit only one new module per pull request. For a detailed explanation, please read the grouped modules documentation |
The test
|
The test
The test
|
The test
The test
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
general good, some comments
pass | ||
|
||
|
||
class AzureRMCustomImageFacts(AzureRMModuleBase): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
DevTestLabCustomImage?
pass | ||
|
||
|
||
class AzureRMEnvironmentFacts(AzureRMModuleBase): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
class name with devtestlab?
type: str | ||
sample: "/subscriptions/xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxxx//resourcegroups/myResourceGroup/providers/microsoft.devtestlab/labs/myLab/v | ||
irtualmachines/myLabVm" | ||
tags: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it has tags, can facts be filtered by tags?
@zikalino Need you update according by the comments! Thanks! |
@yungezz added "Dtl" to class names, also added facts. |
seems you didn't push |
SUMMARY
Several modules for DTL
ISSUE TYPE
COMPONENT NAME
azure_rm_devtestlabschedule_facts
azure_rm_devtestlabpolicy_facts
azure_rm_devtestlavenvironment_facts
azure_rm_devtestlabcustomimage_facts
ADDITIONAL INFORMATION
Currently DTL Tests are disabled, so the modules were tested manually.