Issue 1626: Add tests for LedgerUnderredplicationManager#markLedgerUnderreplicatedAsync

.asf.yaml

@@ -0,0 +1,88 @@
+#
+# 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.
+#
+
+github:
+  description: "Apache BookKeeper - a scalable, fault tolerant and low latency storage service optimized for append-only workloads"
+  homepage: https://bookkeeper.apache.org/
+  labels:
+    - apache
+    - bookkeeper
+    - big-data
+    - distributed-log
+    - distributed-systems
+    - wal
+  features:
+    # Enable wiki for documentation
+    wiki: false
+    # Enable issues management
+    issues: true
+    # Enable projects for project management boards
+    projects: true
+  enabled_merge_buttons:
+    # enable squash button:
+    squash:  true
+    # disable merge button:
+    merge:   false
+    # disable rebase button:
+    rebase:  false
+  protected_branches:
+    master:
+      required_status_checks:
+        # strict means "Require branches to be up to date before merging".
+        strict: false
+        # Contexts are the names of checks that must pass.
+        # See ./github/workflows/README.md for more documentation on this list.
+        contexts:
+          - BookKeeper CI checks completed
+
+      required_pull_request_reviews:
+        dismiss_stale_reviews: false
+        require_code_owner_reviews: true
+        required_approving_review_count: 1
+
+      # squash or rebase must be allowed in the repo for this setting to be set to true.
+      required_linear_history: true
+
+      required_signatures: false
+
+    # The following branch protections only ensure that force pushes are not allowed
+    branch-4.0: {}
+    branch-4.1: {}
+    branch-4.2: {}
+    branch-4.3: {}
+    branch-4.4: {}
+    branch-4.5: {}
+    branch-4.6: {}
+    branch-4.7: {}
+    branch-4.8: {}
+    branch-4.9: {}
+    branch-4.10: {}
+    branch-4.11: {}
+    branch-4.12: {}
+    branch-4.13: {}
+    branch-4.14: {}
+    branch-4.15: {}
+    branch-4.16: {}
+
+notifications:
+  commits:      [email protected]
+  issues:       [email protected]
+  pullrequests: [email protected]
+  discussions:  [email protected]
+  jira_options: link label

.dlc.json

@@ -0,0 +1,89 @@
+{
+  "ignorePatterns": [
+    {
+      "pattern": "^http://localhost"
+    },
+    {
+      "pattern": "^https://kubernetes.io/docs/resources-reference/v1.6/"
+    },
+    {
+      "pattern": "^https://zookeeper.apache.org/doc/"
+    },
+    {
+      "pattern": "^http://zookeeper.apache.org/doc/current/api/"
+    },
+    {
+      "pattern": "^https://github.com/apache/bookkeeper/"
+    },
+    {
+      "pattern": "^https://github.com/sijie/bookkeeper/tree/bookie_shell_refactor"
+    },
+    {
+      "pattern": "^http://pulsar.incubator.apache.org/docs/latest/getting-started/ConceptsAndArchitecture/#persistent-storage"
+    },
+    {
+      "pattern": "^https://zookeeper.apache.org/doc/current/api/org/apache/zookeeper/"
+    },
+    {
+      "pattern": "^http://pulsar.incubator.apache.org/docs/latest/getting-started/ConceptsAndArchitecture/#persistent-storage"
+    },
+    {
+      "pattern": "^https://bookkeeper.apache.org/docs/api/javadoc/"
+    },
+    {
+      "pattern": "^http://hbtc2012.hadooper.cn/subject/track1maheswara2.pdf"
+    },
+    {
+      "pattern": "^https://developer.yahoo.com/blogs/ydn/bookkeeper-durability-scale-54048.html"
+    },
+    {
+      "pattern": "^http://apache.claz.org/bookkeeper"
+    },
+    {
+      "pattern": "^https://dcos.io/"
+    },
+    {
+      "pattern": "^ https://docs.mesosphere.com"
+    },
+    {
+      "pattern": "^http://master.dcos/exhibitor"
+    },
+    {
+      "pattern": "^https://docs.mesosphere.com/"
+    },
+    {
+      "pattern": "^https://distributedlog.io"
+    },
+    {
+      "pattern": "^https://www.opengroup.org/membership/forums/platform/unix"
+    },
+    {
+      "pattern": "^https://docs.oracle.com/javase/8/docs/technotes/guides/security/jgss/tutorials/KerberosReq.html"
+    },
+    {
+      "pattern": "^http://daxue.qq.com/content/content/id/2492"
+    },
+    {
+      "pattern": "^https://calendar.google.com/"
+    },
+    {
+      "pattern": "^#"
+    },
+    {
+      "pattern": ".*\\{\\{.*"
+    },
+    {
+      "pattern": "^//"
+    }
+  ],
+  "timeout": "10s",
+  "retryOn429": true,
+  "retryCount": 10,
+  "fallbackRetryDelay": "1000s",
+  "aliveStatusCodes": [
+    200,
+    400,
+    401,
+    403
+  ]
+}

.github/ISSUE_TEMPLATE.md

@@ -2,7 +2,7 @@ Is this a question, feature request, bug report, or bookkeeper proposal?
 
 **QUESTION**
 
-Have you checked our documentation at http://bookkeeper.apache.org/ , If you could not find an answer there, please consider asking your question in our community forum at [email protected], as it would benefit other members of our community.
+Have you checked our documentation at https://bookkeeper.apache.org/ , If you could not find an answer there, please consider asking your question in our community forum at [email protected], as it would benefit other members of our community.
 
 
 **FEATURE REQUEST**

.github/ISSUE_TEMPLATE/bp.md

