Suppress only builtins.FileNotFoundError in obstore.delete adapter

Author: lukasbindreiter

src/zarr/storage/_obstore.py

@@ -186,16 +186,14 @@ def supports_deletes(self) -> bool:
     async def delete(self, key: str) -> None:
         # docstring inherited
         import obstore as obs
-        from obstore.exceptions import NotFoundError
 
         self._check_writable()
 
-        # Some stores such as local filesystems, GCP and Azure raise an error
+        # Some obstore stores such as local filesystems, GCP and Azure raise an error
         # when deleting a non-existent key, while others such as S3 and in-memory do
-        # not. We suppress the error to make the behavior consistent across all stores
-        # currently obstore raises FileNotFoundError, but in the future might raise
-        # NotFoundError instead, so let's suppress that too
-        with contextlib.suppress(FileNotFoundError, NotFoundError):
+        # not. We suppress the error to make the behavior consistent across all obstore
+        # stores. This is also in line with the behavior of the other Zarr store adapters.
+        with contextlib.suppress(FileNotFoundError):
             await obs.delete_async(self.store, key)
 
     @property