shangxinli opened a new pull request, #18772:
URL: https://github.com/apache/hudi/pull/18772

   ### Describe the issue this Pull Request addresses
   
   `ITAzureStorageLockClientAzurite` fails the entire `integration-tests` CI 
job when Microsoft Container Registry (MCR) blocks the azurite image pull. The 
HTML body returned by MCR makes the cause explicit:
   
   ```
   pull access denied for mcr.microsoft.com/azure-storage/azurite,
   repository does not exist or may require 'docker login': denied
   ...
   <title>Microsoft</title>
   <h2>The request is blocked.</h2>
   ```
   
   This is Azure Front Door's WAF returning 403 — MCR rate-limit or geo-block 
from the GitHub Actions runner IP. Outside the project's control; should not 
gate the build.
   
   Observed on #18765.
   
   ### Summary and Changelog
   
   - Pinned the image to `mcr.microsoft.com/azure-storage/azurite:3.34.0` 
instead of the implicit `:latest`. Tagged manifests are CDN-cached better than 
`:latest` and queried less aggressively, reducing the chance of tripping MCR's 
WAF.
   - Replaced the `@Testcontainers` / `@Container` declarative lifecycle with 
manual `@BeforeAll` / `@AfterAll`. Catch `ContainerFetchException`, 
`ContainerLaunchException`, and infrastructure-flavored `RuntimeException` from 
`container.start()` and translate them to `Assumptions.abort(...)` so the suite 
is reported as **skipped** rather than errored when the image cannot be pulled 
or the container cannot be started.
   - Preserved the original "Docker not installed" behavior: re-throw any 
`RuntimeException` whose message clearly indicates Docker itself is missing, so 
local developers still get a clear diagnostic.
   
   ### Impact
   
   Test-only. No production-code changes. Tests still run normally when MCR is 
reachable; only the failure mode is changed (skip vs error).
   
   ### Risk Level
   
   **low** — single test class; falls back to skip instead of error only when 
external infrastructure is unavailable.
   
   Verified: `mvn -pl hudi-azure -am test-compile` BUILD SUCCESS; checkstyle 0 
violations; apache-rat 0 unapproved.
   
   ### Documentation Update
   
   None.
   
   ### Contributor's checklist
   
   - [x] Read through [contributor's 
guide](https://hudi.apache.org/contribute/how-to-contribute)
   - [x] Enough context is provided in the sections above
   - [x] Adequate tests were added if applicable
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to