-
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 support for GCP Compute Firewalls #39032
Conversation
@GwenaelPellenArkeup @bennojoy @dohoangkhiem @erjohnso @GoogleCloudPlatform @nitaco @robwagner33 @supertom @walbert947 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 |
cddff52
to
d3a893d
Compare
d3a893d
to
dde3c7f
Compare
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.
Please check if you can provide name rather than URL for the network
parameter.
must be a dash, lowercase letter, or digit, except the last character, which cannot | ||
be a dash. | ||
required: false | ||
network: |
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.
This isn't used in the examples. Is it possible to do this by name instead of URL? It seems a little complex to have to provide a full URL to use non-default networks.
SUMMARY
Adding support for GCP Compute Firewalls
ISSUE TYPE
COMPONENT NAME
gcp_compute_firewall
ANSIBLE VERSION
ADDITIONAL INFORMATION
This was autogenerated using Magic Modules