Project

Profile

Help

Issue #4084

When Pulp checks for remote files, reword the log message to sound non-fatal

Added by daviddavis over 1 year ago. Updated about 1 month ago.

Status:
CLOSED - WONTFIX
Priority:
Normal
Assignee:
-
Category:
-
Start date:
Due date:
Severity:
2. Medium
Version:
Platform Release:
Blocks Release:
OS:
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
Pulp 2
QA Contact:
Complexity:
Smash Test:
Sprint:

Description

When downloading an rpm, pulp checks for treeinfo files and logs ominous messages that users are complaining about:

==> /var/log/messages <==
Oct 12 11:32:44 zoo pulp: nectar.downloaders.threaded:INFO: Download failed: Download of https://cdn.redhat.com/content/dist/rhel/server/7/7Server/x86_64/sat-tools/6.3/os/.treeinfo failed with code 404: Not Found
Oct 12 11:32:45 cloud-qe-17 pulp: requests.packages.urllib3.connectionpool:INFO: Starting new HTTPS connection (1): cdn.redhat.com
Oct 12 11:32:46 cloud-qe-17 pulp: nectar.downloaders.threaded:INFO: Download failed: Download of https://cdn.redhat.com/content/dist/rhel/server/7/7Server/x86_64/sat-tools/6.3/os/treeinfo failed with code 404: Not Found
Oct 12 11:32:46 cloud-qe-17 pulp: pulp_rpm.plugins.importers.yum.sync:INFO: [f5963541] Sync complete.

Either don't log these messages, log them at a different level (e.g. DEBUG), or reword it to hide "404" and "Download failed".

History

#2 Updated by daviddavis over 1 year ago

  • Project changed from RPM Support to Pulp
  • Description updated (diff)

So this issue is not limited to the RPM plugin. Puppet also does something similar where it checks for a PULP_MANIFEST file and then logs:

Oct 12 16:51:36 pulp2.dev pulp[2291]: nectar.downloaders.threaded:INFO: Download failed: Download of http://forge.puppetlabs.com/PULP_MANIFEST failed with code 404: Not Found

#3 Updated by daviddavis over 1 year ago

  • Subject changed from Reword the log message to sound non-fatal - Fetching treeinfo and .treeinfo for non kickstart repos showing 404 to When Pulp checks for remote files, reword the log message to sound non-fatal

#4 Updated by daviddavis over 1 year ago

  • Description updated (diff)

#5 Updated by CodeHeeler over 1 year ago

  • Triaged changed from No to Yes

#6 Updated by amacdona@redhat.com over 1 year ago

I think it makes sense to keep this at the INFO level, but change the wording to not sound fatal. The reason behind INFO is that it seems reasonable that the user should know that things didn't download. If we move to DEBUG, the task will succeed and the user would have no reason to check DEBUG.

#7 Updated by bmbouter about 1 year ago

  • Tags Pulp 2 added

#8 Updated by ttereshc about 1 month ago

  • Status changed from NEW to CLOSED - WONTFIX

Pulp 2 is in maintenance mode and currently accepts only critical/security issues. The main focus is on Pulp 3 and some of the requests will be satisfied in the newer version. We have evaluated this request, and while we recognize that it is a valid request, we do not expect this to be implemented in Pulp 2. As this issue is not relevant for Pulp 3, we are therefore closing this out as WONTFIX.

Please register to edit this issue

Also available in: Atom PDF