Skip to content

Commit

Permalink
Use core-module build profile to avoid downloading snapshot jars
Browse files Browse the repository at this point in the history
  • Loading branch information
aahmed-se committed Jul 23, 2020
1 parent 67bd345 commit 25be979
Show file tree
Hide file tree
Showing 8 changed files with 15 additions and 12 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/ci-cpp.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ jobs:
ref: ${{ github.event.pull_request.head.sha }}

- name: build package
run: mvn -B -ntp -q package -pl distribution/server -am -DskipTests
run: mvn -B -ntp -q install -Pcore-modules -DskipTests

- name: build cpp artifacts
run: |
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/ci-unit-broker-broker-gp1.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ jobs:
ref: ${{ github.event.pull_request.head.sha }}

- name: build modules
run: mvn -B -q -ntp install -pl pulsar-broker -am -DskipTests
run: mvn -B -ntp -q install -Pcore-modules -DskipTests

- name: run unit tests for pulsar-broker
run: mvn -B -ntp test -pl pulsar-broker -Dinclude='org/apache/pulsar/broker/**/*.java' -Dexclude='org/apache/pulsar/broker/zookeeper/**/*.java,org/apache/pulsar/broker/loadbalance/**/*.java,org/apache/pulsar/broker/service/**/*.java,**/AdminApiOffloadTest.java'
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/ci-unit-broker-broker-gp2.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ jobs:
ref: ${{ github.event.pull_request.head.sha }}

- name: build modules
run: mvn -B -q -ntp install -pl pulsar-broker -am -DskipTests
run: mvn -B -ntp -q install -Pcore-modules -DskipTests

- name: run flaky test
run: mvn -B -ntp test -pl pulsar-broker -Dinclude='**/MessagePublishBufferThrottleTest.java' -DtestForkCount=1 -DtestReuseFork=true
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/ci-unit-broker-client-api.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ jobs:
ref: ${{ github.event.pull_request.head.sha }}

- name: build modules
run: mvn -B -q -ntp install -pl pulsar-broker -am -DskipTests
run: mvn -B -ntp -q install -Pcore-modules -DskipTests

- name: run unit tests for pulsar-broker "org/apache/pulsar/client/impl"
run: mvn -B -ntp test -pl pulsar-broker -Dinclude='org/apache/pulsar/client/api/**/*.java' -Dexclude='**/DispatcherBlockConsumerTest.java'
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/ci-unit-broker-client-impl.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ jobs:
ref: ${{ github.event.pull_request.head.sha }}

- name: build modules
run: mvn -B -q -ntp install -pl pulsar-broker -am -DskipTests
run: mvn -B -ntp -q install -Pcore-modules -DskipTests

- name: run unit tests for pulsar-broker "org/apache/pulsar/client/impl"
run: mvn -B -ntp test -pl pulsar-broker -Dinclude='org/apache/pulsar/client/impl/**/*.java'
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/ci-unit-broker-other.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ jobs:
ref: ${{ github.event.pull_request.head.sha }}

- name: build modules
run: mvn -B -q -ntp install -pl pulsar-broker -am -DskipTests
run: mvn -B -ntp -q install -Pcore-modules -DskipTests

- name: run unit tests for pulsar-broker exlcude "org/apache/pulsar/broker", "org/apache/pulsar/client"
run: mvn -B -ntp test -pl pulsar-broker -Dexclude='org/apache/pulsar/broker/**/*.java,org/apache/pulsar/client/**/*.java'
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/ci-unit-proxy.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ jobs:
ref: ${{ github.event.pull_request.head.sha }}

- name: build modules pulsar-proxy
run: mvn -B -q -ntp install -pl pulsar-proxy -am -DskipTests
run: mvn -B -ntp -q install -Pcore-modules -DskipTests

- name: run flaky test
run: mvn -B -ntp install -pl pulsar-proxy -Dexclude="**/ProxyRolesEnforcementTest.java,**/ProxyAuthenticationTest.java,**/ProxyTest.java,**/MessagePublishBufferThrottleTest.java"
Expand Down
13 changes: 8 additions & 5 deletions .github/workflows/ci-unit.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -42,20 +42,23 @@ jobs:
fetch-depth: 0
ref: ${{ github.event.pull_request.head.sha }}

- name: build modules
run: mvn -B -ntp install -DskipTests

- name: run flaky test
run: mvn -B -ntp install -pl managed-ledger -Dinclude="**/ManagedLedgerTest.java" -DtestForkCount=1 -DtestReuseFork=true
run: mvn -B -ntp test -pl managed-ledger -Dinclude="**/ManagedLedgerTest.java" -DtestForkCount=1 -DtestReuseFork=true

- name: run flaky test
run: mvn -B -ntp install -pl pulsar-client-kafka-compat/pulsar-client-kafka_0_8 -Dinclude="**/KafkaProducerSimpleConsumerTest.java" -DtestForkCount=1 -DtestReuseFork=true
run: mvn -B -ntp test -pl pulsar-client-kafka-compat/pulsar-client-kafka_0_8 -Dinclude="**/KafkaProducerSimpleConsumerTest.java" -DtestForkCount=1 -DtestReuseFork=true

- name: run flaky test
run: mvn -B -ntp install -pl pulsar-sql/presto-pulsar-plugin -Dinclude="**/TestPulsarKeyValueSchemaHandler.java" -DtestForkCount=1 -DtestReuseFork=true
run: mvn -B -ntp test -pl pulsar-sql/presto-pulsar-plugin -Dinclude="**/TestPulsarKeyValueSchemaHandler.java" -DtestForkCount=1 -DtestReuseFork=true

- name: run flaky test
run: mvn -B -ntp install -pl pulsar-client -Dinclude="**/PrimitiveSchemaTest.java" -DtestForkCount=1 -DtestReuseFork=true
run: mvn -B -ntp test -pl pulsar-client -Dinclude="**/PrimitiveSchemaTest.java" -DtestForkCount=1 -DtestReuseFork=true

- name: run unit tests exclude pulsar-broker and pulsar-proxy
run: mvn -B -ntp install -PbrokerSkipTest -Dexclude="org/apache/pulsar/proxy/**/*.java,**/KafkaProducerSimpleConsumerTest.java,**/ManagedLedgerTest.java,**/TestPulsarKeyValueSchemaHandler.java,**/PrimitiveSchemaTest.java"
run: mvn -B -ntp test -PbrokerSkipTest -Dexclude="org/apache/pulsar/proxy/**/*.java,**/KafkaProducerSimpleConsumerTest.java,**/ManagedLedgerTest.java,**/TestPulsarKeyValueSchemaHandler.java,**/PrimitiveSchemaTest.java"

- name: package surefire artifacts
if: failure()
Expand Down

0 comments on commit 25be979

Please sign in to comment.