New commits detected on ManageIQ/manageiq/gaprindashvili: https://github.com/ManageIQ/manageiq/commit/920e709435e62754456237d03974730d0d0f9228 commit 920e709435e62754456237d03974730d0d0f9228 Author: Joe Rafaniello <jrafanie.github.com> AuthorDate: Thu Mar 8 15:33:38 2018 -0500 Commit: Joe Rafaniello <jrafanie.github.com> CommitDate: Thu Mar 8 15:33:38 2018 -0500 Merge pull request #17046 from NickLaMuro/notification-purging Adds purging for notifications (cherry picked from commit 427257a185aed602d1596dfb81c623254957fbe6) Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1553780 app/models/miq_schedule_worker/jobs.rb | 4 + app/models/miq_schedule_worker/runner.rb | 5 + app/models/notification.rb | 2 + app/models/notification/purging.rb | 24 + config/settings.yml | 5 + spec/models/notification/purging_spec.rb | 40 + 6 files changed, 80 insertions(+)
yes thanks Nick. I changed the created_at and updated_at time in the notifications table to last month and it will get purged now right ? Waiting for it to get purged . Will verify the BZ then.
> I changed the created_at and updated_at time in the notifications table to last month and it will get purged now right ? Yup, that is correct, though the job for it only runs once a day, so depending one when the last one was scheduled to run it might take up to a day for the purge job to run.
Fixed in 5.9.2.0.20180320210136_355c506
Since the problem described in this bug report should be resolved in a recent advisory, it has been closed with a resolution of ERRATA. For information on the advisory, and where to find the updated files, follow the link below. If the solution does not work for you, open a new bug report. https://access.redhat.com/errata/RHSA-2018:1328