Skip to content

Commit

Permalink
Log stacktrace when deployment fails
Browse files Browse the repository at this point in the history
  • Loading branch information
micheljung committed Oct 26, 2017
1 parent dd0f306 commit d8609e7
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 4 deletions.
2 changes: 1 addition & 1 deletion build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -109,7 +109,7 @@ apply plugin: 'org.springframework.boot'
apply plugin: 'propdeps'

group = 'faforever'
version = '0.9.2'
version = '0.9.3'

sourceCompatibility = 1.8
targetCompatibility = 1.8
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -39,9 +39,9 @@ public GitHubDeploymentService(ApplicationContext applicationContext, FafApiProp
void createDeploymentIfEligible(Push push) {
String ref = push.getRef();
Optional<FeaturedMod> optional = featuredModService.findByGitUrlAndGitBranch(
push.getRepository().gitHttpTransportUrl(),
push.getRef().replace("refs/heads/", "")
);
push.getRepository().gitHttpTransportUrl(),
push.getRef().replace("refs/heads/", "")
);

if (!optional.isPresent()) {
log.warn("No configuration present for repository '{}' and ref '{}'", push.getRepository().gitHttpTransportUrl(), push.getRef());
Expand Down Expand Up @@ -76,6 +76,7 @@ public void deploy(Deployment deployment) {
try {
performDeployment(ghDeployment, repository, deploymentId);
} catch (Exception e) {
log.error("Deployment failed", e);
updateDeploymentStatus(deploymentId, repository, GHDeploymentState.FAILURE, e.getMessage());
}
}
Expand Down

0 comments on commit d8609e7

Please sign in to comment.