Skip to content
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
80 commits
Select commit Hold shift + click to select a range
2be9f9b
improve rocksDB:
StevenLuMT Jan 21, 2022
268c825
Auditor should get the LegdgerManagerFactory from the client instance…
merlimat Jan 26, 2022
e089b51
[build] remove Maven POM files (#3009)
nicoloboschi Jan 26, 2022
62aca0a
Upgraded snakeyaml, CVE-2017-18640 (#3014)
dlg99 Jan 27, 2022
de3019e
Upgrading protobuf, CVE-2021-22569 (#3013)
dlg99 Jan 27, 2022
036fc1f
Added OWASP dependency-check (#3002)
dlg99 Jan 27, 2022
568fea8
[build] do not run spotbugsTest by default (#3017)
nicoloboschi Jan 27, 2022
317080d
Forcing the same version of netty across the projects
dlg99 Feb 2, 2022
f02e9e0
Forced the same version of guava (plus upgraded it); fixed deprecatio…
dlg99 Feb 2, 2022
3f06839
Upgrade Netty to 4.1.73.Final
nicoloboschi Feb 2, 2022
d7e5c51
[website] Remove Maven references and remove dead links from README (…
nicoloboschi Feb 3, 2022
b8351cb
CI workflow to check dependencies with OWASP (#3021)
dlg99 Feb 3, 2022
ee8c5bc
[BACKWARD TESTS] add BookKeeper 4.14.4
nicoloboschi Feb 4, 2022
d9c4021
Gradle 6.9.2 (#3022)
mauricebarnum Feb 5, 2022
dcf1281
Bump gradle to 7.3.3, compat java 17 (#2924)
Feb 5, 2022
1eae5e4
[tests] remove backward compatibility test against yahoo-version (apa…
nicoloboschi Feb 8, 2022
0725255
Support specifying bookie http port as a command argument (#2769)
fantapsody Feb 9, 2022
5405ee9
Fix performance issue to avoid unnessary loop. (#3030)
mattisonchao Feb 9, 2022
7d35b67
Upgrade RocksDB
dlg99 Feb 9, 2022
098ceac
Skip update entryLogMetaMap if not modified (#2964)
hangc0276 Feb 11, 2022
eabf7b1
Add rack name invalid check (#2980)
hangc0276 Feb 11, 2022
48a87a8
Support multi ledger directories for rocksdb backend entryMetadataMap…
hangc0276 Feb 11, 2022
0de75bc
[ISSUE 3038] Fixed flaky CompactionTest.testMinorCompactionWithMaxTim…
dlg99 Feb 11, 2022
4debbbf
change log level from error to warn when dns resolver initialize fail…
hangc0276 Feb 14, 2022
4710fed
Ensure BookKeeper process receives sigterm in docker container
Vanlightly Feb 14, 2022
50f5287
make rocksdb format version configurable
hangc0276 Feb 14, 2022
50b8a23
delete duplicated semicolon
gaozhangmin Feb 14, 2022
8530d5c
BP-46: Running without journal proposal
Vanlightly Feb 14, 2022
9fd35f9
Replication stat num-under-replicated-ledgers changed as with the pro…
gaozhangmin Feb 14, 2022
743def3
Explicit error message if an exception other than BKNoSuchLedgerExist…
shustsud Feb 14, 2022
c5a5167
fix(cli): incorrect description for autodiscovery
ericsyh Feb 14, 2022
0d98266
Add flaky-test template to track many flaky-test.
mattisonchao Feb 14, 2022
7a13905
fix gradle implicit dependency (#3029)
mauricebarnum Feb 14, 2022
f19544b
Use OutOfMemoryPolicy when the direct memory is insufficient when rea…
wenbingshen Feb 14, 2022
80a1100
[CI] Dump stacktrace when a job is cancelled
nicoloboschi Feb 15, 2022
a3056b5
update metrics (#2999)
StevenLuMT Feb 15, 2022
93e2249
Bump netty version to 4.1.74.Final (#3045)
Feb 15, 2022
9b1b188
ISSUE #3040: RocksDB segfaulted during CompactionTest
dlg99 Feb 15, 2022
7018d2f
ISSUE #3034: Add extra checks in the mock to help with error troubles…
dlg99 Feb 15, 2022
dd29b8e
change rocksdb init: use OptionsUtil
StevenLuMT Feb 16, 2022
ec74ea7
Auditor should get the LegdgerManagerFactory from the client instance…
merlimat Jan 26, 2022
da97ab0
[build] remove Maven POM files (#3009)
nicoloboschi Jan 26, 2022
4df57a8
Upgraded snakeyaml, CVE-2017-18640 (#3014)
dlg99 Jan 27, 2022
9ce0651
Upgrading protobuf, CVE-2021-22569 (#3013)
dlg99 Jan 27, 2022
145d7fe
Added OWASP dependency-check (#3002)
dlg99 Jan 27, 2022
0c650c1
[build] do not run spotbugsTest by default (#3017)
nicoloboschi Jan 27, 2022
921c24d
Forcing the same version of netty across the projects
dlg99 Feb 2, 2022
ec823fe
Forced the same version of guava (plus upgraded it); fixed deprecatio…
dlg99 Feb 2, 2022
3cc9d47
Upgrade Netty to 4.1.73.Final
nicoloboschi Feb 2, 2022
7044e0a
[website] Remove Maven references and remove dead links from README (…
nicoloboschi Feb 3, 2022
8e0ad18
CI workflow to check dependencies with OWASP (#3021)
dlg99 Feb 3, 2022
ed19227
[BACKWARD TESTS] add BookKeeper 4.14.4
nicoloboschi Feb 4, 2022
1045705
Gradle 6.9.2 (#3022)
mauricebarnum Feb 5, 2022
52c4f9c
Bump gradle to 7.3.3, compat java 17 (#2924)
Feb 5, 2022
74f3d4b
[tests] remove backward compatibility test against yahoo-version (apa…
nicoloboschi Feb 8, 2022
da6c44e
Support specifying bookie http port as a command argument (#2769)
fantapsody Feb 9, 2022
7907a59
Fix performance issue to avoid unnessary loop. (#3030)
mattisonchao Feb 9, 2022
f4eb77d
Upgrade RocksDB
dlg99 Feb 9, 2022
b623fd4
Skip update entryLogMetaMap if not modified (#2964)
hangc0276 Feb 11, 2022
874f038
Add rack name invalid check (#2980)
hangc0276 Feb 11, 2022
2a63101
Support multi ledger directories for rocksdb backend entryMetadataMap…
hangc0276 Feb 11, 2022
0c11c73
[ISSUE 3038] Fixed flaky CompactionTest.testMinorCompactionWithMaxTim…
dlg99 Feb 11, 2022
cf6eeea
change log level from error to warn when dns resolver initialize fail…
hangc0276 Feb 14, 2022
1997934
Ensure BookKeeper process receives sigterm in docker container
Vanlightly Feb 14, 2022
028cf0e
make rocksdb format version configurable
hangc0276 Feb 14, 2022
d0c9e5c
delete duplicated semicolon
gaozhangmin Feb 14, 2022
3443cec
BP-46: Running without journal proposal
Vanlightly Feb 14, 2022
10049f5
Replication stat num-under-replicated-ledgers changed as with the pro…
gaozhangmin Feb 14, 2022
c7d3c69
Explicit error message if an exception other than BKNoSuchLedgerExist…
shustsud Feb 14, 2022
e1e46cc
fix(cli): incorrect description for autodiscovery
ericsyh Feb 14, 2022
cee0627
Add flaky-test template to track many flaky-test.
mattisonchao Feb 14, 2022
f9bd25a
fix gradle implicit dependency (#3029)
mauricebarnum Feb 14, 2022
91a5da8
Use OutOfMemoryPolicy when the direct memory is insufficient when rea…
wenbingshen Feb 14, 2022
80c15c8
[CI] Dump stacktrace when a job is cancelled
nicoloboschi Feb 15, 2022
5fa92ce
update metrics (#2999)
StevenLuMT Feb 15, 2022
c9ba35c
Bump netty version to 4.1.74.Final (#3045)
Feb 15, 2022
e48bcdd
ISSUE #3040: RocksDB segfaulted during CompactionTest
dlg99 Feb 15, 2022
9a71247
ISSUE #3034: Add extra checks in the mock to help with error troubles…
dlg99 Feb 15, 2022
d9c5039
change rocksdb init: use OptionsUtil
StevenLuMT Feb 16, 2022
c1e2804
Merge remote-tracking branch 'origin/master_improveRocksDB' into mast…
StevenLuMT Feb 16, 2022
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
31 changes: 31 additions & 0 deletions .github/ISSUE_TEMPLATE/flaky_test.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,31 @@
---
name: Flaky test
about: Report a flaky test failure
title: 'Flaky-test: test_class.test_method'
labels: ["area/tests", "flaky-tests"]
assignees: ''
---
<!---

Instructions for reporting a flaky test using this issue template:

1. Replace test_class in title and body with the short test class name WITHOUT the package name.
2. Replace test_method in title and body with the test method that failed. Multiple methods are flaky, remove the content that refers to the test method.
3. Replace "relevant parts of the exception stacktrace here" with the a few lines of the stack trace that shows at least the exception message and the line of test code where the stacktrace occurred.
4. Replace "full exception stacktrace here" with the full exception stacktrace from logs. This section will be hidden by default.
5. Remove all unused fields / content to unclutter the reported issue. Remove this comment too.

-->
test_class.test_method is flaky. It fails sporadically.

```
[relevant parts of the exception stacktrace here]
```

<!-- optionally provide the full stacktrace ->
<details>
<summary>Full exception stacktrace</summary>
<code><pre>
full exception stacktrace here
</pre></code>
</details>
10 changes: 5 additions & 5 deletions .github/workflows/backward-compat-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ on:
workflow_dispatch:

env:
MAVEN_OPTS: -Dmaven.wagon.httpconnectionManager.ttlSeconds=25 -Dmaven.wagon.http.retryHandler.count=3
GRADLE_ARGS: -Dtestlogger.theme=plain -DtestHideStandardOut=true

jobs:
test:
Expand All @@ -49,10 +49,10 @@ jobs:
with:
java-version: 1.8
- name: Build
run: ./gradlew stream:server:build -x test
run: ./gradlew stream:server:build -x test ${GRADLE_ARGS}
- name: Test current server with old clients
run: ./gradlew :tests:backward-compat:current-server-old-clients:test || (tail -n +1 tests/backward-compat/current-server-old-clients/build/reports/tests/test/classes/* && tail -n +1 tests/backward-compat/current-server-old-clients/build/container-logs/**/* && exit 1)
run: ./gradlew :tests:backward-compat:current-server-old-clients:test ${GRADLE_ARGS} || (tail -n +1 tests/backward-compat/current-server-old-clients/build/reports/tests/test/classes/* && tail -n +1 tests/backward-compat/current-server-old-clients/build/container-logs/**/* && exit 1)
- name: Test progressive upgrade
run: ./gradlew :tests:backward-compat:upgrade:test || (tail -n +1 tests/backward-compat/upgrade/build/reports/tests/test/classes/* && tail -n +1 tests/backward-compat/upgrade/build/container-logs/**/* && exit 1)
run: ./gradlew :tests:backward-compat:upgrade:test ${GRADLE_ARGS} || (tail -n +1 tests/backward-compat/upgrade/build/reports/tests/test/classes/* && tail -n +1 tests/backward-compat/upgrade/build/container-logs/**/* && exit 1)
- name: Other tests
run: ./gradlew :tests:backward-compat:test -x tests:backward-compat:upgrade:test -x :tests:backward-compat:current-server-old-clients:test
run: ./gradlew :tests:backward-compat:test -x tests:backward-compat:upgrade:test -x :tests:backward-compat:current-server-old-clients:test ${GRADLE_ARGS}
9 changes: 8 additions & 1 deletion .github/workflows/bookie-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,9 @@ on:
- 'site/**'
workflow_dispatch:

env:
GRADLE_ARGS: -Dtestlogger.theme=plain -DtestHideStandardOut=true

jobs:
test:

Expand All @@ -47,4 +50,8 @@ jobs:
java-version: 1.8

- name: Run bookie test
run: ./gradlew bookkeeper-server:test --tests="org.apache.bookkeeper.bookie.*" -Dtestlogger.theme=plain
run: ./gradlew bookkeeper-server:test --tests="org.apache.bookkeeper.bookie.*" ${GRADLE_ARGS}

- name: print JVM thread dumps when cancelled
if: cancelled()
run: ./dev/ci-tool print_thread_dumps
9 changes: 8 additions & 1 deletion .github/workflows/client-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,9 @@ on:
- 'site/**'
workflow_dispatch:

env:
GRADLE_ARGS: -Dtestlogger.theme=plain -DtestHideStandardOut=true

jobs:
test:

Expand All @@ -46,4 +49,8 @@ jobs:
with:
java-version: 1.8
- name: Run client tests
run: ./gradlew bookkeeper-server:test --tests="org.apache.bookkeeper.client.*" -Dtestlogger.theme=plain
run: ./gradlew bookkeeper-server:test --tests="org.apache.bookkeeper.client.*" ${GRADLE_ARGS}

- name: print JVM thread dumps when cancelled
if: cancelled()
run: ./dev/ci-tool print_thread_dumps
8 changes: 7 additions & 1 deletion .github/workflows/compatibility-check-java11.yml
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,9 @@ on:
- 'site/**'
workflow_dispatch:

env:
GRADLE_ARGS: -Dtestlogger.theme=plain -DtestHideStandardOut=true

jobs:
check:

Expand All @@ -47,4 +50,7 @@ jobs:
java-version: 1.11
- name: Build with gradle
run: |
./gradlew test -x bookkeeper-server:test -x tests:integration:cluster:test -x tests:integration:smoke:test -x tests:integration:standalone:test -Dtestlogger.theme=plain -PexcludeTests="**/distributedlog/**, **/statelib/**, **/clients/**, **/*common/**, **/stream/**, **/stream/*bk*/**, **/*backward*/**"
./gradlew test -x bookkeeper-server:test -x tests:integration:cluster:test -x tests:integration:smoke:test -x tests:integration:standalone:test -PexcludeTests="**/distributedlog/**, **/statelib/**, **/clients/**, **/*common/**, **/stream/**, **/stream/*bk*/**, **/*backward*/**" ${GRADLE_ARGS}
- name: print JVM thread dumps when cancelled
if: cancelled()
run: ./dev/ci-tool print_thread_dumps
9 changes: 8 additions & 1 deletion .github/workflows/compatibility-check-java8.yml
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,9 @@ on:
- 'site/**'
workflow_dispatch:

env:
GRADLE_ARGS: -Dtestlogger.theme=plain -DtestHideStandardOut=true

jobs:
check:

Expand All @@ -47,4 +50,8 @@ jobs:
java-version: 1.8
- name: Build with gradle
run: |
./gradlew test -x bookkeeper-server:test -x tests:integration:cluster:test -x tests:integration:smoke:test -x tests:integration:standalone:test -Dtestlogger.theme=plain -PexcludeTests="**/distributedlog/**, **/statelib/**, **/clients/**, **/*common/**, **/stream/**, **/stream/*bk*/**, **/*backward*/**"
./gradlew test -x bookkeeper-server:test -x tests:integration:cluster:test -x tests:integration:smoke:test -x tests:integration:standalone:test -PexcludeTests="**/distributedlog/**, **/statelib/**, **/clients/**, **/*common/**, **/stream/**, **/stream/*bk*/**, **/*backward*/**" ${GRADLE_ARGS}

- name: print JVM thread dumps when cancelled
if: cancelled()
run: ./dev/ci-tool print_thread_dumps
14 changes: 9 additions & 5 deletions .github/workflows/integration-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ on:
workflow_dispatch:

env:
MAVEN_OPTS: -Dmaven.wagon.httpconnectionManager.ttlSeconds=25 -Dmaven.wagon.http.retryHandler.count=3
GRADLE_ARGS: -Dtestlogger.theme=plain -DtestHideStandardOut=true

jobs:
test:
Expand All @@ -50,10 +50,14 @@ jobs:
java-version: 1.8

- name: Build tar
run: ./gradlew stream:server:build -x test
run: ./gradlew stream:server:build -x test ${GRADLE_ARGS}
- name: run cluster integration test
run: ./gradlew :tests:integration:cluster:test -Dtestlogger.theme=plain
run: ./gradlew :tests:integration:cluster:test ${GRADLE_ARGS}
- name: run smoke test
run: ./gradlew tests:integration:smoke:test -Dtestlogger.theme=plain
run: ./gradlew tests:integration:smoke:test ${GRADLE_ARGS}
- name: run standalone test
run: ./gradlew tests:integration:standalone:test -Dtestlogger.theme=plain
run: ./gradlew tests:integration:standalone:test ${GRADLE_ARGS}

- name: print JVM thread dumps when cancelled
if: cancelled()
run: ./dev/ci-tool print_thread_dumps
59 changes: 59 additions & 0 deletions .github/workflows/owasp-dep-check.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,59 @@
#
# Licensed to the Apache Software Foundation (ASF) under one
# or more contributor license agreements. See the NOTICE file
# distributed with this work for additional information
# regarding copyright ownership. The ASF licenses this file
# to you under the Apache License, Version 2.0 (the
# "License"); you may not use this file except in compliance
# with the License. You may obtain a copy of the License at
#
# http://www.apache.org/licenses/LICENSE-2.0
#
# Unless required by applicable law or agreed to in writing,
# software distributed under the License is distributed on an
# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
# KIND, either express or implied. See the License for the
# specific language governing permissions and limitations
# under the License.
#

name: OWASP Dependency Check

on:
push:
pull_request:
branches:
- master
- branch-*
paths-ignore:
- 'site/**'
workflow_dispatch:


jobs:
check:

runs-on: ubuntu-latest
timeout-minutes: 60
steps:
- name: Checkout
uses: actions/checkout@v2

- name: Tune Runner VM
uses: ./.github/actions/tune-runner-vm

- name: Set up JDK 1.8
uses: actions/setup-java@v1
with:
java-version: 1.8

- name: run "clean build dependencyCheckAggregate" to trigger dependency check
run: ./gradlew clean build -x signDistTar -x test dependencyCheckAggregate

- name: Upload report
uses: actions/upload-artifact@v2
if: ${{ cancelled() || failure() }}
continue-on-error: true
with:
name: dependency report
path: build/reports/dependency-check-report.html
8 changes: 6 additions & 2 deletions .github/workflows/pr-validation.yml
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,9 @@ on:
- 'site/**'
workflow_dispatch:

env:
GRADLE_ARGS: -Dtestlogger.theme=plain -DtestHideStandardOut=true


jobs:
check:
Expand All @@ -47,6 +50,7 @@ jobs:
with:
java-version: 1.8
- name: Validate pull request
run: ./gradlew build -x spotbugsTest -x signDistTar -x test
run: ./gradlew build -x signDistTar -x test ${GRADLE_ARGS}

- name: Check license files
run: dev/check-all-licenses-gradle
run: dev/check-all-licenses
10 changes: 9 additions & 1 deletion .github/workflows/remaining-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,10 @@ on:
- 'site/**'
workflow_dispatch:

env:
GRADLE_ARGS: -Dtestlogger.theme=plain -DtestHideStandardOut=true


jobs:
test:

Expand All @@ -46,4 +50,8 @@ jobs:
with:
java-version: 1.8
- name: Run remaining tests
run: ./gradlew bookkeeper-server:test -PexcludeTests="*org.apache.bookkeeper.bookie.*, *org.apache.bookkeeper.client.*, *org.apache.bookkeeper.replication.*, *org.apache.bookkeeper.tls.*" -Dtestlogger.theme=plain
run: ./gradlew bookkeeper-server:test -PexcludeTests="*org.apache.bookkeeper.bookie.*, *org.apache.bookkeeper.client.*, *org.apache.bookkeeper.replication.*, *org.apache.bookkeeper.tls.*" ${GRADLE_ARGS}

- name: print JVM thread dumps when cancelled
if: cancelled()
run: ./dev/ci-tool print_thread_dumps
8 changes: 6 additions & 2 deletions .github/workflows/replication-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ on:
workflow_dispatch:

env:
MAVEN_OPTS: -Dmaven.wagon.httpconnectionManager.ttlSeconds=25 -Dmaven.wagon.http.retryHandler.count=3
GRADLE_ARGS: -Dtestlogger.theme=plain -DtestHideStandardOut=true

jobs:
test:
Expand All @@ -49,4 +49,8 @@ jobs:
with:
java-version: 1.8
- name: Run replication tests
run: ./gradlew bookkeeper-server:test --tests="org.apache.bookkeeper.replication.*" -Dtestlogger.theme=plain
run: ./gradlew bookkeeper-server:test --tests="org.apache.bookkeeper.replication.*" ${GRADLE_ARGS}

- name: print JVM thread dumps when cancelled
if: cancelled()
run: ./dev/ci-tool print_thread_dumps
23 changes: 15 additions & 8 deletions .github/workflows/stream-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,9 @@ on:
- 'site/**'
workflow_dispatch:

env:
GRADLE_ARGS: -Dtestlogger.theme=plain -DtestHideStandardOut=true

jobs:
test:

Expand All @@ -45,19 +48,23 @@ jobs:
with:
java-version: 1.8
- name: Run stream:distributedlog:core tests
run: ./gradlew stream:distributedlog:core:test -Dtestlogger.theme=plain
run: ./gradlew stream:distributedlog:core:test ${GRADLE_ARGS}
- name: Run stream:distributedlog:common tests
run: ./gradlew stream:distributedlog:common:test -Dtestlogger.theme=plain
run: ./gradlew stream:distributedlog:common:test ${GRADLE_ARGS}
- name: Run stream:distributedlog:protocol tests
run: ./gradlew stream:distributedlog:protocol:test -Dtestlogger.theme=plain
run: ./gradlew stream:distributedlog:protocol:test ${GRADLE_ARGS}
- name: Run stream:proto tests
run: ./gradlew stream:proto:test -Dtestlogger.theme=plain
run: ./gradlew stream:proto:test ${GRADLE_ARGS}
- name: Run stream:serve tests
run: ./gradlew stream:server:test -Dtestlogger.theme=plain
run: ./gradlew stream:server:test ${GRADLE_ARGS}
- name: Run stream:statelib tests
run: ./gradlew stream:statelib:test -Dtestlogger.theme=plain
run: ./gradlew stream:statelib:test ${GRADLE_ARGS}
- name: Run stream:storage:api:test tests
run: ./gradlew stream:storage:api:test -Dtestlogger.theme=plain
run: ./gradlew stream:storage:api:test ${GRADLE_ARGS}
- name: Run stream:storage:impl tests
run: ./gradlew stream:storage:impl:test -Dtestlogger.theme=plain
run: ./gradlew stream:storage:impl:test ${GRADLE_ARGS}

- name: print JVM thread dumps when cancelled
if: cancelled()
run: ./dev/ci-tool print_thread_dumps

9 changes: 8 additions & 1 deletion .github/workflows/tls-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,9 @@ on:
- 'site/**'
workflow_dispatch:

env:
GRADLE_ARGS: -Dtestlogger.theme=plain -DtestHideStandardOut=true

jobs:
test:

Expand All @@ -46,4 +49,8 @@ jobs:
with:
java-version: 1.8
- name: Run tls tests
run: ./gradlew bookkeeper-server:test --tests="org.apache.bookkeeper.tls.*" -Dtestlogger.theme=plain
run: ./gradlew bookkeeper-server:test --tests="org.apache.bookkeeper.tls.*" ${GRADLE_ARGS}

- name: print JVM thread dumps when cancelled
if: cancelled()
run: ./dev/ci-tool print_thread_dumps
16 changes: 5 additions & 11 deletions README.md
Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@
<img src="https://pbs.twimg.com/profile_images/545716709311520769/piLLa1iC_400x400.png" alt="logo" style="width: 32px;"/>

[![Coverage Status](https://coveralls.io/repos/github/apache/bookkeeper/badge.svg?branch=master)](https://coveralls.io/github/apache/bookkeeper?branch=master)
[![Maven Central](https://maven-badges.herokuapp.com/maven-central/org.apache.bookkeeper/bookkeeper/badge.svg)](https://maven-badges.herokuapp.com/maven-central/org.apache.bookkeeper/bookkeeper)

# Apache BookKeeper
Expand Down Expand Up @@ -29,7 +28,7 @@ Please visit the [Documentation](https://bookkeeper.apache.org/docs/latest/overv

### Report a Bug

For filing bugs, suggesting improvements, or requesting new features, help us out by [opening a Github issue](https://github.com/apache/bookkeeper/issues) or [opening an Apache jira](https://issues.apache.org/jira/browse/BOOKKEEPER).
For filing bugs, suggesting improvements, or requesting new features, help us out by [opening a Github issue](https://github.com/apache/bookkeeper/issues).

### Need Help?

Expand All @@ -45,15 +44,10 @@ We feel that a welcoming open community is important and welcome contributions.

### Contributing Code

1. See [Developer Setup](https://cwiki.apache.org/confluence/display/BOOKKEEPER/Developer+Setup) to get your local environment setup.
1. See our [installation guide](https://bookkeeper.apache.org/docs/latest/getting-started/installation/) to get your local environment setup.

2. Take a look at our open issues: [JIRA Issues](https://issues.apache.org/jira/browse/BOOKKEEPER) [Github Issues](https://github.com/apache/bookkeeper/issues).
2. Take a look at our open issues: [Github Issues](https://github.com/apache/bookkeeper/issues).

3. Review our [coding style](https://cwiki.apache.org/confluence/display/BOOKKEEPER/Coding+Guide) and follow our [pull requests](https://github.com/apache/bookkeeper/pulls) to learn about our conventions.

4. Make your changes according to our [contribution guide](https://cwiki.apache.org/confluence/display/BOOKKEEPER/Contributing+to+BookKeeper).

### Improving Website and Documentation

1. See [Building the website and documentation](https://cwiki.apache.org/confluence/display/BOOKKEEPER/Building+the+website+and+documentation) on how to build the website and documentation.
3. Review our [coding style](https://bookkeeper.apache.org/community/coding_guide/) and follow our [pull requests](https://github.com/apache/bookkeeper/pulls) to learn more about our conventions.

4. Make your changes according to our [contributing guide](https://bookkeeper.apache.org/community/contributing/)
2 changes: 1 addition & 1 deletion bin/bookkeeper
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,7 @@ where command is one of:
[service commands]

bookie Run a bookie server
autorecovery Run AutoRecovery service daemon
autorecovery Run AutoRecovery service
zookeeper Run zookeeper server

[development commands]
Expand Down
Loading