Skip to content

Commit

Permalink
fix process compose logs (#2544)
Browse files Browse the repository at this point in the history
* Fix process compose logs on CI

* CI: maybe fix process compose logs

Remove -D detach when running process compose. Currently we only get the
process compose internal logs which aren't useful. We would like to have
the logs from the native demo.

---------

Co-authored-by: Mathis <[email protected]>
  • Loading branch information
imabdulbasit and sveitser authored Mar 7, 2025
1 parent ccde27c commit 01b5c08
Showing 1 changed file with 8 additions and 2 deletions.
10 changes: 8 additions & 2 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -212,10 +212,10 @@ jobs:
version: [02,99]
include:
- version: 02
compose: "-f process-compose.yaml -D"
compose: "-f process-compose.yaml"

- version: 99
compose: "-f process-compose.yaml -f process-compose-mp.yml -D"
compose: "-f process-compose.yaml -f process-compose-mp.yml"
fail-fast: false
runs-on: ubuntu-latest
steps:
Expand Down Expand Up @@ -255,13 +255,19 @@ jobs:
--workspace-remap $PWD $(if [ "${{ matrix.version }}" == "2" ]; then echo " smoke"; fi)
timeout-minutes: 10

- name: Show end of logs
if: always()
run: |
tail -n 1000 ${{ env.PC_LOGS }}
- name: Upload process compose logs
if: always()
uses: actions/upload-artifact@v4
with:
name: process-compose-logs-integration-v${{ matrix.version }}
path: ${{ env.PC_LOGS }}


demo-native:
needs: build-test-bins
runs-on: ubuntu-latest
Expand Down

0 comments on commit 01b5c08

Please sign in to comment.