.buildkite/pipeline.yml

@@ -1,67 +1,115 @@
+container:
+  kubernetes: &kubernetes
+    gitEnvFrom:
+      - secretRef:
+          name: oss-github-ssh-credentials
+    sidecars:
+    - image: us-west1-docker.pkg.dev/ci-compute/buildkite-images/buildkite-dind:v1
+      volumeMounts:
+        - mountPath: /var/run/
+          name: docker-sock
+      securityContext:
+        privileged: true
+        allowPrivilegeEscalation: true
+    mirrorVolumeMounts: true # CRITICAL: this must be at the same indentation level as sidecars
+    podSpec: &podSpec
+      containers:
+        - &commandContainer
+          image: us-west1-docker.pkg.dev/ci-compute/buildkite-images/buildkite-command-container:v2
+          command:
+          - |-
+            echo "Command step was not overridden."
+            exit 1
+          volumeMounts:
+            - mountPath: /var/run/
+              name: docker-sock
+          resources:
+            requests:
+              cpu: 7500m
+              memory: 30G
+      volumes:
+      - name: docker-sock
+        emptyDir: {}
+
+agents:
+  queue: buildkite-gcp
+
 steps:
-  - label: "fossa analyze"
-    agents:
-      queue: "init"
-      docker: "*"
-    command: ".buildkite/scripts/fossa.sh"
+  - label: "Lint Check"
+    plugins:
+      - kubernetes:
+          <<: *kubernetes
+          podSpec:
+            <<: *podSpec
+            containers:
+              - <<: *commandContainer
+                command:
+                - |-
+                  .buildkite/scripts/lint.sh
+      - docker-compose#v3.0.0:
+          run: unit-test-test-service
+          config: docker/buildkite/docker-compose.yaml
+
   - label: ":java: Unit test with test services"
-    agents:
-      queue: "workers"
-      docker: "*"
-    command: "./gradlew --no-daemon test jacocoTestReport"
     artifact_paths:
-      - "build/reports/jacoco/test/*.xml"
-    timeout_in_minutes: 15
+      - "build/reports/jacoco/test/jacocoTestReport.xml"
+      - "build/reports/metadata.txt"
+    timeout_in_minutes: 30
     retry:
       automatic:
         - exit_status: "*"
           limit: 3
     plugins:
+      - kubernetes:
+          <<: *kubernetes
+          podSpec:
+            <<: *podSpec
+            containers:
+              - <<: *commandContainer
+                command:
+                - |-
+                  ./gradlew --no-daemon test jacocoTestReport
+                  .buildkite/scripts/coverage_metadata.sh build/reports/metadata.txt
       - docker-compose#v3.0.0:
           run: unit-test-test-service
           config: docker/buildkite/docker-compose.yaml
 
   - label: ":java: Unit test with docker services sticky on"
-    agents:
-      queue: "workers"
-      docker: "*"
-    command: "./gradlew --no-daemon test"
-    timeout_in_minutes: 15
+    timeout_in_minutes: 30
     retry:
       automatic:
         - exit_status: "*"
           limit: 3
     plugins:
+      - kubernetes:
+          <<: *kubernetes
+          podSpec:
+            <<: *podSpec
+            containers:
+              - <<: *commandContainer
+                command:
+                - |-
+                  ./gradlew --no-daemon test
       - docker-compose#v3.0.0:
           run: unit-test-docker-sticky-on
           config: docker/buildkite/docker-compose.yaml
 
   - label: ":java: Unit test with docker services sticky off"
-    agents:
-      queue: "workers"
-      docker: "*"
-    command: "./gradlew --no-daemon test"
-    timeout_in_minutes: 15
+    timeout_in_minutes: 30
     retry:
       automatic:
         - exit_status: "*"
           limit: 3
     plugins:
+      - kubernetes:
+          <<: *kubernetes
+          podSpec:
+            <<: *podSpec
+            containers:
+              - <<: *commandContainer
+                command:
+                - |-
+                  ./gradlew --no-daemon test
       - docker-compose#v3.0.0:
           run: unit-test-docker-sticky-off
           config: docker/buildkite/docker-compose.yaml
-  - wait
-
-  - label: ":java: Report test coverage"
-    agents:
-      queue: "workers"
-      docker: "*"
-    command: ".buildkite/scripts/coverage.sh"
-    retry:
-      automatic:
-        - exit_status: "*"
-          limit: 3
-    plugins:
-      - docker-compose#v3.0.0:
-          run: test-coverage-report
-          config: docker/buildkite/docker-compose.yaml

.buildkite/scripts/coverage_metadata.sh

@@ -0,0 +1,11 @@
+#!/bin/sh
+set -ex
+# This script generates coverage metadata for the coverage report.
+# Output is used by SonarQube integration in Uber and not used by OS repo coverage tool itself.
+# Example output:
+#   commit-sha: 6953daa563e8e44512bc349c9608484cfd4ec4ff
+#   timestamp: 2024-03-04T19:29:16Z
+output_path="$1"
+echo "commit-sha: $(git rev-parse HEAD)" > "$output_path"
+echo "timestamp: $(date -u +%Y-%m-%dT%H:%M:%SZ)" >> "$output_path"
+echo "Coverage metadata written to $output_path"

.buildkite/scripts/lint.sh

@@ -0,0 +1,12 @@
+#!/bin/sh
+
+set -ex
+
+./gradlew goJF
+
+if [ -n "$(git status --porcelain)" ]; then
+  echo "There are changes after linting (used goJF) cmd: ./gradlew goJF"
+  echo "Please rerun the command and commit the changes"
+  git status --porcelain
+  exit 1
+fi

.github/CODEOWNERS

@@ -0,0 +1,6 @@
+# This is a comment.
+# Each line is a file pattern followed by one or more owners.
+# Order is important; the last matching pattern takes the most
+# precedence.
+
+* @Groxx @abhishekj720 @natemort @shijiesheng @dkrotx @taylanisikdemir @demirkayaender

.github/pull_request_template.md

@@ -0,0 +1,21 @@
+<!-- Describe what has changed in this PR -->
+**What changed?**
+
+
+<!-- Tell your future self why have you made these changes -->
+**Why?**
+
+
+<!-- How have you verified this change? Tested locally? Added a unit test? Checked in staging env? -->
+**How did you test it?**
+
+
+<!-- Assuming the worst case, what can be broken when deploying this change to production? -->
+<!-- If you are upgrading a dependency, please mention the major version change. Read changelogs and capture any breaking changes here. -->
+**Potential risks**
+
+<!-- Is it notable for release? e.g. schema updates, configuration or data migration required? If so, please mention it, and also update CHANGELOG.md -->
+**Release notes**
+
+<!-- Is there any documentation updates should be made for config, https://cadenceworkflow.io/docs/operation-guide/setup/ ? If so, please open an PR in https://github.com/uber/cadence-docs -->
+**Documentation Changes**

.github/workflows/breaking_change_pr_template.md

@@ -0,0 +1,18 @@
+**Detailed Description**
+[In-depth description of the changes made to the interfaces, specifying new fields, removed fields, or modified data structures]
+
+**Impact Analysis**
+- **Backward Compatibility**: [Analysis of backward compatibility]
+- **Forward Compatibility**: [Analysis of forward compatibility]
+
+**Testing Plan**
+- **Unit Tests**: [Do we have unit test covering the change?]
+- **Persistence Tests**: [If the change is related to a data type which is persisted, do we have persistence tests covering the change?]
+- **Integration Tests**: [Do we have integration test covering the change?]
+- **Compatibility Tests**: [Have we done tests to test the backward and forward compatibility?]
+
+**Rollout Plan**
+- What is the rollout plan?
+- Does the order of deployment matter?
+- Is it safe to rollback? Does the order of rollback matter?
+- Is there a kill switch to mitigate the impact immediately?

.github/workflows/breaking_change_reminder.yml

@@ -0,0 +1,42 @@
+name: Workflow for Breaking Change Reminder
+on:
+  pull_request:
+    paths:
+      # below file(s) do not cover all the exposed types/funcs, but it's a good start to detect potentially breaking changes
+      - src/main/java/com/uber/cadence/client/WorkflowStub.java
+
+jobs:
+  breaking-change-pr-template-reminder:
+    runs-on: ubuntu-latest
+
+    steps:
+    - name: Checkout
+      uses: actions/checkout@v4
+      with:
+        fetch-depth: 0
+
+    - name: Fail if PR description is missing breaking change template
+      if: steps.pr-changes.outputs.changes != '[]'
+      run: |
+        PR_NUMBER=${{ github.event.pull_request.number }}
+        PR_URL="https://api.github.com/repos/${{ github.repository }}/pulls/${PR_NUMBER}"
+        BODY=$(curl $PR_URL | jq '.body')
+        CHECKLIST=(
+          "Detailed Description"
+          "Impact Analysis"
+          "Testing Plan"
+          "Rollout Plan"
+        )
+        TEMPLATE=$(cat .github/workflows/breaking_change_pr_template.md)
+
+        for i in "${CHECKLIST[@]}"; do
+            if [[ "$BODY" == *"$i"* ]]; then
+                continue
+            else
+                echo "Potential breaking changes detected! Please update the PR description to include following template:"
+                echo "---"
+                echo "$TEMPLATE"
+                echo "---"
+                exit 1
+            fi
+        done

.github/workflows/codecov.yml

@@ -0,0 +1,17 @@
+name: Workflow for Codecov
+on: [push, pull_request]
+jobs:
+  run:
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v4
+        with:
+          submodules: 'true'
+      - name: Install dependencies, run tests, and collect coverage
+        run: docker compose -f docker/buildkite/docker-compose.yaml run unit-test-test-service
+      - name: Upload coverage to Codecov
+        uses: codecov/codecov-action@v4
+        env:
+          CODECOV_TOKEN: ${{ secrets.CODECOV_TOKEN }}
+          slug: cadence-workflow/cadence-java-client

.gitigmore

@@ -0,0 +1 @@
+.dependencies

.travis.yml

@@ -7,7 +7,7 @@ env:
 
 before_install:
   - pushd /tmp
-  - wget https://www-us.apache.org/dist/thrift/0.9.3/thrift-0.9.3.tar.gz
+  - wget https://archive.apache.org/dist/thrift/0.9.3/thrift-0.9.3.tar.gz
   - tar xfz thrift-0.9.3.tar.gz
   - cd thrift-0.9.3
   - ./configure --enable-libs=no --enable-tests=no --enable-tutorial=no --with-cpp=no --with-c_glib=no --with-java=yes --with-ruby=no --with-erlang=no --with-go=no --with-nodejs=no --with-python=no