Skip to content
This repository has been archived by the owner on Mar 31, 2022. It is now read-only.

GKE + CloudSQL deployment added #126

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

yorambenyaacov
Copy link

Hi,
Since many of us already have a GKE cluster running, I thought it might be a good idea to have a set of deployment YAMLs with a step by step howto for this deployment

@yorambenyaacov yorambenyaacov requested a review from a team as a code owner December 5, 2020 09:49
Copy link
Contributor

@d4nilos d4nilos left a comment

Choose a reason for hiding this comment

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

This was a good initiative, thanks for doing that! 💯

gke/README.md Outdated Show resolved Hide resolved
gke/README.md Outdated Show resolved Hide resolved

```bash
gcloud sql instances create ${DB_INSTANCE_NAME} \
--database-version=POSTGRES_9_6 \
Copy link
Contributor

Choose a reason for hiding this comment

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

Since we are doing this, maybe it's interesting to use Cloud SQL on Postgres version 13 instead of 9.6.
What do you think?
It has a bunch of interesting performance improvements.
E.g. Smaller indexes and quicker queries.

Copy link
Author

Choose a reason for hiding this comment

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

I was not sure if there are any specific dependencies on version 9.6. If the dev team is OK with version 13, I can do the change :-)

gke/README.md Outdated Show resolved Hide resolved
balaji and others added 3 commits December 8, 2020 16:55
Co-authored-by: Danilo Augusto Silva <[email protected]>
Co-authored-by: Danilo Augusto Silva <[email protected]>
Co-authored-by: Danilo Augusto Silva <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants