<p>George Joseph has uploaded this change for <strong>review</strong>.</p><p><a href="https://gerrit.asterisk.org/9396">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">CI: Update cleanup steps and permissions<br><br>Change-Id: I7ca92935979d94845af8e1caf4468cbd6209b7de<br>---<br>M tests/CI/gates.jenkinsfile<br>M tests/CI/unittests.jenkinsfile<br>2 files changed, 13 insertions(+), 9 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/96/9396/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/tests/CI/gates.jenkinsfile b/tests/CI/gates.jenkinsfile</span><br><span>index 40c7916..6f489a7 100644</span><br><span>--- a/tests/CI/gates.jenkinsfile</span><br><span>+++ b/tests/CI/gates.jenkinsfile</span><br><span>@@ -60,6 +60,8 @@</span><br><span> /* Here's where we switch to scripted pipeline */</span><br><span> script {</span><br><span> stage ("Checkout") {</span><br><span style="color: hsl(120, 100%, 40%);">+ sh "sudo chown -R jenkins:users ." </span><br><span style="color: hsl(120, 100%, 40%);">+ </span><br><span> /*</span><br><span> * Jenkins has already automatically checked out the base branch</span><br><span> * for this change but we now need to check out the change itself</span><br><span>diff --git a/tests/CI/unittests.jenkinsfile b/tests/CI/unittests.jenkinsfile</span><br><span>index 0925910..7486823 100644</span><br><span>--- a/tests/CI/unittests.jenkinsfile</span><br><span>+++ b/tests/CI/unittests.jenkinsfile</span><br><span>@@ -65,6 +65,7 @@</span><br><span> steps {</span><br><span> script {</span><br><span> stage ("Checkout") {</span><br><span style="color: hsl(120, 100%, 40%);">+ sh "sudo chown -R jenkins:users ." </span><br><span> /*</span><br><span> * Jenkins has already automatically checked out the base branch</span><br><span> * for this change but we now need to check out the change itself</span><br><span>@@ -75,6 +76,7 @@</span><br><span> * The Gerrit Trigger provides all the URLs and refspecs to</span><br><span> * check out the change.</span><br><span> */</span><br><span style="color: hsl(120, 100%, 40%);">+ sh "sudo chown -R jenkins:users ." </span><br><span> checkout scm: [$class: 'GitSCM',</span><br><span> branches: [[name: env.GERRIT_BRANCH ]],</span><br><span> extensions: [</span><br><span>@@ -134,20 +136,20 @@</span><br><span> keepLongStdio: true</span><br><span> }</span><br><span> }</span><br><span style="color: hsl(0, 100%, 40%);">-</span><br><span style="color: hsl(0, 100%, 40%);">- stage ('Cleanup') {</span><br><span style="color: hsl(0, 100%, 40%);">- sh "sudo make distclean || : "</span><br><span style="color: hsl(0, 100%, 40%);">- }</span><br><span> }</span><br><span> }</span><br><span> }</span><br><span> }</span><br><span style="color: hsl(0, 100%, 40%);">- /*</span><br><span style="color: hsl(0, 100%, 40%);">- * The Gerrit Trigger will automatically post the "Verified" results back</span><br><span style="color: hsl(0, 100%, 40%);">- * to Gerrit but the verification publisher publishes extra stuff in the</span><br><span style="color: hsl(0, 100%, 40%);">- * "Code Review" section of the review.</span><br><span style="color: hsl(0, 100%, 40%);">- */</span><br><span> post {</span><br><span style="color: hsl(120, 100%, 40%);">+ always {</span><br><span style="color: hsl(120, 100%, 40%);">+ sh "sudo make distclean || : "</span><br><span style="color: hsl(120, 100%, 40%);">+ sh "sudo rm -rf tests/CI/output || : "</span><br><span style="color: hsl(120, 100%, 40%);">+ }</span><br><span style="color: hsl(120, 100%, 40%);">+ /*</span><br><span style="color: hsl(120, 100%, 40%);">+ * The Gerrit Trigger will automatically post the "Verified" results back</span><br><span style="color: hsl(120, 100%, 40%);">+ * to Gerrit but the verification publisher publishes extra stuff in the</span><br><span style="color: hsl(120, 100%, 40%);">+ * "Code Review" section of the review.</span><br><span style="color: hsl(120, 100%, 40%);">+ */</span><br><span> success {</span><br><span> gerritverificationpublisher verifyStatusValue: 1, verifyStatusCategory: 'Passed',</span><br><span> verifyStatusURL: '', verifyStatusComment: '',</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.asterisk.org/9396">change 9396</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.asterisk.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.asterisk.org/9396"/><meta itemprop="name" content="View Change"/></div></div>
<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: newchange </div>
<div style="display:none"> Gerrit-Change-Id: I7ca92935979d94845af8e1caf4468cbd6209b7de </div>
<div style="display:none"> Gerrit-Change-Number: 9396 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: George Joseph <gjoseph@digium.com> </div>