Fix incorrect comment that security report posted 33/44833/1
authorTrevor Bramwell <tbramwell@linuxfoundation.org>
Wed, 11 Oct 2017 18:42:35 +0000 (11:42 -0700)
committerTrevor Bramwell <tbramwell@linuxfoundation.org>
Wed, 11 Oct 2017 18:42:35 +0000 (11:42 -0700)
A string is added to the build that a security report is being
reported to Gerrit, even when it is not. This moves the string to just
before the line when the comment is posted so we aren't lying to
developers.

Change-Id: I73840d025e8be86b6ac02772b22b22c4abd29422
Signed-off-by: Trevor Bramwell <tbramwell@linuxfoundation.org>
jjb/ci_gate_security/anteater-report-to-gerrit.sh

index 6710862..51a8ac4 100644 (file)
@@ -6,7 +6,6 @@ EXITSTATUS=0
 
 # This Log should always exist
 if [[ -e securityaudit.log ]] ; then
-    echo -e "\nposting security audit report to gerrit...\n"
 
     #check if log has errors
     if grep ERROR securityaudit.log; then
@@ -17,6 +16,7 @@ if [[ -e securityaudit.log ]] ; then
 
     # Only report to Gerrit when there are errors to report.
     if [[ -s shortlog ]]; then
+        echo -e "\nposting security audit report to gerrit...\n"
         ssh -p 29418 gerrit.opnfv.org \
             "gerrit review -p $GERRIT_PROJECT \
             -m \"$(cat shortlog)\" \