chore: Add logs for default DB url for CI test workflows (#36506)
## Description ## Summary by CodeRabbit - **New Features** - Introduced a new job step to check the database type in the CI workflow, ensuring proper database configurations. - Added a step to determine if the build is for PostgreSQL in the server build workflow, providing clarity on build types. - **Bug Fixes** - Improved conditional execution in the CI workflow to enhance reliability. <!-- end of auto-generated comment: release notes by coderabbit.ai --> /test Sanity ### 🔍 Cypress test results <!-- This is an auto-generated comment: Cypress test results --> > [!CAUTION] > 🔴 🔴 🔴 Some tests have failed. > Workflow run: <https://github.com/appsmithorg/appsmith/actions/runs/11104222888> > Commit: df82014823c449954a96b6970168e456125c4f5e > <a href="https://internal.appsmith.com/app/cypress-dashboard/rundetails-65890b3c81d7400d08fa9ee5?branch=master&workflowId=11104222888&attempt=1&selectiontype=test&testsstatus=failed&specsstatus=fail" target="_blank">Cypress dashboard</a>. > Tags: @tag.Sanity > Spec: > The following are new failures, please fix them before merging the PR: <ol> > <li>cypress/e2e/Regression/ClientSide/Git/ExistingApps/v1.9.24/DSCrudAndBindings_Spec.ts</ol> > <a href="https://internal.appsmith.com/app/cypress-dashboard/identified-flaky-tests-65890b3c81d7400d08fa9ee3?branch=master" target="_blank">List of identified flaky tests</a>. > <hr>Mon, 30 Sep 2024 11:01:27 UTC <!-- end of auto-generated comment: Cypress test results --> ## Communication Should the DevRel and Marketing teams inform users about this change? - [ ] Yes - [ ] No <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit - **New Features** - Added a conditional check for the database URL in the CI workflow to ensure proper database configurations are in place. - Introduced a step to identify the database type during the server build process, enhancing build clarity. - **Bug Fixes** - Implemented error handling for missing database URLs to prevent build failures. <!-- end of auto-generated comment: release notes by coderabbit.ai --> --------- Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
This commit is contained in:
parent
bb11bf5fca
commit
367ac87c64
16
.github/workflows/ci-test-custom-script.yml
vendored
16
.github/workflows/ci-test-custom-script.yml
vendored
|
|
@ -197,6 +197,22 @@ jobs:
|
|||
uses: actions/setup-node@v4
|
||||
with:
|
||||
node-version-file: app/client/package.json
|
||||
|
||||
- name: Check DB URL
|
||||
if: steps.run_result.outputs.run_result != 'success'
|
||||
run: |
|
||||
db_url=$(grep -oP 'APPSMITH_DB_URL=\K[^ ]+' cicontainerlocal/stacks/configuration/docker.env || echo "")
|
||||
if [[ -z "$db_url" ]]; then
|
||||
echo "::error::APPSMITH_DB_URL not found in the environment file"
|
||||
exit 1
|
||||
fi
|
||||
if [[ $db_url == "postgresql"* ]]; then
|
||||
echo "Database type: Postgres. Ensure PostgreSQL-specific configurations are in place."
|
||||
elif [[ $db_url == "mongo"* ]]; then
|
||||
echo "Database type: MongoDB. Verify MongoDB connection settings if issues arise."
|
||||
else
|
||||
echo "::warning::Unknown database type. Please verify the database configuration."
|
||||
fi
|
||||
|
||||
# actions/setup-node@v4 doesn’t work properly with Yarn 3
|
||||
# when the project lives in a subdirectory: https://github.com/actions/setup-node/issues/488
|
||||
|
|
|
|||
3
.github/workflows/server-build.yml
vendored
3
.github/workflows/server-build.yml
vendored
|
|
@ -72,6 +72,9 @@ jobs:
|
|||
|
||||
- name: Figure out the PR number
|
||||
run: echo ${{ inputs.pr }}
|
||||
|
||||
- name: Default database URL
|
||||
run: echo "Is this a PG build? ${{ inputs.is-pg-build }}"
|
||||
|
||||
- name: Print the Github event
|
||||
run: echo ${{ github.event_name }}
|
||||
|
|
|
|||
Loading…
Reference in New Issue
Block a user