Skip to content

(infra): Changing Runner Base Image to Include PDO Extension

Zack Wynne requested to merge fix/registered-runner into master

Ticket(s) / Related Merge Requests

Issue: #2342 (closed)

Summary of Changes

After adding the MySQL repository code, some runners are now failing with this error:

[06-Jul-2022 20:34:18 UTC] Minds.CRITICAL: Error: Undefined constant PDO::MYSQL_ATTR_SSL_CA in /var/www/Minds/engine/Core/Data/MySQL/Client.php:35

This is due to the mysql client being used in the absence of the PHP-PDO extension. One example runner is: minds-runner-registered.

Testing Considerations

We can test this by creating a new user and observing the above error in this registered runner.

Deployment Considerations

This is a runner change, so for testing you'll need to use the image for the appropriate pipeline ID rather than latest.

Regression Scope

This should affect the base image used for all of the runners.

Platform Affected (web, mobile, etc)

Web

Developer Testing Completed

Tested deploying to Sandbox with the fix and creating a user.

Screenshots / Screen Recording

image

Does this impact

  • Localization
  • Dark/light mode
  • Guest mode

Definition of Done Checklist

  • The Acceptance Criteria has been met
  • Code is tested: Testing includes unit/spec, E2E/automated and manual testing
  • Merge requests description has been filled out
Edited by Zack Wynne

Merge request reports