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

Added zalando database support for rest of the geoweb backends #58

Merged
merged 2 commits into from
Jan 11, 2024

Conversation

Jusaa
Copy link
Contributor

@Jusaa Jusaa commented Dec 22, 2023

Add support for using Zalando databases for geoweb presets, taf and warnings backends. This is basically copy from earlier PR where support was added to opmet-backend, so copying the descriptions from there: #39

Added option to use Zalando Operator Postgres database instead of connection string to external database

  • If useZalandoOperatorDb == true
    • Add postgresql resource
    • Use generated username & password created by postgresql resource to automatically set PRESETS_BACKEND_DB, AVIATION_TAF_BACKEND_DB and WARNINGS_BACKEND_DB instead of using db_secret
    • Don't generate db secret using db_secret
  • If useZalandoOperatorDb == false
    • No changes except for changed default database name (and encoded database string)
  • cleanInstall determines if database is started clean or restored from a backup

How to test:

Closes https://gitlab.com/opengeoweb/fmi/fmi-aws-config/-/issues/118 & https://gitlab.com/opengeoweb/fmi/fmi-aws-config/-/issues/196 & https://gitlab.com/opengeoweb/fmi/fmi-aws-config/-/issues/197

@Jusaa Jusaa changed the title Draft: Added zalando database support for rest of the geoweb backends Added zalando database support for rest of the geoweb backends Dec 22, 2023
@kasarit kasarit self-requested a review January 10, 2024 12:10
@EarthModule EarthModule self-requested a review January 11, 2024 08:31
Copy link
Contributor

@EarthModule EarthModule left a comment

Choose a reason for hiding this comment

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

GJ! Approved!

@Jusaa Jusaa merged commit 3e17cb0 into main Jan 11, 2024
1 check passed
@kasarit kasarit deleted the 118-196-197-add-zalando-databases branch January 16, 2024 15:32
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