Run app tests in the builder image #319

Merged
jgroman merged 1 commit from feature/313-sync-build-and-test-images into develop 2026-06-25 12:44:58 +00:00
Owner
  • Set testing image to ubi9/python-311
  • Skip installing packages which are already present in the image

Fixes #313

Assisted-by: Claude Code

- Set testing image to ubi9/python-311 - Skip installing packages which are already present in the image Fixes #313 Assisted-by: Claude Code
jgroman self-assigned this 2026-06-24 14:29:27 +00:00
Run app tests in the builder image
Some checks failed
Run tests and linters / test (pull_request) Failing after 21s
5748396c4b
- Set testing image to ubi9/python-311
- Skip installing packages which are already present in the image

Fixes #313

Assisted-by: Claude Code
Remove setuptools install step
All checks were successful
Run tests and linters / test (pull_request) Successful in 2m26s
b2a11672c8
kparal approved these changes 2026-06-25 08:45:08 +00:00
kparal left a comment

This shaves 20s from the CI run, which is great. I'm just not sure if we're still keeping the intended goal of matching the production environment?

If you think it's fine, then thumbs up.

This shaves 20s from the CI run, which is great. I'm just not sure if we're still keeping the intended goal of matching the production environment? If you think it's fine, then thumbs up.
@ -29,11 +29,8 @@ jobs:
- 3000:3000
container:
# This is chosen to match our production environment
Owner

Is this comment still valid?

Is this comment still valid?
Author
Owner

In fact the comment was not 100% accurate before but now it is fully valid - we use the same image to build and run the app in stg and prod environments.

In fact the comment was not 100% accurate before but now it is fully valid - we use the same image to build and run the app in stg and prod environments.
jgroman marked this conversation as resolved
jgroman force-pushed feature/313-sync-build-and-test-images from b2a11672c8
All checks were successful
Run tests and linters / test (pull_request) Successful in 2m26s
to faf72c2a77
All checks were successful
Run tests and linters / test (pull_request) Successful in 2m24s
2026-06-25 12:39:23 +00:00
Compare
jgroman merged commit faf72c2a77 into develop 2026-06-25 12:44:58 +00:00
jgroman deleted branch feature/313-sync-build-and-test-images 2026-06-25 12:44:58 +00:00
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
quality/blockerbugs!319
No description provided.