Skip to content
This repository was archived by the owner on Nov 18, 2022. It is now read-only.
/ community-functions Public archive

[WiP] Lack of JENKINS-50777 metadata should not be fatal #11

Merged
rtyler merged 1 commit into from
May 3, 2018
Merged

[WiP] Lack of JENKINS-50777 metadata should not be fatal #11

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 5 additions & 1 deletion incrementals-publisher/index.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -83,7 +83,11 @@ module.exports = async (context, data) => {

if (!buildMetadataParsed.hash) {
context.log.error('Unable to retrieve a hash or pullHash', buildMetadataJSON);
return failRequest(context, 'Unable to retrieve a hash or pullHash');
context.res = {
status: 200,
body: 'Did not find a Git commit hash associated with this build. Some plugins on ' + JENKINS_HOST + ' may not yet have been updated with JENKINS-50777 REST API enhancements. Skipping deployment.\n'
};
return;
}

let folderMetadata = await fetch(process.env.FOLDER_METADATA_URL || pipeline.getFolderApiUrl(buildUrl), jenkinsOpts);
Expand Down