Suppress download errors for optional (likely to fail) downloads

backports: #9135 https://pulp.plan.io/issues/9135 fixes #9318 (cherry picked from commit 0c560f2)
pulp · Aug 31, 2021 · f8fd779 · f8fd779
1 parent 6e510e1
commit f8fd779
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/CHANGES/9318.misc b/CHANGES/9318.misc
@@ -0,0 +1,2 @@
+Silenced some unnecessary warnings which occurred when downloading optional files was unsuccessfully attempted.
+(backported from #9135)
diff --git a/pulp_rpm/app/tasks/synchronizing.py b/pulp_rpm/app/tasks/synchronizing.py
@@ -677,7 +677,8 @@ async def run_repomdrecord_download(name, location_href, downloader):
                     for file_href in ["repodata/repomd.xml.asc", "repodata/repomd.xml.key"]:
                         try:
                             downloader = self.remote.get_downloader(
-                                url=urlpath_sanitize(self.remote_url, file_href)
+                                url=urlpath_sanitize(self.remote_url, file_href),
+                                silence_errors_for_response_status_codes={403, 404},
                             )
                             result = await downloader.run()
                             store_metadata_for_mirroring(self.repository, result.path, file_href)
@@ -688,7 +689,8 @@ async def run_repomdrecord_download(name, location_href, downloader):
                     # extra files to copy, e.g. EULA, LICENSE
                     try:
                         downloader = self.remote.get_downloader(
-                            url=urlpath_sanitize(self.remote_url, "extra_files.json")
+                            url=urlpath_sanitize(self.remote_url, "extra_files.json"),
+                            silence_errors_for_response_status_codes={403, 404},
                         )
                         result = await downloader.run()
                         store_metadata_for_mirroring(