How often does promotion step git-wait-for-pr query github?

[nissessenap]

Hey, I just started playing with Kargo and added git-wait-for-pr to my promotion.
I got it to work, but it takes ages to actually get a success, even though I merged the creation PR quickly.
I guess it's some kind of query that re-runs instead of a webhook to get a notification from github about the PR getting merged.

Is it triggered every time the reconcile loop runs? How often is that?
Is there any way of speeding this up? Might it be worth to create some kind of even listener for git-wait-for-pr in kargo?
I first thought about using the warehouse feature, but I assume that would only create a new freight, which I of course don't want.

Any help in this area would be helpful.

[krancour]

Is it triggered every time the reconcile loop runs?

Yes. Exactly.

How often is that?

🤷‍♂️ If the Promotion is still "running" (waiting on something external like a PR, really), it gets requeued after five minutes... unless something else makes it get requeued sooner.

Once it's on the queue, the time until it's reconciled is obviously a function of things like queue depth, number of workers, and what the workers are up to.

So the exact interval is indefinite, unfortunately.

There's an open issue for adding a webhook: #647

Priorities have shifted between then and now, so it hasn't gotten any real attention, but it's absolutely still something we want.

unless something else makes it get requeued sooner

It may interest you to know that if you refresh the Stage, it will force the Promotion back onto the queue as well. This is what I do if I'm in a hurry.

[nissessenap]

Thanks a lot for your great and quick answer.

Btw how do you refresh the stage?

[krancour]

If you click the Stage in the UI, the upper right corner of the more detailed Stage view has a refresh button. The CLI can also do something like kargo refresh stage <name> --project <project>

[joebowbeer]

Open issue #4150

[krancour]

#4150 only refreshes Warehouses. i.e. Triggers artifact discovery processes.

Triggering other things in response to other events (such as triggering re-reconciliation of a Promotion when a PR is merged or closed) is additional functionality to be added later.

[joebowbeer]

@krancour Is there another open issue?

#647 and #1449 are closed.