Skip to content
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

Merged
merged 2 commits into from
Jan 30, 2025
Merged

Add GCP CloudSQL example #218

merged 2 commits into from
Jan 30, 2025

Conversation

barney-s
Copy link
Member

@barney-s barney-s changed the title [WIP] Add GCP CloudSQL example [WIP DoNotMerge] Add GCP CloudSQL example Jan 28, 2025
a-hilaly
a-hilaly previously approved these changes Jan 28, 2025
Copy link
Member

@a-hilaly a-hilaly left a 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 !

@barney-s barney-s force-pushed the gcp-cloudsql branch 3 times, most recently from ada15ee to 71df947 Compare January 29, 2025 01:20
@barney-s barney-s changed the title [WIP DoNotMerge] Add GCP CloudSQL example Add GCP CloudSQL example Jan 29, 2025
@barney-s barney-s force-pushed the gcp-cloudsql branch 2 times, most recently from 3fcdfc3 to d88743d Compare January 29, 2025 04:07
@barney-s
Copy link
Member Author

Causes #225
The definition seems good and can be merged.

name: iam-enablement
spec:
resourceID: iam.googleapis.com
- id: servceusageEnable
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nit: serviceUsageEnable

Copy link
Contributor

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?

@barney-s barney-s added this to the 0.2.0 milestone Jan 29, 2025
@barney-s
Copy link
Member Author

This is verified on latest with explicit dependencies added. Works. Ready for review and merge.

Copy link
Member

@a-hilaly a-hilaly left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@a-hilaly a-hilaly merged commit 79e0b5a into kro-run:main Jan 30, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants