search

open-formulieren / open-forms

Smart and dynamic forms
https://open-forms.readthedocs.io
Other
37 stars 26 forks source link

Clean up e-mails older than 90 days #4712

Closed sjoerdie closed 2 weeks ago

sjoerdie commented 2 months ago

Product versie / Product version

2.7.5

Customer reference

No response

Omschrijf het probleem / Describe the bug

It does not work.

Stappen om te reproduceren / Steps to reproduce

No response

Verwacht gedrag / Expected behavior

No response

Screen resolution

None

Device

None

OS

None

Browser

No response

sjoerdie commented 2 months ago

https://github.com/open-formulieren/open-forms/issues/4215

I don't see it working, please explain. I will fix it on the Ops side for now

joeribekker commented 1 month ago

Refinement: This is part of the celery beat config in (at least 2.7) so we need to dive deeper into whats wrong. @sjoerdie did you just see older emails somewhere or did you see a sentry error about it, or did someone else reported this?

robinmolen commented 1 month ago

I have tested this in my local development environment for versions 2.7.5 and 2.7.0. In both cases I tested with the following steps:

  1. created a test mail
  2. changed the date of my laptop to 90 days in the future a minute before midnight
  3. checked if test mail still excited
  4. started celery worker and celery beat
  5. after midnight, checked if test mail was delete

Both versions seem to work fine.. Maybe the celery isn't working correctly for this environment?

sergei-maertens commented 1 month ago

Adding blocked label - we need more info about the environment so we can investigate there - as reported above it seems to be working as intended.

joeribekker commented 2 weeks ago

No followup, closing.