[Fix] Add postgresql client to app service

[petertgiles]

🤖 Resolves #10777

👋 Introduction

This branch adds the postgresql to the app service setup to ensure that a fresh setup is possible. It also adds it to the local webserver docker container to mirror the setup.

🕵️ Details

I created a custom Dockerfile for the webserver image. Before, we were pretty strict about using a pristine image but this is a convenient way to mimic some of the customization we perform on the image during post-deploy.

🧪 Testing

🏠 Locally

1. Rebuild the webserver image docker-compose build webserver
2. Composer install by whatever method you normally use

• Can perform a database reset in the webserver container docker-compose exec webserver sh -c "php /home/site/wwwroot/api/artisan migrate:fresh"

☁️ Azure

1. Deploy this branch to the dev vertical
2. SSH in

• Can perform a database reset php /home/site/wwwroot/api/artisan migrate:fresh

📸 Screenshot

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 67.89%. Comparing base (1ed0a49) to head (a7c438c).
Report is 2 commits behind head on main.

Additional details and impacted files

@@              Coverage Diff              @@
##               main   #10781       +/-   ##
=============================================
+ Coverage     38.27%   67.89%   +29.61%     
  Complexity     1652     1652               
=============================================
  Files          1021      234      -787     
  Lines         31191     6052    -25139     
  Branches       6509        0     -6509     
=============================================
- Hits          11939     4109     -7830     
+ Misses        19075     1943    -17132     
+ Partials        177        0      -177     

Flag	Coverage Δ
integrationtests	67.89% <ø> (ø)
unittests	?

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[tristan-orourke]

I believe I've followed all the steps, but I'm getting an error locally 😕 . I'm using docker + WSL.

[petertgiles]

I believe I've followed all the steps, but I'm getting an error locally 😕 . I'm using docker + WSL.

Perhaps you missed the step to rebuild the webserver container?
docker-compose build webserver

Or if you did, maybe it didn't do it and you can try the --force-recreate flag?

[tristan-orourke]

Okay, I ran through the steps again and it worked. 😅
php artisan migrate:fresh now works both in the webserver and maintenance containers. This kind of makes #10773 redundant, but I don't think there's a downside either.

[tristan-orourke]

Tested, looks good

[petertgiles]

This kind of makes #10773 redundant, but I don't think there's a downside either.

Yeah, true. I asked Eric if he wanted me to roll back those changes in this PR but he said the way it is now is fine.