Issue #1248
Base FileDistributor does not log errors, it only puts information about an error into the report for the user.
Description
While working with the Puppet subclass of the platform's FileDistributor implementation I encountered an Exception, but the actual traceback of the exception wasn't in the logs. There was a small exception where the task logs that the publish experienced some kind of error, but the actual error traceback is not there. This is where I expected the error to be logged, but it was not.
I expected that the actual traceback would also be present in the logs, but all I saw was this:
pulp[10842]: pulp.server.controllers.repository:ERROR: (10842-98784) Exception caught from plugin during publish for repo [zoo-puppet]
pulp[10842]: pulp.server.controllers.repository:ERROR: (10842-98784) Traceback (most recent call last):
pulp[10842]: pulp.server.controllers.repository:ERROR: (10842-98784) File "/home/bmbouter/Documents/pulp/server/pulp/server/controllers/repository.py", line 784, in _do_publish
pulp[10842]: pulp.server.controllers.repository:ERROR: (10842-98784) distributor_id=dist_id
pulp[10842]: pulp.server.controllers.repository:ERROR: (10842-98784) PulpCodedException: The distributor asdf indicated a failed response when publishing repository zoo-puppet.
Associated revisions
Revision a80cd6d0
View on GitHub
Adds error logging to platform base FileDistributor exceptions
Write the exception and its traceback to the log at error level when FileDistributor publish has an error.
History
#1
Updated by bmbouter over 5 years ago
- Status changed from NEW to ASSIGNED
#2
Updated by bmbouter over 5 years ago
- Status changed from ASSIGNED to POST
- Platform Release set to master
#3
Updated by bmbouter over 5 years ago
- Status changed from POST to MODIFIED
- % Done changed from 0 to 100
Applied in changeset pulp|a80cd6d0b5890d43464bedb26af25c38b1182922.
#4
Updated by mhrivnak over 5 years ago
- Assignee set to bmbouter
- Triaged changed from No to Yes
#5
Updated by mhrivnak about 5 years ago
- Platform Release changed from master to 2.8.0
#6
Updated by dkliban@redhat.com almost 5 years ago
- Status changed from MODIFIED to 5
#7
Updated by dkliban@redhat.com almost 5 years ago
- Status changed from 5 to CLOSED - CURRENTRELEASE
#8
Updated by bmbouter almost 2 years ago
- Tags Pulp 2 added
Please register to edit this issue
Adds error logging to platform base FileDistributor exceptions
Write the exception and its traceback to the log at error level when FileDistributor publish has an error.
https://pulp.plan.io/issues/1248 closes #1248