Skip to content

Commit 1c8ca44

Browse files
committed
fix skip message not saved
1 parent eca70ca commit 1c8ca44

File tree

1 file changed

+2
-5
lines changed

1 file changed

+2
-5
lines changed

core/src/main/java/com/flowci/core/job/manager/JobActionManagerImpl.java

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -646,12 +646,10 @@ private void dispatch(Job job, Agent agent) throws ScriptException {
646646
if (!canExecute) {
647647
nextStep.setStartAt(new Date());
648648
nextStep.setFinishAt(new Date());
649-
nextStep.setError(Step.MessageSkippedOnCondition);
650-
651649
updateJobTime(job, nextStep, tree, next);
652650

653651
setJobStatusAndSave(job, Job.Status.RUNNING, null);
654-
stepService.toStatus(nextStep, Executed.Status.SKIPPED, null);
652+
stepService.toStatus(nextStep, Executed.Status.SKIPPED, Step.MessageSkippedOnCondition);
655653

656654
JobSmContext context = new JobSmContext();
657655
context.job = job;
@@ -711,11 +709,10 @@ private boolean toNextStep(JobSmContext context) throws ScriptException {
711709
if (!canExecute) {
712710
nextStep.setStartAt(new Date());
713711
nextStep.setFinishAt(new Date());
714-
nextStep.setError(Step.MessageSkippedOnCondition);
715712
updateJobTime(job, nextStep, tree, next.get());
716713

717714
setJobStatusAndSave(job, Job.Status.RUNNING, null);
718-
stepService.toStatus(nextStep, Step.Status.SKIPPED, null);
715+
stepService.toStatus(nextStep, Step.Status.SKIPPED, Step.MessageSkippedOnCondition);
719716
return toNextStep(context);
720717
}
721718

0 commit comments

Comments
 (0)