#858 - build the log link in the message

uselagoon · Jan 30, 2019 · c150618 · c150618
1 parent 4d846dc
commit c150618
Showing 1 changed file with 6 additions and 6 deletions.
diff --git a/services/openshiftbuilddeploymonitor/src/index.js b/services/openshiftbuilddeploymonitor/src/index.js
@@ -174,38 +174,38 @@ const messageConsumer = async msg => {
       try {
         const buildLog = await buildsLogGet()
         const s3UploadResult = await saveBuildLog(buildName, projectName, branchName, buildLog, buildstatus)
-        logLink = `<${s3UploadResult.Location}|Logs>`
+        logLink = s3UploadResult.Location
         meta.logLink = logLink
       } catch (err) {
         logger.warn(`${openshiftProject} ${buildName}: Error while getting and uploading Logs to S3, Error: ${err}. Continuing without log link in message`)
         meta.logLink = ''
       }
       sendToLagoonLogs('warn', projectName, "", `task:builddeploy-openshift:${buildPhase}`, meta,
-        `*[${projectName}]* ${logMessage} Build \`${buildName}\` cancelled. ${logLink}`
+        `*[${projectName}]* ${logMessage} Build \`${buildName}\` cancelled. <${logLink}|Logs>`
       )
       break;
 
     case "failed":
       try {
         const buildLog = await buildsLogGet()
         const s3UploadResult = await saveBuildLog(buildName, projectName, branchName, buildLog, buildstatus)
-        logLink = `<${s3UploadResult.Location}|Logs>`
+        logLink = s3UploadResult.Location
         meta.logLink = logLink
       } catch (err) {
         logger.warn(`${openshiftProject} ${buildName}: Error while getting and uploading Logs to S3, Error: ${err}. Continuing without log link in message`)
         meta.logLink = ''
       }
 
       sendToLagoonLogs('error', projectName, "", `task:builddeploy-openshift:${buildPhase}`, meta,
-        `*[${projectName}]* ${logMessage} Build \`${buildName}\` failed. ${logLink}`
+        `*[${projectName}]* ${logMessage} Build \`${buildName}\` failed. $<${logLink}|Logs>`
       )
       break;
 
     case "complete":
       try {
         const buildLog = await buildsLogGet()
         const s3UploadResult = await saveBuildLog(buildName, projectName, branchName, buildLog, buildstatus)
-        logLink = `<${s3UploadResult.Location}|Logs>`
+        logLink = s3UploadResult.Location
         meta.loglink = loglink
       } catch (err) {
         logger.warn(`${openshiftProject} ${buildName}: Error while getting and uploading Logs to S3, Error: ${err}. Continuing without log link in message`)
@@ -229,7 +229,7 @@ const messageConsumer = async msg => {
       meta.route = route
       meta.routes = routes
       sendToLagoonLogs('info', projectName, "", `task:builddeploy-openshift:${buildPhase}`, meta,
-        `*[${projectName}]* ${logMessage} Build \`${buildName}\` complete. ${logLink} \n ${route}\n ${routes.join("\n")}`
+        `*[${projectName}]* ${logMessage} Build \`${buildName}\` complete. <${logLink}|Logs> \n ${route}\n ${routes.join("\n")}`
       )
       try {
         const updateEnvironmentResult = await updateEnvironment(