[Feature:System] Change Postgres version to 14#12452
Conversation
automateprojectmangement
Bot
moved this from Seeking Reviewer
to Work in Progress
in Submitty Development
Rkoester47
changed the title
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #12452 +/- ##
=========================================
Coverage 21.62% 21.62%
Complexity 9819 9819
=========================================
Files 268 268
Lines 36812 36812
Branches 490 490
=========================================
Hits 7960 7960
Misses 28366 28366
Partials 486 486
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
Rkoester47
changed the title
Christian2147
left a comment
Christian2147
left a comment
There was a problem hiding this comment.
Hey @Rkoester47 , thank you for your contribution!
Please update your PR title to use our required format
[<TYPE>:<MODULE>] <SUBJECT>, where <SUBJECT> is 40 chars max.
See also: https://submitty.org/developer/getting_started/make_a_pull_request
Please also use our PR description template:
https://raw.githubusercontent.com/Submitty/Submitty/refs/heads/main/.github/PULL_REQUEST_TEMPLATE.md
Since your changes affect the user interface, please include relevant screenshots before and after your change to demonstrate that you tested and debugged your PR. This will help reviewers check your work.
Be sure to link the issue that you are addressing in this PR. If the issue is completely closed with this PR say “Fixes #1234”, if it only partially addresses the issue, “work related to issue #1234”
Once this pull request adheres to all Submitty conventions, we will proceed with a full review.
This PR is a draft and a work in progress. It will be formatted correctly once I mark it ready for review. |
github-project-automation
Bot
moved this from Work in Progress
to Done
in Submitty Development
Rkoester47
changed the title
The VM and production prostres version is 14.20
Rkoester47
changed the title
DB check is failing, it seems like the version change requires the restrict tokens. Will revert if this is the incorrect solution.
This reverts commit 8ca0872.
|
@Rkoester47 What's the status of this? Let me know if there's anything I can do to help. |
dagemcn
removed
the
Abandoned PR - Needs New Owner
| @@ -0,0 +1,2 @@ | |||
| #!/bin/bash | |||
5 participants
Why is this Change Important & Necessary?
Currently, the version of postgres used in our CI testing is conflicting with the latest version, and is causing issues with migrations.
What is the New Behavior?
After unifying the version used between development VM, CI tests and production, migrations will no longer be impacted negatively.
What steps should a reviewer take to reproduce or test the bug or new feature?
Verify that the appropriate steps have been taken to ensure all relevant instances of Postgres are version 14.
We should also verify that nothing is broken when a migration is run and the DB check passes.
Automated Testing & Documentation
No automated testing.
Other information