It's a public method, we don't need that option anymore, let's assume we always ask for our AutoQA user. --- lib/python/bodhi_utils.py | 7 +++---- 1 files changed, 3 insertions(+), 4 deletions(-)
diff --git a/lib/python/bodhi_utils.py b/lib/python/bodhi_utils.py index 2df40f8..57123a1 100644 --- a/lib/python/bodhi_utils.py +++ b/lib/python/bodhi_utils.py @@ -102,12 +102,11 @@ def bodhi_list(params, limit=100): params['tg_paginate_no'] += 1 return updates
-def bodhi_already_commented(update, user, testname, arch): - '''Check if the comment is already posted. +def bodhi_already_commented(update, testname, arch): + '''Check if AutoQA comment is already posted.
Args: update -- Bodhi update object --or-- update title --or-- update ID - user -- username that posted the comment testname -- the name of the test arch -- tested architecture
@@ -219,7 +218,7 @@ def bodhi_post_testresult(update, testname, result, url, arch = 'noarch', karma comment = 'AutoQA: %s test %s on %s. The result can be found at: %s.' \ % (testname, result, arch, url) try: - (old_result, comment_time) = bodhi_already_commented(update, user, testname, arch) + (old_result, comment_time) = bodhi_already_commented(update, testname, arch) time_span = int(autoqa_conf.get('notifications', 'bodhi_posting_comments_span'))
if not _is_bodhi_testresult_needed(old_result, comment_time, result, time_span):
autoqa-devel@lists.fedorahosted.org