diff mbox series

[3/6] fetch2: Drop ESTALE error handling

Message ID 20260813124834.1770307-3-richard.purdie@linuxfoundation.org
State New
Headers show
Series [1/6] fetch/__init__: Clean up minor mirror code duplication | expand

Commit Message

Richard Purdie Aug. 13, 2026, 12:48 p.m. UTC 
This is old code and nobody I've talked to using NFS has seen this. Translating
this into a checksum error probably isn't correct, even if we did want to silently
handle it.

I propose we remove this code and see if anyone des really need something like
this, if so, writing it correctly without the ChecksumError usage.

Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
---
 lib/bb/fetch2/__init__.py | 12 ------------
 1 file changed, 12 deletions(-)
diff mbox series

Patch

diff --git a/lib/bb/fetch2/__init__.py b/lib/bb/fetch2/__init__.py
index 51db8304c65..2676e0962b9 100644
--- a/lib/bb/fetch2/__init__.py
+++ b/lib/bb/fetch2/__init__.py
@@ -1156,12 +1156,6 @@  def try_mirror_url(fetch, origud, ud, ld, check = False):
     except bb.fetch2.NetworkAccess:
         raise
 
-    except IOError as e:
-        if e.errno in [errno.ESTALE]:
-            logger.warning("Stale Error Observed %s." % ud.url)
-            return False
-        raise
-
     except bb.fetch2.BBFetchException as e:
         if isinstance(e, ChecksumError):
             logger.warning("Mirror checksum failure for url %s (original url: %s)\nCleaning and trying again." % (ud.url, origud.url))
@@ -1976,12 +1970,6 @@  class Fetch(object):
 
                 m.update_donestamp(ud, d)
 
-            except IOError as e:
-                if e.errno in [errno.ESTALE]:
-                    logger.error("Stale Error Observed %s." % u)
-                    raise ChecksumError("Stale Error Detected")
-                raise
-
             except BBFetchException as e:
                 if isinstance(e, NoChecksumError):
                     (message, _) = e.args