Project

Profile

Help

Issue #4084

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

Added by daviddavis about 1 year ago. Updated 6 months ago.

Status:
NEW
Priority:
Normal
Assignee:
-
Category:
-
Sprint/Milestone:
-
Start date:
Due date:
Severity:
2. Medium
Version:
Platform Release:
Blocks Release:
OS:
Backwards Incompatible:
No
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
Pulp 2
QA Contact:
Complexity:
Smash Test:
Verified:
No
Verification Required:
No
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 about 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 about 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 about 1 year ago

  • Description updated (diff)

#5 Updated by CodeHeeler about 1 year ago

  • Triaged changed from No to Yes

#6 Updated by amacdona@redhat.com 12 months 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 6 months ago

  • Tags Pulp 2 added

Please register to edit this issue

Also available in: Atom PDF