Bug 1032887 - rhts-sync-block spams beah log file with "Failure: twisted.python.failure.DefaultException: Timeout"
Summary: rhts-sync-block spams beah log file with "Failure: twisted.python.failure.Def...
Keywords:
Status: CLOSED NOTABUG
Alias: None
Product: Beaker
Classification: Retired
Component: beah
Version: 0.10
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: beaker-dev-list
QA Contact: tools-bugs
URL:
Whiteboard: Misc
Depends On: 894159
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-11-21 07:21 UTC by Nick Coghlan
Modified: 2019-04-17 10:48 UTC (History)
9 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of: 894159
Environment:
Last Closed: 2019-04-17 10:44:50 UTC
Embargoed:


Attachments (Terms of Use)

Description Nick Coghlan 2013-11-21 07:21:15 UTC
rhts-sync-block runs a polling loop to check if other hosts are ready. The internal timeouts within the polling loop are suppressed in rhts-sync-block itself, but still create a lot of noise in the beah log.

This also means that the "/debug/task_beah_unexpected" file will always be uploaded for tasks using rhts_sync_block.

+++ This bug was initially created as a clone of Bug #894159 +++

Description of problem:
In multihost testing, server side does:
    rhts-sync-set -s READY
    rhts-sync-block -s DONE $CLIENTS

and for the whole test run rhts generates following messages on console and in logs: 

(RHEL7)
rhts-sync-block: <Fault 8002: 'error'> 
2013-01-10 15:57:11,250 rhts_task.twisted emit: ERROR Unhandled Error 
Traceback (most recent call last): 
Failure: twisted.python.failure.DefaultException: Timeout
...

(RHEL6)
rhts-sync-block: <Fault 8002: 'error'>
...

--- Additional comment from Nick Coghlan on 2013-11-12 16:10:12 EST ---

I started on a patch that changes the beah XML-RPC API to allow rhts to pass the timeout through to the beah XML-RPC server rather than having to poll locally:

Draft rhts change: http://gerrit.beaker-project.org/#/c/2455/
Draft beah XML-RPC API change: http://gerrit.beaker-project.org/#/c/2454/

--- Additional comment from Dan Callaghan on 2013-11-13 14:46:22 EST ---

(In reply to Nick Coghlan from comment #8)
> I started on a patch that changes the beah XML-RPC API to allow rhts to pass
> the timeout through to the beah XML-RPC server rather than having to poll
> locally:
> 
> Draft rhts change: http://gerrit.beaker-project.org/#/c/2455/
> Draft beah XML-RPC API change: http://gerrit.beaker-project.org/#/c/2454/

There are a few additional complexities to this approach (moving the timeout to beah's fake RHTS XMLRPC server) which means it may not be worthwhile.

This patch just suppresses all XMLRPC faults in rhts-sync-block, under the assumption that they are timeouts:

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

but this still leaves the "Timeout" errors being spewed on the RHTS side, like this:

Traceback (most recent call last): 
Failure: twisted.python.failure.DefaultException: Timeout

Comment 2 Tomas Klohna 🔧 2019-04-17 10:44:50 UTC
No longer applies


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