summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorTrevor Bramwell <tbramwell@linuxfoundation.org>2017-10-11 11:42:35 -0700
committerTrevor Bramwell <tbramwell@linuxfoundation.org>2017-10-11 11:42:35 -0700
commit4cbd49f4b38f80ef4bde13bf1f290378ae0db0a9 (patch)
tree17307fb2c5391237dfacf9d5d3b28a1ab3f750a1
parentdc304463a3b26edb4abf24cb7292750400c42466 (diff)
Fix incorrect comment that security report posted
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>
-rw-r--r--jjb/ci_gate_security/anteater-report-to-gerrit.sh2
1 files changed, 1 insertions, 1 deletions
diff --git a/jjb/ci_gate_security/anteater-report-to-gerrit.sh b/jjb/ci_gate_security/anteater-report-to-gerrit.sh
index 6710862a7..51a8ac47b 100644
--- a/jjb/ci_gate_security/anteater-report-to-gerrit.sh
+++ b/jjb/ci_gate_security/anteater-report-to-gerrit.sh
@@ -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)\" \