-
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 sql firewall rule facts #45059
Conversation
The test
The test
The test
|
de688fb
to
2e74768
Compare
@Madhura-CSI @brusMX @devigned @gustavomcarmo @haroldwongms @julienstroheker @lmazuel @obsoleted @sozercan @tripdubroot @trstringer @tstringer @xscript @yaweiw @yuwzho As a maintainer of a module in the same namespace this new module has been submitted to, your vote counts for shipits. Please review this module and add |
The test
|
--- | ||
module: azure_rm_sqlfirewallrule_facts | ||
version_added: "2.8" | ||
short_description: Get SQL Firewall Rule facts. |
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.
Azure
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.
fixed
options: | ||
resource_group: | ||
description: | ||
- The name of the resource group that contains the resource. You can obtain this value from the Azure Resource Manager API or the portal. |
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.
i think the "You can obtain...." sentence is not necessary, but it's minor
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.
removed
resource_group: | ||
description: | ||
- The name of the resource group that contains the resource. You can obtain this value from the Azure Resource Manager API or the portal. | ||
required: True |
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.
resoruce group requried?
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.
yes
description: | ||
- The name of the server. | ||
required: True | ||
name: |
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.
not support tag?
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.
no, firewall rule is subresource that doesn't have own tags
SUMMARY
Adding SQL Firewall Rule facts module
ISSUE TYPE
COMPONENT NAME
azure_rm_sqlfirewallrule_facts
ANSIBLE VERSION
2.7
ADDITIONAL INFORMATION