Signed-off-by: Hal Spang <halspang@microsoft.com>

…DTS emulator connectivity (#248)

* Initial plan

* Initial plan

Co-authored-by: torosent <17064840+torosent@users.noreply.github.com>

* Fix 'default messager has already been registered' warning

- Remove duplicate plugin application in samples-azure-functions/build.gradle
- Remove duplicate plugin application in endtoendtests/build.gradle
- Move 'java' plugin to plugins block
- Upgrade azure-functions plugin from 1.11.1 to 1.16.1

Co-authored-by: torosent <17064840+torosent@users.noreply.github.com>

* Revert unintended changes to orchestrator_service.proto

The proto file was accidentally modified by the downloadProtoFiles task
during the build. This reverts it to the original state.

Co-authored-by: torosent <17064840+torosent@users.noreply.github.com>

* Revert plugin version to 1.11.1 to fix Gradle 7.4 compatibility

The azure-functions plugin 1.16.1 brings in jackson-core 2.17.0 which
contains Java 21 class files that Gradle 7.4's ASM cannot handle.

Keep the duplicate plugin application removal fix which helps reduce
the "default messager" warning.

Co-authored-by: torosent <17064840+torosent@users.noreply.github.com>

* Fix SpotBugs Java version - run with JDK 11

SpotBugs 4.9.2 requires Java 11+ (class file version 55.0) but the
workflow was running it with Java 8. Updated workflow to use JDK_11
environment variable for SpotBugs tasks.

Co-authored-by: torosent <17064840+torosent@users.noreply.github.com>

* Fix SpotBugs - run build task with JDK 11 too

SpotBugs runs as part of the 'build' task, not just in the explicit
SpotBugs step. Updated the build step to also use JDK_11 since
SpotBugs 4.9.2 requires Java 11+.

Co-authored-by: torosent <17064840+torosent@users.noreply.github.com>

* Update all CI jobs to use JDK 11

- Updated Integration Tests to use JDK_11
- Updated functions-e2e-tests job to use JDK 11 instead of JDK 8
- Updated functions-sample-tests job to use JDK 11 instead of JDK 8

Regarding Gradle 8.0: Upgrading would require also updating the
protobuf plugin from 0.8.16 to 0.9.x for full compatibility, which
is a larger change that could be done in a separate PR.

Co-authored-by: torosent <17064840+torosent@users.noreply.github.com>

* Fix DTS emulator connectivity with proper health check loop

Replace fixed 10-second sleep with a robust health check that:
- Actively polls port 4001 using nc until the emulator is ready
- Allows up to 60 seconds for the emulator to start
- Provides better error diagnostics if startup fails
- Adds 5 extra seconds after port is open for gRPC initialization

Co-authored-by: torosent <17064840+torosent@users.noreply.github.com>

* Use bash /dev/tcp for more portable health check

Address code review feedback:
- Replace nc with bash's /dev/tcp for portability
- Rename max_attempts to MAX_WAIT_SECONDS with comment
- Add comment explaining the TCP check mechanism

Co-authored-by: torosent <17064840+torosent@users.noreply.github.com>

* Simplify Java setup: remove JDK 8, use only JDK 11

- Remove JDK 8 setup since it's not needed
- Remove JDK_11 environment variable workaround
- Remove all `export JAVA_HOME=$JDK_11` lines
- Upgrade setup-java from v2 to v4 in all jobs
- Workflow now uses only JDK 11 throughout

Co-authored-by: torosent <17064840+torosent@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: torosent <17064840+torosent@users.noreply.github.com>

Co-authored-by: Sophia Tevosyan <stevosyan@microsoft.com>