Skip to content

Fix duplicate plugin application, simplify CI to use JDK 11, and fix DTS emulator connectivity #248

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
jviau merged 11 commits into from
Dec 10, 2025
Merged

Fix duplicate plugin application, simplify CI to use JDK 11, and fix DTS emulator connectivity #248

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
3fbb68e
Initial plan
Copilot Dec 10, 2025
41c1deb
Initial plan
Copilot Dec 10, 2025
090c941
Fix 'default messager has already been registered' warning
Copilot Dec 10, 2025
3e26e32
Revert unintended changes to orchestrator_service.proto
Copilot Dec 10, 2025
5472f1b
Revert plugin version to 1.11.1 to fix Gradle 7.4 compatibility
Copilot Dec 10, 2025
a6d684f
Fix SpotBugs Java version - run with JDK 11
Copilot Dec 10, 2025
2cae306
Fix SpotBugs - run build task with JDK 11 too
Copilot Dec 10, 2025
5080337
Update all CI jobs to use JDK 11
Copilot Dec 10, 2025
19df1cd
Fix DTS emulator connectivity with proper health check loop
Copilot Dec 10, 2025
570d630
Use bash /dev/tcp for more portable health check
Copilot Dec 10, 2025
45316b1
Simplify Java setup: remove JDK 8, use only JDK 11
Copilot Dec 10, 2025
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
61 changes: 34 additions & 27 deletions .github/workflows/build-validation.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -22,22 +22,11 @@ jobs:
- uses: actions/checkout@v2

- name: Set up JDK 11
uses: actions/setup-java@v2
uses: actions/setup-java@v4
with:
java-version: '11'
distribution: 'microsoft'

- name: set JDK_11 environment variable test compiling and running
env:
ACTIONS_ALLOW_UNSECURE_COMMANDS: true
run: echo ::set-env name=JDK_11::$(echo $JAVA_HOME)

- name: Set up JDK 8
uses: actions/setup-java@v2
with:
java-version: '8'
distribution: 'temurin'

- name: Setup Gradle
uses: gradle/gradle-build-action@v2

Expand All @@ -56,9 +45,7 @@ jobs:
if-no-files-found: ignore

- name: Run Unit Tests with Gradle
run: |
export JAVA_HOME=$JDK_11
./gradlew clean test || echo "UNIT_TEST_FAILED=true" >> $GITHUB_ENV
run: ./gradlew clean test || echo "UNIT_TEST_FAILED=true" >> $GITHUB_ENV
continue-on-error: true

- name: Upload test reports if tests failed
Expand All @@ -77,11 +64,31 @@ jobs:
run: docker run --name durabletask-emulator -p 4001:8080 -d mcr.microsoft.com/dts/dts-emulator:latest

- name: Display Durable Task Emulator Logs
run: nohup docker logs --since=0 durabletask-emulator > durabletask-emulator.log 2>&1 &
run: nohup docker logs -f durabletask-emulator > durabletask-emulator.log 2>&1 &

# wait for 10 seconds, so sidecar container can be fully up, this will avoid intermittent failing issues for integration tests causing by failed to connect to sidecar
- name: Wait for 10 seconds
run: sleep 10
- name: Wait for Durable Task Emulator to be ready
run: |
echo "Waiting for Durable Task Emulator to be ready on port 4001..."
# Maximum wait time of 60 seconds for emulator startup
MAX_WAIT_SECONDS=60
attempt=0
# Use bash's /dev/tcp for portability instead of nc
while ! (echo > /dev/tcp/localhost/4001) 2>/dev/null; do
attempt=$((attempt + 1))
if [ $attempt -ge $MAX_WAIT_SECONDS ]; then
echo "ERROR: Emulator not ready after $MAX_WAIT_SECONDS seconds"
echo "Docker container status:"
docker ps -a
echo "Container logs:"
docker logs durabletask-emulator 2>&1 | tail -50
exit 1
fi
echo "Attempt $attempt/$MAX_WAIT_SECONDS - waiting for emulator..."
sleep 1
done
echo "Durable Task Emulator is ready on port 4001"
# Give additional time for gRPC service to fully initialize after port is open
sleep 5

- name: Integration Tests with Gradle
run: ./gradlew integrationTest || echo "TEST_FAILED=true" >> $GITHUB_ENV
Expand Down Expand Up @@ -120,11 +127,11 @@ jobs:
steps:
- uses: actions/checkout@v2

- name: Set up JDK 8
uses: actions/setup-java@v2
- name: Set up JDK 11
uses: actions/setup-java@v4
with:
java-version: '8'
distribution: 'temurin'
java-version: '11'
distribution: 'microsoft'

- name: Setup Gradle
uses: gradle/gradle-build-action@v2
Expand Down Expand Up @@ -159,11 +166,11 @@ jobs:
steps:
- uses: actions/checkout@v2

- name: Set up JDK 8
uses: actions/setup-java@v2
- name: Set up JDK 11
uses: actions/setup-java@v4
with:
java-version: '8'
distribution: 'temurin'
java-version: '11'
distribution: 'microsoft'

- name: Setup Gradle
uses: gradle/gradle-build-action@v2
Expand Down
3 changes: 1 addition & 2 deletions endtoendtests/build.gradle
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,8 +1,7 @@
plugins {
id 'java'
id "com.microsoft.azure.azurefunctions" version "1.11.1"
}
apply plugin: 'java'
apply plugin: "com.microsoft.azure.azurefunctions"

group 'com.durabletask.endtoend'
version '0.0.0-SNAPSHOT'
Expand Down
3 changes: 1 addition & 2 deletions samples-azure-functions/build.gradle
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,8 +1,7 @@
plugins {
id 'java'
id "com.microsoft.azure.azurefunctions" version "1.11.1"
}
apply plugin: 'java'
apply plugin: "com.microsoft.azure.azurefunctions"

group 'com.functions'
version '0.1.0-SNAPSHOT'
Expand Down
Loading