Skip to content

Commit 09f3210

Browse files
authored
Merge pull request #566 from iExecBlockchainComputing/bugfix/improve-log-on-can-update-replicate-status
Improve log on `canUpdateReplicateStatus` method
2 parents 27ac58f + 1309ab4 commit 09f3210

File tree

2 files changed

+5
-3
lines changed

2 files changed

+5
-3
lines changed

CHANGELOG.md

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,8 +6,9 @@ All notable changes to this project will be documented in this file.
66

77
### New Features
88
### Bug Fixes
9-
* Remove explicit version on `micrometer-registry-prometheus` dependency. (#563)
10-
* Send a `TaskNotificationType` to worker through a 2XX HTTP status code. (#564)
9+
- Remove explicit version on `micrometer-registry-prometheus` dependency. (#563)
10+
- Send a `TaskNotificationType` to worker through a 2XX HTTP status code. (#564)
11+
- Improve log on `canUpdateReplicateStatus` method. (#566)
1112
### Quality
1213
### Dependency Upgrades
1314

src/main/java/com/iexec/core/replicate/ReplicatesService.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -182,7 +182,8 @@ public ReplicateStatusUpdateError canUpdateReplicateStatus(String chainTaskId,
182182

183183
boolean hasAlreadyTransitionedToStatus = replicate.containsStatus(newStatus);
184184
if (hasAlreadyTransitionedToStatus) {
185-
log.warn("Cannot update replicate, status {} already reported.", newStatus);
185+
log.warn("Cannot update replicate, status already reported {}",
186+
getStatusUpdateLogs(chainTaskId, replicate, statusUpdate));
186187
return ReplicateStatusUpdateError.ALREADY_REPORTED;
187188
}
188189

0 commit comments

Comments
 (0)