diff options
| author |   Trevor Bramwell <tbramwell@linuxfoundation.org> | 2017-10-11 11:42:35 -0700 |
|---|---|---|
| committer | Trevor Bramwell <tbramwell@linuxfoundation.org> | 2017-10-11 11:42:35 -0700 |
| commit | 4cbd49f4b38f80ef4bde13bf1f290378ae0db0a9 (patch) | |
| tree | 17307fb2c5391237dfacf9d5d3b28a1ab3f750a1 /jjb/ci_gate_security | |
| parent | dc304463a3b26edb4abf24cb7292750400c42466 (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>
Diffstat (limited to 'jjb/ci_gate_security')
| -rw-r--r-- | jjb/ci_gate_security/anteater-report-to-gerrit.sh | 2 |
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)\" \ |