@@ -0,0 +1,19 @@
+---
+name: BookKeeper Proposal
+about: Propose a major change to BookKeeper
+title: ''
+labels: type/proposal
+assignees: ''
+
+---
+
+**BP**
+
+> Follow the instructions at https://bookkeeper.apache.org/community/bookkeeper-proposals/ to create a proposal.
+
+This is the master ticket for tracking BP-xyz :
+
+[a short description for this BP]
+
+<!-- add a proposal PR link below -->
+Proposal PR - #abc

.github/ISSUE_TEMPLATE/bug_report.md

@@ -0,0 +1,34 @@
+---
+name: Bug report
+about: Create a bug report to help us improve
+title: ''
+labels: type/bug
+assignees: ''
+
+---
+
+**BUG REPORT**
+
+***Describe the bug***
+
+A clear and concise description of what the bug is.
+
+***To Reproduce***
+
+Steps to reproduce the behavior:
+1. Go to '...'
+2. Click on '....'
+3. Scroll down to '....'
+4. See error
+
+***Expected behavior***
+
+A clear and concise description of what you expected to happen.
+
+***Screenshots***
+
+If applicable, add screenshots to help explain your problem.
+
+***Additional context***
+
+Add any other context about the problem here.

.github/ISSUE_TEMPLATE/feature_request.md

@@ -0,0 +1,18 @@
+---
+name: Feature request
+about: Suggest an idea for this project
+title: ''
+labels: type/feature
+assignees: ''
+
+---
+
+**FEATURE REQUEST**
+
+1. Please describe the feature you are requesting.
+
+2. Indicate the importance of this issue to you (blocker, must-have, should-have, nice-to-have).
+   Are you currently using any workarounds to address this issue?
+
+3. Provide any additional detail on your proposed use case for this feature.
+

.github/ISSUE_TEMPLATE/flaky_test.md

@@ -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>

.github/ISSUE_TEMPLATE/question.md

@@ -0,0 +1,14 @@
+---
+name: Question
+about: Ask the community for answering questions of BookKeeper
+labels: type/question
+assignees: ''
+
+---
+
+**QUESTION**
+
+Have you checked our documentation at https://bookkeeper.apache.org/?
+If you could not find an answer there, please consider asking your question
+in our community mailing list at [email protected], as it would benefit
+other members of our community.

.github/PULL_REQUEST_TEMPLATE.md

@@ -1,6 +1,16 @@
 Descriptions of the changes in this PR:
 
+<!-- Either this PR fixes an issue, -->
 
+Fix #xyz
+
+<!-- or this PR is one task of an issue -->
+
+Main Issue: #xyz
+
+<!-- If the PR belongs to a BP, please add the BP link here -->
+
+BP: #xyz
 
 ### Motivation
 
@@ -10,27 +20,12 @@ Descriptions of the changes in this PR:
 
 (Describe: what changes you have made)
 
-Master Issue: #<master-issue-number>
-
 > ---
 > In order to uphold a high standard for quality for code contributions, Apache BookKeeper runs various precommit
-> checks for pull requests. A pull request can only be merged when it passes precommit checks. However running all
-> the precommit checks can take a long time, some trivial changes don't need to run all the precommit checks. You
-> can check following list to skip the tests that don't need to run for your pull request. Leave them unchecked if
-> you are not sure, committers will help you:
+> checks for pull requests. A pull request can only be merged when it passes precommit checks.
 >
-> - [ ] [skip bookkeeper-server bookie tests]: skip testing `org.apache.bookkeeper.bookie` in bookkeeper-server module.
-> - [ ] [skip bookkeeper-server client tests]: skip testing `org.apache.bookkeeper.client` in bookkeeper-server module.
-> - [ ] [skip bookkeeper-server replication tests]: skip testing `org.apache.bookkeeper.replication` in bookkeeper-server module.
-> - [ ] [skip bookkeeper-server tls tests]: skip testing `org.apache.bookkeeper.tls` in bookkeeper-server module.
-> - [ ] [skip bookkeeper-server remaining tests]: skip testing all other tests in bookkeeper-server module.
-> - [ ] [skip integration tests]: skip docker based integration tests. if you make java code changes, you shouldn't skip integration tests.
-> - [ ] [skip build java8]: skip build on java8. *ONLY* skip this when *ONLY* changing files under documentation under `site`.
-> - [ ] [skip build java9]: skip build on java9. *ONLY* skip this when *ONLY* changing files under documentation under `site`.
-> ---
-
 > ---
-> Be sure to do all of the following to help us incorporate your contribution
+> Be sure to do all the following to help us incorporate your contribution
 > quickly and easily:
 >
 > If this PR is a BookKeeper Proposal (BP):

.github/actions/bot/package.json

@@ -0,0 +1,31 @@
+{
+  "name": "github-action-bot",
+  "description": "Bot for github actions",
+  "version": "1.1.1",
+  "author": "GitHub",
+  "license": "Apache LICENSE 2.0",
+  "main": "dist/index.js",
+  "private": true,
+  "scripts": {
+    "build": "ncc build src/run.js"
+  },
+  "dependencies": {
+    "@actions/core": "^1.2.4",
+    "@actions/github": "^2.2.0",
+    "@actions/io": "^1.0.2"
+  },
+  "devDependencies": {
+    "@types/jest": "^25.1.4",
+    "@typescript-eslint/eslint-plugin": "^2.33.0",
+    "@typescript-eslint/parser": "^2.33.0",
+    "@zeit/ncc": "^0.22.0",
+    "eslint": "^7.0.0",
+    "eslint-config-prettier": "^6.11.0",
+    "husky": "^4.2.5",
+    "jest": "^25.1.0",
+    "npm-run-all": "^4.1.5",
+    "prettier": "^2.0.5",
+    "ts-jest": "^25.2.1",
+    "typescript": "^3.8.3"
+  }
+}