R
Railwayβ€’7mo ago
rpuls

Upstream repo, check for updates not working consitently?

Hello, I just updated my upstream repo that my template is using in two services. At first I used the "check for updates" in both services and updated them to latest, I then had a build error and had to push a new fix, however, since that the "check for updates" fails to find ny new commits. What can I do? I have seen this before in other project where it did not work at all when I pushed, but I have some hopes here since it did work the first time just 10 minutes ago. πŸ™
No description
Solution:
Anyway, just merged a fix for this so it doesn't use the cache when manually checking. πŸ‘
Jump to solution
14 Replies
Percy
Percyβ€’7mo ago
Project ID: f40468c7-29f6-4806-9ec5-30050947b99d
rpuls
rpulsOPβ€’7mo ago
f40468c7-29f6-4806-9ec5-30050947b99d
Brody
Brodyβ€’7mo ago
the check for updates is cached for 24 hours afaik cc @fp what would be the reason for such a long cache on that? is it an API intensive check?
Faraz
Farazβ€’7mo ago
Good question. It is intensive + GH API rate limits for public repos aren't great even when authenticated.
Brody
Brodyβ€’7mo ago
am i correct with the 24 hour number?
Faraz
Farazβ€’7mo ago
Yes.
Brody
Brodyβ€’7mo ago
would dropping that down to 12 hours be okay, or would it be worth it to instead purse alterative methods to check for updates?
Faraz
Farazβ€’7mo ago
I think we could make this fetch fresh on click and the automatic fetch can remain 24h.
Brody
Brodyβ€’7mo ago
best of both worlds! would need a limit per hour or a dead time though, check #πŸ—£ο½œforum-feed if you wanna know why i say that :kekw:
rpuls
rpulsOPβ€’7mo ago
That sounds good! Thanks! πŸ™‚ today I was able to fetch the update Btw, aren't there some GH hooks that would work the other way around? So you don't have to poll for updates?
Faraz
Farazβ€’6mo ago
This is kinda finicky as these are public repos. Receiving webhooks only works if the repo owner has given us access to the repo. Us being our GH app.
Solution
Faraz
Farazβ€’6mo ago
Anyway, just merged a fix for this so it doesn't use the cache when manually checking. πŸ‘
rpuls
rpulsOPβ€’6mo ago
Amazing ! πŸ˜„ thank you !!!
Want results from more Discord servers?
Add your server