Revert "Set salt test pipeline status to ABORTED in case of skip these tests"
This reverts commit 38fbda33eed3436ca0b69fa59b66dc0fa832738e.
Change-Id: I43772d4734d464d3ad0a58f7ceea9a42569359cf
diff --git a/test-salt-models-pipeline.groovy b/test-salt-models-pipeline.groovy
index 96ef843..de6a612 100644
--- a/test-salt-models-pipeline.groovy
+++ b/test-salt-models-pipeline.groovy
@@ -47,12 +47,10 @@
def gerritChange = gerrit.getGerritChange(GERRIT_NAME, GERRIT_HOST, GERRIT_CHANGE_NUMBER, CREDENTIALS_ID, true)
// test if gerrit change is already Verified
if(gerrit.patchsetHasApproval(gerritChange.currentPatchSet,"Verified", "+")){
- common.successMsg("Gerrit change ${GERRIT_CHANGE_NUMBER} patchset ${GERRIT_PATCHSET_NUMBER} already has Verified, skipping tests")
- currentBuild.result = 'ABORTED'
+ common.successMsg("Gerrit change ${GERRIT_CHANGE_NUMBER} patchset ${GERRIT_PATCHSET_NUMBER} already has Verified, skipping tests") // do nothing
// test WIP contains in commit message
}else if (gerritChange.commitMessage.contains("WIP")) {
- common.successMsg("Commit message contains WIP, skipping tests")
- currentBuild.result = 'ABORTED'
+ common.successMsg("Commit message contains WIP, skipping tests") // do nothing
} else {
def merged = gerritChange.status == "MERGED"
if(!merged){
@@ -61,7 +59,6 @@
])
} else{
common.successMsg("Change ${GERRIT_CHANGE_NUMBER} is already merged, no need to test them")
- currentBuild.result = 'ABORTED'
}
}
} else if(defaultGitRef && defaultGitUrl) {