Resolve "Fix review app / demo functionality"
Description
- The external postgres repo is now added in the
base
layer rather than thedev
layer - Version used is specified via
VAR POSTGRES_VERSION
to ensure consistency without hardcoding versions - The
review
layerCOPY
commands had the hardcoded version removed and an additional directory added - CI rules updated to let the pipeline pass even if
start_review
fails, as a deploy failure should not block an MR
Related issues
Notes to review (code/docs/QA)
Tests
General
-
Tests have been added that prove my fix is effective or that my feature works -
Related tests have been modified/removed
Hypothesis testing:
-
Soak testing has been done -
Distribution between positive / negative cases has been checked
Database
-
Includes changes to database schema -
Includes necessary database migrations
Configuration
-
Includes changes to configuration -
Includes configuration migration instructions in documentation
Closes #2123 (closed)
Edited by Tor Solli-Nowlan