ZOOKEEPER-3896: Added cleanup bits to all ZK builds

Recently I noticed some weird null pointer exception issues during compile in out new builds. I usually resolve this by a full "git clean" locally, so I added it to the Jenkinsfiles too.

Author: Andor Molnar <[email protected]>

Reviewers: Enrico Olivelli <[email protected]>

Closes apache#1431 from anmolnar/ZOOKEEPER-3896

Author: anmolnar

Jenkinsfile

@@ -52,13 +52,24 @@ pipeline {
                     stage('BuildAndTest') {
                         steps {
                             git 'https://github.com/apache/zookeeper'
+                            sh "git clean -fxd"
                             sh "mvn verify spotbugs:check checkstyle:check -Pfull-build -Dsurefire-forkcount=4"
                         }
                         post {
                             always {
                                junit '**/target/surefire-reports/TEST-*.xml'
                                archiveArtifacts '**/target/*.jar'
                             }
+                            // Jenkins pipeline jobs fill slaves on PRs without this :(
+                            cleanup() {
+                                script {
+                                    sh label: 'Cleanup workspace', script: '''
+                                        # See HADOOP-13951
+                                        chmod -R u+rxw "${WORKSPACE}"
+                                        '''
+                                    deleteDir()
+                                }
+                            }
                         }
                     }
                 }

Jenkinsfile-PreCommit

@@ -37,6 +37,7 @@ pipeline {
         stage('BuildAndTest') {
             steps {
                 git 'https://github.com/apache/zookeeper'
+                sh "git clean -fxd"
                 sh "mvn verify spotbugs:check checkstyle:check -Pfull-build -Dsurefire-forkcount=4"
             }
             post {