ci: Run server tests on pg branch (#34372)
<!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit - **Chores** - Updated workflow conditions to ensure PostgreSQL build is triggered when the pull request head reference is 'pg'. - Simplified logic to unconditionally start the PostgreSQL container in the build process. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
This commit is contained in:
parent
ae161b14c8
commit
6ef4d2a6af
2
.github/workflows/quality-checks.yml
vendored
2
.github/workflows/quality-checks.yml
vendored
|
|
@ -45,7 +45,7 @@ jobs:
|
||||||
secrets: inherit
|
secrets: inherit
|
||||||
with:
|
with:
|
||||||
pr: ${{ github.event.pull_request.number }}
|
pr: ${{ github.event.pull_request.number }}
|
||||||
is-pg-build: ${{ github.event.pull_request.base.ref == 'pg' }}
|
is-pg-build: ${{ github.event.pull_request.base.ref == 'pg' || github.event.pull_request.head.ref == 'pg' }}
|
||||||
|
|
||||||
client-build:
|
client-build:
|
||||||
name: client-build
|
name: client-build
|
||||||
|
|
|
||||||
5
.github/workflows/server-build.yml
vendored
5
.github/workflows/server-build.yml
vendored
|
|
@ -139,11 +139,10 @@ jobs:
|
||||||
java-version: "17"
|
java-version: "17"
|
||||||
|
|
||||||
- name: Conditionally start PostgreSQL
|
- name: Conditionally start PostgreSQL
|
||||||
|
if: |
|
||||||
|
inputs.is-pg-build == 'true' && inputs.skip-tests != 'true'
|
||||||
run: |
|
run: |
|
||||||
if [[ ${{ inputs.is-pg-build }} == 'true' ]]; then
|
|
||||||
echo "Requesting PostgreSQL container to be started for pg specific build."
|
|
||||||
docker run --name appsmith-pg -p 5432:5432 -d -e POSTGRES_PASSWORD=password postgres:alpine postgres -N 1500
|
docker run --name appsmith-pg -p 5432:5432 -d -e POSTGRES_PASSWORD=password postgres:alpine postgres -N 1500
|
||||||
fi
|
|
||||||
|
|
||||||
# Retrieve maven dependencies from cache. After a successful run, these dependencies are cached again
|
# Retrieve maven dependencies from cache. After a successful run, these dependencies are cached again
|
||||||
- name: Cache maven dependencies
|
- name: Cache maven dependencies
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue
Block a user