Move gerrit comment logic to its own file 73/36073/1
authorAric Gardner <agardner@linuxfoundation.org>
Wed, 14 Jun 2017 02:59:54 +0000 (10:59 +0800)
committerAric Gardner <agardner@linuxfoundation.org>
Wed, 14 Jun 2017 02:59:54 +0000 (10:59 +0800)
this file will become too complicated to escape inside the job
definition.

Change-Id: I3e167bee5d315a7ff3b52e7274b68c3146dfbd03
Signed-off-by: Aric Gardner <agardner@linuxfoundation.org>
jjb/ci_gate_security/anteater-report-to-gerrit.sh [new file with mode: 0644]
jjb/ci_gate_security/anteater-security-audit.sh
jjb/ci_gate_security/opnfv-ci-gate-security.yml

diff --git a/jjb/ci_gate_security/anteater-report-to-gerrit.sh b/jjb/ci_gate_security/anteater-report-to-gerrit.sh
new file mode 100644 (file)
index 0000000..6a5a0a5
--- /dev/null
@@ -0,0 +1,19 @@
+#!/bin/bash
+set -o errexit
+set -o pipefail
+export PATH=$PATH:/usr/local/bin/
+
+# If no violations were found, no lint log will exist.
+if [[ -e securityaudit.log ]] ; then
+    echo -e "\nposting security audit report to gerrit...\n"
+
+    cat securityaudit.log  | awk -F"ERROR\ \-\ " '{print $2}' > shortlog
+
+    ssh -p 29418 gerrit.opnfv.org \
+        "gerrit review -p $GERRIT_PROJECT \
+         -m \"$(cat shortlog)\" \
+         $GERRIT_PATCHSET_REVISION \
+         --notify NONE"
+
+    exit 1
+fi
index b11e68a..ecc5c6f 100644 (file)
@@ -1,6 +1,4 @@
 #!/bin/bash
 #!/bin/bash
-set -o pipefail
-
 cd $WORKSPACE
 echo "Generating patchset file to list changed files"
 git diff HEAD^1 --name-only | sed "s#^#/home/opnfv/anteater/$PROJECT/#" > $WORKSPACE/patchset
 cd $WORKSPACE
 echo "Generating patchset file to list changed files"
 git diff HEAD^1 --name-only | sed "s#^#/home/opnfv/anteater/$PROJECT/#" > $WORKSPACE/patchset
@@ -31,4 +29,3 @@ echo "--------------------------------------------------------"
 echo "Stopping docker container with ID $container_id"
 docker stop $container_id
 exit 0
 echo "Stopping docker container with ID $container_id"
 docker stop $container_id
 exit 0
-#exit $exit_code
index c364186..2270124 100644 (file)
 - builder:
     name: report-security-audit-result-to-gerrit
     builders:
 - builder:
     name: report-security-audit-result-to-gerrit
     builders:
-        - shell: |
-            #!/bin/bash
-            set -o errexit
-            set -o pipefail
-            export PATH=$PATH:/usr/local/bin/
-
-            # If no violations were found, no lint log will exist.
-            if [[ -e securityaudit.log ]] ; then
-                echo -e "\nposting security audit report to gerrit...\n"
-
-                cat securityaudit.log
-                echo
-
-                ssh -p 29418 gerrit.opnfv.org \
-                    "gerrit review -p $GERRIT_PROJECT \
-                     -m \"$(cat securityaudit.log)\" \
-                     $GERRIT_PATCHSET_REVISION \
-                     --notify NONE"
-
-                exit 1
-            fi
+        - shell:
+            !include-raw: ./anteater-report-to-gerrit.sh