-
Notifications
You must be signed in to change notification settings - Fork 1
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
Turn fil-deal-ingester into service #23
Comments
This could potentially be automated using github actions. |
Great idea! |
There seems to be a limit on duration and storage for Github runners (in our requirements we state that job between 3 - 5hrs and could take up to 40GB of space). We could self-host Github runners to mitigate this problem. |
Simpler version would be just to run scheduled machines on fly.io |
@pyropy and me discussed this and agreed that this would be nice to add to M4.12, since it's a quick win and will remove an annoying manual task |
It would be great if the new automated process could post a message to Slack with the summary. I think that's the very minimum we need for observability - if we see the message, then we know all is good; if we don't then we know we need to investigate.
|
Good idea to make this visible. Posting to Slack is one option, tracking in Influx, with Grafana alerts, is another, if we know how to set the alerts. |
Instead of manually running the script, automate it in a service. This way we can also increase the ingestion frequency.
Add monitoring:
added
below thresholddeleted
above thresholdThe text was updated successfully, but these errors were encountered: