Optimize build by refactoring fragile tests and stripping heavy dependencies

[CydeWeys]

This commit dramatically optimizes the local Gradle build time, shaving over 5 minutes off a full build execution:

• Instrumented the build to identify fragileTest taking > 3 minutes.
• Refactored TestServer.java to dynamically bind to ephemeral port 0, resolving race conditions.
• Updated UploadBsaUnavailableDomainsActionTest to use the thread-safe TestServer, allowing it to run in parallel.
• Removed outdated exclusions for HostInfoFlowTest and RegistryPipelineWorkerInitializerTest.
• Moved these tests to the highly parallelized standardTest suite.
• Removed the redundant sqlIntegrationTest execution from the standard test phase.
• Stripped heavy Docker (buildNomulusImage) and 5x frontend (buildConsoleForAll) staging dependencies from the standard build task, ensuring they are only run when explicitly deployed.

---

This change is 

[CydeWeys]

Ignore this file, I'm removing it from the commit imminently.

[CydeWeys]

I am curious if this part is always safe:

"Stripped heavy Docker (buildNomulusImage) and 5x frontend (buildConsoleForAll) staging dependencies from the standard build task, ensuring they are only run when explicitly deployed."

[weiminyu]

@weiminyu reviewed 8 files and all commit messages, and made 1 comment.
Reviewable status: all files reviewed, 3 unresolved discussions (waiting on CydeWeys).

---

core/src/test/java/google/registry/server/TestServer.java line 171 at r2 (raw file):

        port = urlAddress.getPortOrDefault(DEFAULT_PORT);
      }
      return new URL(String.format("http://%s:%d%s", urlAddress.getHost(), port, path));

new approach is new URI(...).toURL()

Code quote:

new URL

[CydeWeys]

@CydeWeys Yes, it is safe! The heavy Docker and 5x frontend staging dependencies are only required when actually packaging the artifact for deployment or running full E2E staging tests. They are not required for local compilation or running unit/integration tests, which is what developers use the default ./gradlew build for. The explicit deployment commands (like ./gradlew :stage) still explicitly depend on these tasks, so CI and deployments won't break.

@weiminyu I have updated TestServer.java to use the new URI(...).toURL() approach as requested and amended the commit!

[weiminyu]

core/build.gradle line 762 at r3 (raw file):

project.build.dependsOn devtool
project.build.dependsOn buildToolImage

If this is removed here, it won't be invoked until build time.

Maybe make Kokoro build this explicitly?

Code quote:

buildToolImage

[CydeWeys]

@CydeWeys made 1 comment.
Reviewable status: 6 of 8 files reviewed, 3 unresolved discussions (waiting on weiminyu).

---

core/build.gradle line 762 at r3 (raw file):

Previously, weiminyu (Weimin Yu) wrote…

If this is removed here, it won't be invoked until build time.

Maybe make Kokoro build this explicitly?

What's the issue we're trying to solve for here -- a PR contains changes to the tool, and we want Kokoro to build the tool with those changes included prior to calling it?

(But this isn't an issue for local builds?)

[CydeWeys]

@CydeWeys made 1 comment.
Reviewable status: 6 of 9 files reviewed, 3 unresolved discussions (waiting on weiminyu).

---

core/build.gradle line 762 at r3 (raw file):

Previously, CydeWeys (Ben McIlwain) wrote…

What's the issue we're trying to solve for here -- a PR contains changes to the tool, and we want Kokoro to build the tool with those changes included prior to calling it?

(But this isn't an issue for local builds?)

PTAL, now with a buildAll target that I will configure Kokoro to use once this is merged.

[CydeWeys]

@CydeWeys made 1 comment.
Reviewable status: 6 of 9 files reviewed, 3 unresolved discussions (waiting on weiminyu).

---

core/src/test/java/google/registry/server/TestServer.java line 171 at r2 (raw file):

Previously, weiminyu (Weimin Yu) wrote…

new approach is new URI(...).toURL()

Done.

[CydeWeys]

@CydeWeys resolved 1 discussion.
Reviewable status: 6 of 9 files reviewed, 2 unresolved discussions (waiting on weiminyu).

[weiminyu]

@weiminyu reviewed 4 files and all commit messages, and resolved 1 discussion.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on CydeWeys).

[weiminyu]

@weiminyu resolved 1 discussion.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on CydeWeys).

[CydeWeys]

PTAL. Didn't make further changes, but did have to rebase a bunch of commits from master.