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

CloudSQL fixes #903

Merged
merged 6 commits into from
Aug 24, 2022
Merged

CloudSQL fixes #903

merged 6 commits into from
Aug 24, 2022

Conversation

arueth
Copy link
Collaborator

@arueth arueth commented Aug 24, 2022

Background

While trying to deploy the Bank of Anthos application using CloudSQL I encountered several issues with the populate jobs not completing. It was determined that this was due to the populate jobs having a dependency on the config.yaml manifest

Change Summary

  • Updated to PostgreSQL 14 to match the in-cluster database
  • Moved the ConfigMaps for the populate jobs to be before the Job itself.
  • Added an apply of the config.yaml file before applying the populate jobs
  • Added the runAsNonRoot securityContext to populate-ledger-db.yaml.
  • Updated to the Bank of Anthos 0.5.6 version
  • Fixed an issue with the timestamp format for one of the populate jobs. The default date utility on Alpine does not support
    %N as it is a GNU extension.

Testing Procedure

Tested in a Qwiklab environment using a GKE Standard cluster.

@arueth arueth requested a review from a team as a code owner August 24, 2022 21:29
@snippet-bot
Copy link

snippet-bot bot commented Aug 24, 2022

Here is the summary of changes.

You are about to add 2 region tags.
You are about to delete 2 region tags.

This comment is generated by snippet-bot.
If you find problems with this result, please file an issue at:
https://github.com/googleapis/repo-automation-bots/issues.
To update this comment, add snippet-bot:force-run label or use the checkbox below:

  • Refresh this comment

@github-actions
Copy link

🚀 PR staged at http://104.197.66.143

1 similar comment
@github-actions
Copy link

🚀 PR staged at http://104.197.66.143

Copy link
Member

@bourgeoisor bourgeoisor left a comment

Choose a reason for hiding this comment

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

This LGTM!

@github-actions
Copy link

🚀 PR staged at http://104.197.66.143

@arueth arueth merged commit 25b25ee into main Aug 24, 2022
@arueth arueth deleted the cloudsql-fixes branch August 24, 2022 22:39
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