First round of review, just looking at tests/compose_tree for now.
On Monday, I'll try to actually run it :)
control.autoqa:
# The test can be run for post-tree-compose, or post-koji-build
if hook != 'post-koji-build':
execute = False
What about the post-tree-compose? An outdated comment?
compose_tree.py:
try:
out = utils.system_output(cmd, retain_output=True)
except error.CmdError, e:
self.result = 'FAILED'
out = e.result_obj.stdout
else:
self.outputs = out
Maybe 'finally' instead of 'else'?