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

Posted to commits@devlake.apache.org by "mappjzc (via GitHub)" <gi...@apache.org> on 2023/03/27 11:59:50 UTC

[GitHub] [incubator-devlake] mappjzc opened a new pull request, #4787: fix: fix cron

mappjzc opened a new pull request, #4787:
URL: https://github.com/apache/incubator-devlake/pull/4787

   ### Summary
   Fix the cron using the error blueprint to run pipelines.
   
   ### Does this close any open issues?
   Closes #4761 
   
   ### Screenshots
   ![image](https://user-images.githubusercontent.com/2921251/227935580-16339977-65ab-4c2c-af0e-8983a2e3978a.png)
   


-- 
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

[GitHub] [incubator-devlake] likyh merged pull request #4787: fix: fix cron

Posted by "likyh (via GitHub)" <gi...@apache.org>.

likyh merged PR #4787:
URL: https://github.com/apache/incubator-devlake/pull/4787


-- 
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

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

Posted by "likyh (via GitHub)" <gi...@apache.org>.

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

   Why did this issue happen?


-- 
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

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

Posted by "klesh (via GitHub)" <gi...@apache.org>.

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