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

Sandbox stability improvements. #1244

Merged
merged 4 commits into from
Jan 16, 2023

Conversation

bcipriano
Copy link
Collaborator

Some minor changes to hopefully make the sandbox Docker compose setup more stable.

  • Lock the Postgres version to 15.1 to prevent breakage from new Postgres versions we haven't tested yet.
  • Switch the Flyway image from centos to almalinux and lock the version. Centos images are no longer officially supported, and Almalinux seems to be the closest replacement.
    • This allows us to remove some yum repo customizations that were needed on Cent.
  • Upgrade Flyway to a more recent version.
  • Update the Flyway migration script to improve debugging:
    • Use set -e to fail the script when migrations (or any other step) fails. This should make failures more obvious; without it, the container finishes with exit code zero even when it has failed.
    • Print a message when starting migrations to make the log output more clear.
  • Add a new script get-latest-release-tag.sh. This will be used by the quickstart guides to let users fetch the latest release version number. Currently this version number is hardcoded in the quickstart guides and needs to be updated every time there's a new release, which is easily forgotten.

@bcipriano
Copy link
Collaborator Author

Merging this TBR: changes are all outside the main codebase, and I'd like to unblock getting AcademySoftwareFoundation/opencue.io#270 merged and rolled out.

@bcipriano bcipriano merged commit 27bc9e1 into AcademySoftwareFoundation:master Jan 16, 2023
@bcipriano bcipriano deleted the sandbox-updates branch January 16, 2023 19:25
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.

None yet

1 participant