Skip to content
Snippets Groups Projects

Clean up minidumps in RemoveOldReports

Open Marc Gonzalez requested to merge mgonzalez/CrashDragon:cron-cleanup into develop
1 unresolved thread

AfterDelete is broken?

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
287 287
288 288 // RemoveOldReports removes reports older than the configured report retention time
289 289 func RemoveOldReports() {
290 var report_ids []string
290 291 removeAge := time.Now().Add(-viper.GetDuration("Housekeeping.ReportRetentionTime"))
291 DB.Delete(&Report{}, "created_at < ?", removeAge)
292 DB.Table("reports").Where("created_at < ?", removeAge).Pluck("id", &report_ids)
293 DeleteReports(report_ids)
  • So turns out the AfterDelete not working here is kind of expected as the actual objects are not queried:

    https://github.com/go-gorm/gorm/issues/5445

  • Please register or sign in to reply
    Loading