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

Terraform to manage the required resources #85

Closed
wants to merge 2 commits into from

Conversation

ev1lm0nk3y
Copy link
Contributor

This sets up the CloudSQL db, SecretsManager and the IAM role and service account to access those.

As Helm will be used to manage the app, this will help manage the resources.

@ev1lm0nk3y ev1lm0nk3y requested a review from a team as a code owner April 18, 2024 17:50
@kigster kigster changed the base branch from main to kig/rails-7-new-app April 18, 2024 20:24
@kigster
Copy link
Collaborator

kigster commented Apr 18, 2024

It's really important that the base branch for these PRs is kig/rails7-new-app

Copy link
Collaborator

@kigster kigster left a comment

Choose a reason for hiding this comment

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

Not qualified for the review.

Please ask @mike-matera

@kigster kigster requested a review from mike-matera April 18, 2024 20:26
@ev1lm0nk3y
Copy link
Contributor Author

it was late, i'll repoen with a proper mapping

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.

2 participants