Skip to content

Commit ad78dce

Browse files
Merge pull request #1065 from stoneshi-yunify/master
fix time parse problem
2 parents f45c986 + fb18153 commit ad78dce

File tree

1 file changed

+5
-4
lines changed

1 file changed

+5
-4
lines changed

pkg/client/devops/jenkins/pipeline.go

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -833,10 +833,11 @@ func (p *Pipeline) CheckCron() (*devops.CheckCronRes, error) {
833833
res.Result = "error"
834834
})
835835
if res.Result == "ok" {
836-
res.LastTime, res.NextTime, err = parseCronJobTime(res.Message)
837-
if err != nil {
838-
klog.Error(err)
839-
return interanlErrorMessage(), err
836+
var parseErr error
837+
res.LastTime, res.NextTime, parseErr = parseCronJobTime(res.Message)
838+
if parseErr != nil {
839+
// don't exit, just log the error
840+
klog.Error(parseErr)
840841
}
841842
}
842843

0 commit comments

Comments
 (0)