From: Sébastien Han <seb@redhat.com>
Date: Fri, 12 May 2017 08:08:42 +0000 (+0200)
Subject: ceph-docker-lint: exclude codes
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=d0eb2df108e8d54c9672d657c312798ec82ca279;p=ceph-build.git

ceph-docker-lint: exclude codes

Excluding the following code:

* SC1091: since our entrypoint copies files in / during this shellcheck
the files can not be found. Basically we source ./file but presently the
file is under osd_directory/file so can not be found.
* SC2015: because we use 'set -e' we don't have other choice to use '||
true' here.
* SC2009: pgrep returns processes ID where we are looking for a specific
string in the process line.
* SC2001: our sed expressions are too complex we can not substitute them
with bash ${variable//search/replace}.

Signed-off-by: Sébastien Han <seb@redhat.com>
---

diff --git a/ceph-docker-lint/build/build b/ceph-docker-lint/build/build
index 8b3a173b..6add50d3 100755
--- a/ceph-docker-lint/build/build
+++ b/ceph-docker-lint/build/build
@@ -3,8 +3,7 @@
 set -e
 set -x
 
-#IGNORE_THESE_CODES="SC2005,SC2068,SC2086,SC2148,SC2162"
-IGNORE_THESE_CODES=""
+IGNORE_THESE_CODES="SC1091,SC2015,SC2009,SC2001"
 
 function generate_filelist(){
    if [[ "$pull_request_id" -eq "" || "${ghprbCommentBody:-}" = "jenkins lint all" ]]