-
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 Target Https Proxys #38908
Adding support for GCP Compute Target Https Proxys #38908
Conversation
The test
|
42a4233
to
38226d3
Compare
@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 |
38226d3
to
5253f23
Compare
5253f23
to
fa42ecb
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.
Since this only has a one-argument difference (SSL cert) from this module #38622 , is it possible to merge them and distinguish whether to create an HTTP or HTTPS target proxy?
I'd prefer not to merge them. It's going to be very difficult with the code generated approach. Additionally, these are treated as completely independent objects from GCP's perspective. They use different REST endpoints, different gcloud CLI commands, etc. While the objects are very similar currently, that's an assumption we can't make for the future, since they're considered completely separate resources. My worry is that we create a merged TargetHttp(s) proxy object now and then have to retrain users to think of those resources separately when the APIs inevitably diverge down the line. |
rebuild_merge |
SUMMARY
Adding support for GCP Compute Target Https Proxys
ISSUE TYPE
COMPONENT NAME
gcp_compute_target_https_proxy
ANSIBLE VERSION
ADDITIONAL INFORMATION
This was autogenerated using Magic Modules