Bug 1019537 - Lab controller daemons do not log details of unhandled XML-RPC exceptions
Summary: Lab controller daemons do not log details of unhandled XML-RPC exceptions
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Beaker
Classification: Retired
Component: lab controller
Version: 0.14
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: 0.15.3
Assignee: Dan Callaghan
QA Contact: tools-bugs
URL:
Whiteboard:
Depends On: 1003709
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-10-16 02:38 UTC by Nick Coghlan
Modified: 2018-02-06 00:41 UTC (History)
7 users (show)

Fixed In Version:
Clone Of: 1003709
Environment:
Last Closed: 2014-02-03 04:52:24 UTC
Embargoed:


Attachments (Terms of Use)

Description Nick Coghlan 2013-10-16 02:38:40 UTC
+++ This bug was initially created as a clone of Bug #1003709 +++

Description of problem:
The exception details are not logged if an XML-RPC call throws an exception in one of the Beaker lab controller daemons.

Version-Release number of selected component (if applicable):
0.14

Steps to Reproduce (for example):
1. Break beaker-proxy's ability to run 'sudo beaker-clear-netboot', for example by commenting out the lines in /etc/sudoers.d/beaker_proxy_clear_netboot
2. Make a call to beaker-proxy to clear netboot for some system, using XML-RPC:

>>> import xmlrpclib
>>> s = xmlrpclib.ServerProxy('http://localhost:8000/RPC2')
>>> s.clear_netboot('testsystem.example.com')

Actual results:
Clients receive an XML-RPC fault, which is correct, but on the beaker-proxy side nothing useful is logged indicating that the request failed, or why.

Expected results:
Log messages in /var/log/beaker/proxy.log showing that the request failed, and why (including, for the particular example, the error message from sudo or beaker-clear-netboot if there was one).

Additional info:
This was originally reported as part of bug 1003709, but the resolution there only addressed the HTTP side of the problem. This bug now covers the XML-RPC side.

Comment 1 Dan Callaghan 2014-01-21 00:56:20 UTC
On Gerrit:

http://gerrit.beaker-project.org/2703

Comment 5 Nick Coghlan 2014-02-03 04:52:24 UTC
This change is included in the Beaker 0.15.3 maintenance release:

http://beaker-project.org/docs/whats-new/release-0.15.html#beaker-0-15-3


Note You need to log in before you can comment on or make changes to this bug.