-
Notifications
You must be signed in to change notification settings - Fork 83
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 GCP CloudSQL example #218
Conversation
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.
🎉 🎉 🎉 thank you @barney-s !
ada15ee
to
71df947
Compare
3fcdfc3
to
d88743d
Compare
Causes #225 |
d88743d
to
5da8156
Compare
examples/gcp/cloud-sql/rg.yaml
Outdated
name: iam-enablement | ||
spec: | ||
resourceID: iam.googleapis.com | ||
- id: servceusageEnable |
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.
Nit: serviceUsageEnable
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 would also be nice to make this optional, one less thing to worry about when writing a template. Obviously we couldn't refer to "unnamed variables" ... but maybe it is important for a stable status?
5da8156
to
1998442
Compare
This is verified on latest with explicit dependencies added. Works. Ready for review and merge. |
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.
👍
Has a primary and single replica.
source: https://github.com/GoogleCloudPlatform/k8s-config-connector/tree/master/experiments/compositions/samples/CloudSQL/composition