Skip to content

Commit 5393d27

Browse files
committed
Simplify grepping log for 'Jenkins event:'
to diagnose why scabot misses jenkins job end notification, while start is ok
1 parent c740bb1 commit 5393d27

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

Diff for: gui/app/controllers/Scabot.scala

+1-1
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,7 @@ class Scabot @Inject() (val system: ActorSystem) extends Controller with GithubS
5858
handleWith(jenkinsEvent) match {
5959
case Success(message) => Ok(message)
6060
case Failure(ex) =>
61-
system.log.error(s"Couldn't handle Jenkins event ${request.body}.\n Fail: $ex")
61+
system.log.error(s"Couldn't handle Jenkins event: ${request.body}.\n Fail: $ex")
6262
InternalServerError(ex.getMessage)
6363
}
6464
}

Diff for: jenkins/src/main/scala/scabot/jenkins/JenkinsService.scala

+1-1
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ trait JenkinsService extends JenkinsApi {
55

66
def jenkinsEvent(jobState: JobState): String = jobState match {
77
case JobState(name, _, BuildState(number, phase, parameters, scm, result, full_url, log)) =>
8-
system.log.info(s"Jenkins notification $name [$number]: $phase ($result) at $full_url.\n Scm: scm\n Params: $parameters\n $log")
8+
system.log.info(s"Jenkins event: $name [$number]: $phase ($result) at $full_url.\n Scm: scm\n Params: $parameters\n $log")
99

1010
{
1111
for {

0 commit comments

Comments
 (0)