Skip to content

docs: Add retirement concept doc - #1488

Open
bmtcril wants to merge 3 commits into
mainfrom
bmtcril/user_retirement_concept
Open

docs: Add retirement concept doc#1488
bmtcril wants to merge 3 commits into
mainfrom
bmtcril/user_retirement_concept

Conversation

@bmtcril

@bmtcril bmtcril commented Jul 27, 2026

Copy link
Copy Markdown
Contributor

Documentation for the user retirement feature:

  • A high level concept doc
  • An operator how-to for configuring and operating the feature, including both Tutor and non-tutor instructions

Comment thread source/site_ops/concepts/user_retirement.rst Outdated
===================

After a deletion request, the account sits in a waiting state (called
``PENDING``) for a configurable number of *cool-off days*. During this window a

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How are cool-off days configured?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure that belongs in this doc, but it is covered in the new how-to I just added. I can link it here if you think that makes sense.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it'd be good to reference the how-to here, yes

The retirement process
======================

The retirement process is the batch job that actually carries out deletions.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I assume the retirement process is not enabled by default?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The process itself isn't, but the UI to delete your account is

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Makes sense, maybe this could be clarified with something like

Suggested change
The retirement process is the batch job that actually carries out deletions.
The retirement process is a manually-configured batch job that actually carries out deletions.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's somewhat confusing in that if you install the Tutor plugin and are on k8s it will actually create a cron for it and start running the pipeline. In all other cases you need to do something manual to kick the job. For this doc I was just trying to outline the concept of the process so didn't want to have to get into the weeds of various ways things might be configured.

Deploying user retirement with Tutor
************************************

Setting up user retirement by hand involves several steps: creating a dedicated

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what do you mean "by hand" here?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Manually, vs using the Tutor plugin. I can clarify

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The section title here is "Deploying user retirement with Tutor", should it just be "without Tutor" then?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've removed "by hand" since it remains true that the process needs those steps whether the Tutor plugin does them or an operator configures it some other way. Does it read more clearly now?

+--------------+-------------------------------+----------------+--------------------------------+
| Review Date | Working Group Reviewer | Release |Test situation |
+--------------+-------------------------------+----------------+--------------------------------+
| | | | |

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would fill this out as accurate - is it accurate for Verawood, or only currently on main (eg Willow)?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should be up to date going back several releases. The process itself hasn't changed in years, but the Tutor plugin only goes back a few releases AFAIK.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would you be the working group reviewer here?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Gotcha. Usually I put the person with expertise here - so you - or you could put "Data WG" if you don't want your own name in the docs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants