ZARR: emit an error when reading from a JSON/Kerchunk reference store and one of the pointed file cannot be opened

Fixes #13126

Author: rouault

autotest/gdrivers/data/zarr/kerchunk_parquet/parquet_ref_2_dim/ar/4.bin

@@ -245,16 +245,14 @@ def test_zarr_kerchunk_json_fail_exception(
         """{".zgroup": {"zarr_format":2}, "foo":["https://localhost:1/",0,1]}""",
     ],
 )
-def test_zarr_kerchunk_json_open_fail_no_exception(tmp_vsimem, content):
+def test_zarr_kerchunk_json_open_fail(tmp_vsimem, content):
 
     json_filename = str(tmp_vsimem / "test.json")
     gdal.FileFromMemBuffer(json_filename, content)
 
-    with gdal.quiet_errors():
-        assert (
-            gdal.VSIFOpenL("/vsikerchunk_json_ref/{" + json_filename + "}/foo", "rb")
-            is None
-        )
+    with pytest.raises(Exception):
+        with gdal.VSIFile("/vsikerchunk_json_ref/{" + json_filename + "}/foo", "rb"):
+            pass
 
 
 ###############################################################################

autotest/gdrivers/zarr_kerchunk_json.py

@@ -11,6 +11,9 @@
 # SPDX-License-Identifier: MIT
 ###############################################################################
 
+import os
+import shutil
+
 import pytest
 
 from osgeo import gdal
@@ -318,6 +321,26 @@ def test_zarr_kerchunk_parquet_gdal_open_dim2():
 ###############################################################################
 
 
+def test_zarr_kerchunk_parquet_gdal_open_missing_bin_file(tmp_path):
+
+    shutil.copytree(
+        "data/zarr/kerchunk_parquet/parquet_ref_2_dim", tmp_path / "parquet_ref_2_dim"
+    )
+    os.unlink(tmp_path / "parquet_ref_2_dim/ar/4.bin")
+
+    with gdal.OpenEx(
+        f"ZARR:/vsikerchunk_parquet_ref/{{{tmp_path}/parquet_ref_2_dim}}",
+        gdal.OF_MULTIDIM_RASTER,
+    ) as ds:
+        rg = ds.GetRootGroup()
+        ar = rg.OpenMDArray("ar")
+        with pytest.raises(Exception):
+            ar.Read()
+
+
+###############################################################################
+
+
 @pytest.mark.require_driver("PARQUET")
 def test_zarr_kerchunk_parquet_invalid_parquet_struct():
     with gdal.OpenEx(

autotest/gdrivers/zarr_kerchunk_parquet.py

@@ -21,6 +21,7 @@
 #include "cpl_json_streaming_parser.h"
 #include "cpl_json_streaming_writer.h"
 #include "cpl_mem_cache.h"
+#include "cpl_vsi_error.h"
 #include "cpl_vsi_virtual.h"
 
 #include "gdal_priv.h"
@@ -1575,7 +1576,14 @@ VSIKerchunkJSONRefFileSystem::Open(const char *pszFilename,
                      osPath.c_str());
         CPLConfigOptionSetter oSetter("GDAL_DISABLE_READDIR_ON_OPEN",
                                       "EMPTY_DIR", false);
-        return VSIFOpenL(osPath.c_str(), "rb");
+        auto fp = VSIFOpenEx2L(osPath.c_str(), "rb", true, nullptr);
+        if (!fp)
+        {
+            if (!VSIToCPLError(CE_Failure, CPLE_FileIO))
+                CPLError(CE_Failure, CPLE_FileIO, "Cannot open %s",
+                         osPath.c_str());
+        }
+        return fp;
     }
 }
 

frmts/zarr/vsikerchunk_json_ref.cpp

@@ -15,6 +15,7 @@
 
 #include "cpl_json.h"
 #include "cpl_mem_cache.h"
+#include "cpl_vsi_error.h"
 #include "cpl_vsi_virtual.h"
 
 #include "gdal_priv.h"
@@ -550,7 +551,14 @@ VSIVirtualHandle *VSIKerchunkParquetRefFileSystem::Open(
                              osPath.c_str());
                 CPLConfigOptionSetter oSetter("GDAL_DISABLE_READDIR_ON_OPEN",
                                               "EMPTY_DIR", false);
-                return VSIFOpenL(osPath.c_str(), "rb");
+                auto fp = VSIFOpenEx2L(osPath.c_str(), "rb", true, nullptr);
+                if (!fp)
+                {
+                    if (!VSIToCPLError(CE_Failure, CPLE_FileIO))
+                        CPLError(CE_Failure, CPLE_FileIO, "Cannot open %s",
+                                 osPath.c_str());
+                }
+                return fp;
             }
         }
 

frmts/zarr/vsikerchunk_parquet_ref.cpp

@@ -338,10 +338,10 @@ int VSISubFileFilesystemHandler::DecomposePath(const char *pszPath,
 /*                                Open()                                */
 /************************************************************************/
 
-VSIVirtualHandle *
-VSISubFileFilesystemHandler::Open(const char *pszFilename,
-                                  const char *pszAccess, bool /* bSetError */,
-                                  CSLConstList /* papszOptions */)
+VSIVirtualHandle *VSISubFileFilesystemHandler::Open(const char *pszFilename,
+                                                    const char *pszAccess,
+                                                    bool bSetError,
+                                                    CSLConstList papszOptions)
 
 {
     if (!STARTS_WITH_CI(pszFilename, "/vsisubfile/"))
@@ -371,7 +371,8 @@ VSISubFileFilesystemHandler::Open(const char *pszFilename,
     /* -------------------------------------------------------------------- */
     /*      Open the underlying file.                                       */
     /* -------------------------------------------------------------------- */
-    VSILFILE *fp = VSIFOpenL(osSubFilePath, pszAccess);
+    VSILFILE *fp =
+        VSIFOpenEx2L(osSubFilePath, pszAccess, bSetError, papszOptions);
 
     if (fp == nullptr)
         return nullptr;