Skip to content

Resolve "Fix review app / demo functionality"

Tor Solli-Nowlan requested to merge 2123-fix-review-app-demo-functionality into dev

Description

  • The external postgres repo is now added in the base layer rather than the dev layer
  • Version used is specified via VAR POSTGRES_VERSION to ensure consistency without hardcoding versions
  • The review layer COPY 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

Merge request reports