opendev/elastic-recheck
Code Issues Proposed changes

Fix query_builder.result_ready() query for zuulv3 results

Browse Source 
The first condition does not hit because the message with
the path to post-ssh.yaml was not hitting without the .yaml
suffix on the path. Assuming the unescaped . was messing
with the query.

Change-Id: I293e9c6fa215cb3f8638763895fccb4bfcf3c235
This commit is contained in:
Matt Riedemann 2018-05-08 17:57:42 -04:00
parent 610841f9a3
commit 320932bbc1
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
elastic_recheck/query_builder.py
View File

@ -78,7 +78,7 @@ def result_ready(change, patchset, name, short_uuid):
# TODO(dmsimard): Revisit this query once Zuul v2 is no longer supported
# Let's value legibility over pep8 line width here...
query = (
'((filename:"job-output.txt" AND message:"POST-RUN END" AND message:"project-config/playbooks/base/post-ssh")' # flake8: noqa
'((filename:"job-output.txt" AND message:"POST-RUN END" AND message:"project-config/playbooks/base/post-ssh.yaml")' # flake8: noqa
' OR '
'(filename:"console.html" AND (message:"[Zuul] Job complete" OR message:"[SCP] Copying console log" OR message:"Grabbing consoleLog")))' # flake8: noqa
' AND build_status:"FAILURE"'