Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Warm up rpki-client cache 30 min before scheduled run #25

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

fjahr
Copy link
Collaborator

@fjahr fjahr commented Sep 29, 2024

This is an idea to improve the issue described in #19. We warm up the rpki-client cache once 30min before the scheduled run.

Keeping this as a draft for now. I need to think about if we want to do some more refactorings, if this is really helping and if it is stable. I don't think we need to have this soon. It would be good enough for the next runs in the short-term that we have debug logging and see if the issue #19 comes up more regularly. If it comes up more regularly maybe we see some other issues in those logs that give us a better idea to actually fix the root cause and this might turn out to not be needed at all.

@jurraca
Copy link
Contributor

jurraca commented Sep 30, 2024

Would it make sense to fetch the repository data regularly and maintain the latest set in the asmap-data repo?

@fjahr
Copy link
Collaborator Author

fjahr commented Sep 30, 2024

Would it make sense to fetch the repository data regularly and maintain the latest set in the asmap-data repo?

No, I don't think that makes sense. First of all, we would need to pay for the storage space we would need for that. And I also don't see any upside to it. The whole RPKI repository infrastructure is optimized for (decentralized) data availability so if there is a reason why we can't get the data from it reliably we should first find out what the issue is. GitHub is also not a very reliable service IMO so we would need to think about redundancy etc. and we would also make RPKI a centralized data source because if something goes wrong only our data might be used. I don't want to risk that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants