Skip to content

autotest: gate test_zarr_read_simple_sharding on netCDF driver#14940

Merged
rouault merged 1 commit into
OSGeo:masterfrom
flaviut:fix-zarr-sharding-test-netcdf
Jul 16, 2026
Merged

autotest: gate test_zarr_read_simple_sharding on netCDF driver#14940
rouault merged 1 commit into
OSGeo:masterfrom
flaviut:fix-zarr-sharding-test-netcdf

Conversation

@flaviut

@flaviut flaviut commented Jul 11, 2026

Copy link
Copy Markdown

What does this PR do?

Disables this test on minimal builds. open_with_cache_tile_presence_option() depends on netCDF.

What are related issues/pull requests?

NixOS/nixpkgs#540609

AI tool usage

  • AI (Y-a-t-il-un-Copilot-dans-l'avion, Chat-j'ai-pété, Jean-Claude Dusse or something similar) supported my development of this PR. See our policy about AI tool use. Use of AI tools must be indicated.

Claude Opus 4.8 used to debug the issue.

Tasklist

  • Make sure code is correctly formatted (cf pre-commit configuration)
  • Add test case(s)
  • Add documentation
  • Updated Python API documentation (swig/include/python/docs/)
  • Review
  • Adjust for comments
  • All CI builds and checks have passed
  • ADD YOUR TASKS HERE

Environment

Provide environment details, if relevant:

  • OS:
  • Compiler:

The test asserts that opening a sharded array with CACHE_TILE_PRESENCE=YES
writes the tile-presence cache sidecar (zarr.json.gmac). That cache is
serialized through the netCDF driver (GDALMDArray::GetCacheRootGroup),
so the assertion fails on builds without netCDF.

The sibling test_zarr_cache_tile_presence already carries this marker;
add it here too.
@rouault
rouault merged commit e89845b into OSGeo:master Jul 16, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants