Browse code

Rename DCO check param in Jenkinsfile

Previously it was a negative parameter for skiping the DCO check, but
this is different from other checks. It was requested that I change this
in #40023 but I'm factoring it out as an unrelated change.

Signed-off-by: Sam Whited <sam@samwhited.com>

Sam Whited authored on 2019/11/01 00:51:45
Showing 1 changed files
... ...
@@ -15,7 +15,7 @@ pipeline {
15 15
         booleanParam(name: 'ppc64le', defaultValue: true, description: 'PowerPC (ppc64le) Build/Test')
16 16
         booleanParam(name: 'windowsRS1', defaultValue: false, description: 'Windows 2016 (RS1) Build/Test')
17 17
         booleanParam(name: 'windowsRS5', defaultValue: true, description: 'Windows 2019 (RS5) Build/Test')
18
-        booleanParam(name: 'skip_dco', defaultValue: false, description: 'Skip the DCO check')
18
+        booleanParam(name: 'dco', defaultValue: true, description: 'Run the DCO check')
19 19
     }
20 20
     environment {
21 21
         DOCKER_BUILDKIT     = '1'
... ...
@@ -41,7 +41,7 @@ pipeline {
41 41
         stage('DCO-check') {
42 42
             when {
43 43
                 beforeAgent true
44
-                expression { !params.skip_dco }
44
+                expression { params.dco }
45 45
             }
46 46
             agent { label 'amd64 && ubuntu-1804 && overlay2' }
47 47
             steps {
... ...
@@ -266,7 +266,7 @@ pipeline {
266 266
                                 sh '''
267 267
                                 # todo: include ip_vs in base image
268 268
                                 sudo modprobe ip_vs
269
-                
269
+
270 270
                                 docker build --force-rm --build-arg APT_MIRROR -t docker:${GIT_COMMIT} .
271 271
                                 '''
272 272
                             }
... ...
@@ -276,7 +276,7 @@ pipeline {
276 276
                                 sh '''#!/bin/bash
277 277
                                 # bash is needed so 'jobs -p' works properly
278 278
                                 # it also accepts setting inline envvars for functions without explicitly exporting
279
- 
279
+
280 280
                                 run_tests() {
281 281
                                         [ -n "$TESTDEBUG" ] && rm= || rm=--rm;
282 282
                                         docker run $rm -t --privileged \