Skip to content

[SPOC-83] Refactor test container for running regression tests#82

Merged
pct960 merged 6 commits intomainfrom
task/SPOC-83-docker-regress-refactor
Jul 11, 2025
Merged

[SPOC-83] Refactor test container for running regression tests#82
pct960 merged 6 commits intomainfrom
task/SPOC-83-docker-regress-refactor

Conversation

@mason-sharp
Copy link
Member

@mason-sharp mason-sharp commented Jul 9, 2025

Previously Postgres and Spock were compiled in entrypoint.sh as part of running spockbench. This is now moved to building these in the container, allowing us to use the same container to run regression tests as a separate step that can also be more easily viewed in the github workflow output.

Screenshot 2025-07-08 at 2 02 31 PM

@mason-sharp mason-sharp requested a review from pct960 July 9, 2025 01:45
@mason-sharp mason-sharp marked this pull request as draft July 9, 2025 01:50
@mason-sharp mason-sharp force-pushed the task/SPOC-83-docker-regress-refactor branch from bf88751 to aabc300 Compare July 10, 2025 17:27
@mason-sharp mason-sharp marked this pull request as ready for review July 10, 2025 17:49

WORKDIR /home/pgedge
RUN echo "Cloning Spock at $GITHUB_REF from $REPO_URL"
RUN git clone ${REPO_URL}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The repo is already checked out by the action. For the next update, we could perhaps simply copy the files while building the image.

@pct960 pct960 merged commit 069f5ba into main Jul 11, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants