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

Cron implementation #659

Closed
aldy505 opened this issue Jul 2, 2023 · 2 comments · Fixed by #661
Closed

Cron implementation #659

aldy505 opened this issue Jul 2, 2023 · 2 comments · Fixed by #661

Comments

@aldy505
Copy link
Contributor

aldy505 commented Jul 2, 2023

Implement this feature: https://docs.sentry.io/product/crons/

(@cleptric) I would imagine something similar to what we did in PHP getsentry/sentry-php#1467 & getsentry/sentry-php#1511

Node implementation: getsentry/sentry-javascript#8039

It turns out there's a parent issue: getsentry/sentry#48560

UPDATE: Wrong parent issue 🤦

@tonyo
Copy link
Member

tonyo commented Jul 3, 2023

Thanks for opening the issue @aldy505

Curious to hear about use cases for Go specifically. What kind of periodic jobs do you want to monitor? Do you use any frameworks for periodic tasks?

@aldy505
Copy link
Contributor Author

aldy505 commented Jul 3, 2023

@tonyo Sure!

What kind of periodic jobs do you want to monitor?

Basically a database clean-up job, there are a few tasks that I also need to create with Go in mind. I also have 1 job written in plain C to clean up self-hosted Github Actions runner's VM including downloaded Docker images and so on forth. It executes some commands and have some logics that's hard to understand if it was written in pure Bash.

Do you use any frameworks for periodic tasks?

Nope. Just time.Sleep and infinite for loop. But maybe others are using some libraries like robfig/cron, similar to what I did here: getsentry/vroom#284

@tonyo tonyo changed the title Cron implementation feat: Initial Cron implementation Jul 17, 2023
@tonyo tonyo changed the title feat: Initial Cron implementation Cron implementation Jul 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants