ci: improve build names and descriptions
Currently the logic is a bit messy and results are not great. Signed-off-by: Jakub Sokołowski <jakub@status.im>
This commit is contained in:
parent
29b78cc12e
commit
5902bbbeee
|
@ -91,6 +91,7 @@ pipeline {
|
||||||
when { expression { params.BUILD_SOURCE.startsWith('http') } }
|
when { expression { params.BUILD_SOURCE.startsWith('http') } }
|
||||||
steps { timeout(5) { script {
|
steps { timeout(5) { script {
|
||||||
sh 'mkdir -p ./tmp/pkg/'
|
sh 'mkdir -p ./tmp/pkg/'
|
||||||
|
setBuildDescFromFile(params.BUILD_SOURCE)
|
||||||
fileOperations([
|
fileOperations([
|
||||||
fileDownloadOperation(
|
fileDownloadOperation(
|
||||||
url: params.BUILD_SOURCE,
|
url: params.BUILD_SOURCE,
|
||||||
|
@ -112,6 +113,7 @@ pipeline {
|
||||||
selector: lastWithArtifacts(),
|
selector: lastWithArtifacts(),
|
||||||
target: './tmp'
|
target: './tmp'
|
||||||
)
|
)
|
||||||
|
setBuildDescFromFile(utils.findFile('tmp/pkg/*tar.gz'))
|
||||||
} } }
|
} } }
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -168,34 +170,25 @@ pipeline {
|
||||||
}
|
}
|
||||||
|
|
||||||
def setNewBuildName() {
|
def setNewBuildName() {
|
||||||
/* Skip PR builds for QA automation repo. */
|
if (currentBuild.upstreamBuilds) {
|
||||||
if (JOB_BASE_NAME == 'prs') {
|
|
||||||
return
|
|
||||||
}
|
|
||||||
/* For URLs we need to parse the filename to get attributes. */
|
|
||||||
if (params.BUILD_SOURCE.startsWith('http')) {
|
|
||||||
def tokens = utils.parseFilename(utils.baseName(params.BUILD_SOURCE))
|
|
||||||
currentBuild.displayName = tokens.build.replace(/^pr/, 'PR-')
|
|
||||||
currentBuild.description = formatMap([
|
|
||||||
Node: NODE_NAME,
|
|
||||||
Build: tokens.build,
|
|
||||||
Commit: tokens.commit,
|
|
||||||
Version: (tokens.tstamp ?: tokens.version),
|
|
||||||
])
|
|
||||||
} else {
|
|
||||||
def parent = utils.parentOrCurrentBuild()
|
def parent = utils.parentOrCurrentBuild()
|
||||||
if (!(parent.getFullDisplayName() ==~ /prs/)) {
|
currentBuild.displayName = parent.getFullDisplayName().minus('status-desktop » ')
|
||||||
currentBuild.displayName = parent.getFullDisplayName().minus('status-desktop » ')
|
|
||||||
} else {
|
|
||||||
currentBuild.displayName = utils.baseName(params.BUILD_SOURCE)
|
|
||||||
}
|
|
||||||
currentBuild.description = formatMap([
|
|
||||||
Node: NODE_NAME,
|
|
||||||
Build: params.BUILD_SOURCE.minus('status-desktop/'),
|
|
||||||
])
|
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
def setBuildDescFromFile(fileNameOrPath) {
|
||||||
|
def tokens = utils.parseFilename(utils.baseName(fileNameOrPath))
|
||||||
|
if (tokens.build.startsWith('pr')) {
|
||||||
|
currentBuild.displayName = tokens.build.replace(/^pr/, 'PR-')
|
||||||
|
}
|
||||||
|
currentBuild.description = formatMap([
|
||||||
|
Node: NODE_NAME,
|
||||||
|
Build: tokens.build,
|
||||||
|
Commit: tokens.commit,
|
||||||
|
Version: (tokens.tstamp ?: tokens.version),
|
||||||
|
])
|
||||||
|
}
|
||||||
|
|
||||||
def updateGitHubStatus() {
|
def updateGitHubStatus() {
|
||||||
/* For PR builds update check status. */
|
/* For PR builds update check status. */
|
||||||
if (params.BUILD_SOURCE ==~ /.*\/PR-[0-9]+\/?$/) {
|
if (params.BUILD_SOURCE ==~ /.*\/PR-[0-9]+\/?$/) {
|
||||||
|
|
Loading…
Reference in New Issue