celery: add update_domain_status task

[carlinmack]

❤️ Thank you for your contribution!

(Part of) Close #2777

Description

Please describe briefly your pull request.

Run update domain statistics task every hour. These statistics change when users create accounts, get blocked, etc

We simply never added a task to update these regularly

Checklist

Ticks in all boxes and 🟢 on all GitHub actions status checks are required to merge:

• I'm aware of the code of conduct.
• I've created logical separate commits and followed the commit message format.
• I've added relevant test cases.
• I've added relevant documentation.
• I've marked translation strings.
• I've identified the copyright holder(s) and updated copyright headers for touched files (>15 lines contributions).
• I've NOT included third-party code (copy/pasted source code or new dependencies).
  • If you have added third-party code (copy/pasted or new dependencies), please reach out to an architect.

Frontend

• I've followed the CSS/JS and React guidelines.
• I've followed the web accessibility guidelines.
• I've followed the user interface guidelines.

Reminder

By using GitHub, you have already agreed to the GitHub’s Terms of Service including that:

1. You license your contribution under the same terms as the current repository’s license.
2. You agree that you have the right to license your contribution under the current repository’s license.

[slint]

Suggested change

	"schedule": timedelta(minutes=60),
	"schedule": timedelta(hours=4),

minor: a bit arbitrary, but since the underlying query is also performing updates to the database, I think seeing updates 2-3 times during working hours is good enough.

[lnielsen]

Note, the task doesn't update the index, just the db. Probably the task https://github.com/inveniosoftware/invenio-accounts/blob/24f9ac29879f2daa7e49c94c0370444fc4dc89aa/invenio_accounts/tasks.py#L81 should be updated to also index the domains that it changes.

[carlinmack]

FYI chose hourly as this comment says it ideally should be run regularly so that the changeset doesn't get too big https://github.com/inveniosoftware/invenio-accounts/blob/master/invenio_accounts/tasks.py#L123

Would probably be good to change it so that it doesn't require small changesets but I would like for some of my PRs to not result in fixing a larger issue 😅

[slint]

Once we've updated the task to also reindex (using the bulk queue), I would say we merge and see if it needs adjustments in terms of the regularity, bulk size, or similar.

[carlinmack]

updating indexed shelved here inveniosoftware/invenio-accounts#502

[zubeydecivelek]

LGTU! Peer reviewed witj @jrcastro2