Project

Profile

Help

Issue #4084

closed

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

Added by daviddavis over 5 years ago. Updated almost 4 years ago.

Status:
CLOSED - WONTFIX
Priority:
Normal
Assignee:
-
Category:
-
Sprint/Milestone:
-
Start date:
Due date:
Estimated time:
Severity:
2. Medium
Version:
Platform Release:
OS:
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
Pulp 2
Sprint:
Quarter:

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".

Actions #2

Updated by daviddavis over 5 years 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
Actions #3

Updated by daviddavis over 5 years 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
Actions #4

Updated by daviddavis over 5 years ago

  • Description updated (diff)
Actions #5

Updated by CodeHeeler over 5 years ago

  • Triaged changed from No to Yes
Actions #6

Updated by amacdona@redhat.com over 5 years 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.

Actions #7

Updated by bmbouter about 5 years ago

  • Tags Pulp 2 added
Actions #8

Updated by ttereshc almost 4 years 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.

Also available in: Atom PDF