Skip to content

Commit

Permalink
[MINOR] Batch mode mvn automation
Browse files Browse the repository at this point in the history
This commit change the most obvious places of maven
in github actions to run in batch mode. This removes the
printing of the download process from the github logs.
That sometimes would increase the build log from 300 lines to 300k
lines in just the build test.
  • Loading branch information
Baunsgaard committed Jul 19, 2023
1 parent fb828dd commit f3f4fca
Show file tree
Hide file tree
Showing 5 changed files with 6 additions and 6 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/build-cron.yml
Original file line number Diff line number Diff line change
Expand Up @@ -64,4 +64,4 @@ jobs:
cache: 'maven'

- name: Build
run: mvn package
run: mvn package -B
2 changes: 1 addition & 1 deletion .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -78,4 +78,4 @@ jobs:
cache: 'maven'

- name: Build
run: mvn package -D maven.test.skip=true
run: mvn package -D maven.test.skip=true -B
2 changes: 1 addition & 1 deletion .github/workflows/license.yml
Original file line number Diff line number Diff line change
Expand Up @@ -64,4 +64,4 @@ jobs:
cache: 'maven'

- name: Build
run: mvn package -P rat
run: mvn package -P rat -B
2 changes: 1 addition & 1 deletion .github/workflows/python.yml
Original file line number Diff line number Diff line change
Expand Up @@ -86,7 +86,7 @@ jobs:
key: ${{ runner.os }}-${{ hashFiles('.github/workflows/python.yml') }}

- name: Maven clean & package
run: mvn -ntp clean package -P distribution
run: mvn -ntp clean package -P distribution -B

- name: Setup Python ${{ matrix.python-version }}
uses: actions/setup-python@v4
Expand Down
4 changes: 2 additions & 2 deletions docker/entrypoint.sh
Original file line number Diff line number Diff line change
Expand Up @@ -29,8 +29,8 @@ cd /github/workspace
export MAVEN_OPTS="-Xmx512m -XX:MaxPermSize=128m"

log="/tmp/sysdstest.log"
mvn -ntp test-compile 2>&1 | grep -E "BUILD|Total time:|---|Building SystemDS"
mvn -ntp test -D maven.test.skip=false -D automatedtestbase.outputbuffering=true -D test=$1 2>&1 | grep -v "already exists in destination." | tee $log
mvn -ntp -B test-compile 2>&1 | grep -E "BUILD|Total time:|---|Building SystemDS"
mvn -ntp -B test -D maven.test.skip=false -D automatedtestbase.outputbuffering=true -D test=$1 2>&1 | grep -v "already exists in destination." | tee $log

grep_args="SUCCESS"
grepvals="$( tail -n 100 $log | grep $grep_args)"
Expand Down

0 comments on commit f3f4fca

Please sign in to comment.