You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@devlake.apache.org by "klesh (via GitHub)" <gi...@apache.org> on 2023/03/27 14:57:06 UTC

[GitHub] [incubator-devlake] klesh commented on pull request #4787: fix: fix cron

klesh commented on PR #4787:
URL: https://github.com/apache/incubator-devlake/pull/4787#issuecomment-1485269941

   > Why did this issue happen?
   
   It happened because of the `Variable Scope`, the `AddFunc` is nested inside a `for loop` which effectively are all using the same `blueprint` to create new `pipelines`, when multiple cronjobs get triggered, the first one would succeed but the rest would fail because running multiple pipelines of the same bp is not allowed:
   
   https://github.com/apache/incubator-devlake/blob/1e2b351f4cd2ba86e913c3f11159f409e62718cc/backend/server/services/pipeline_helper.go#L44
   
   This should close the #4779 and #4538 as well


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@devlake.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org