Skip to content

Conversation

@mlm483
Copy link
Contributor

@mlm483 mlm483 commented Apr 9, 2025

Description

Story: https://breedinginsight.atlassian.net/browse/BI-2212

Update the postgres version to 17.
This change will affect local and the changes to bi-docker-stack (PR here) will affect qa-test, but not rel-test or production, which use RDS (those are a separate task).

Note that our CI already uses the postgres:latest image (see .github/workflows/build.yml for example).

Dependencies

None

Testing

Running this without existing data will work without issues.
To upgrade data locally, you can use pg_dump with the old database and then restore to the new postgres instance, but unless you really need that data locally, it's probably not worth it.

The upgrade for rel-test and production will be similar to the process I described, but slightly different because we use RDS, not a local postgres docker container.

Checklist:

  • I have performed a self-review of my own code
  • I have tested my code and ensured it meets the acceptance criteria of the story
  • I have tested that my code works with both the brapi-java-server and BreedBase
  • I have create/modified unit tests to cover this change
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to documentation
  • I have run TAF: <please include a link to TAF run>

@mlm483 mlm483 changed the title [BI-2212] - Update Postgres Version BI-2212 - Update Postgres Version Apr 9, 2025
@mlm483 mlm483 requested a review from sb2597 April 9, 2025 19:44
@mlm483 mlm483 requested a review from nickpalladino May 28, 2025 14:53
@mlm483 mlm483 merged commit 9d584e4 into develop May 28, 2025
1 check failed
@mlm483 mlm483 deleted the task/BI-2212 branch May 28, 2025 18:57
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