External versions: keep build log

Currently, when a PR is closed/merged, we delete the version, and it deletes the build objects with it, but we do keep the files in storage. The problem with this is that it feels weird lo lose the information from the build after the PR is closed.

• One idea is to not cascade delete the build objects, but do we want to do the same behavior for normal versions?
• If we delete the version and left it as null, we still need to save some extra information in the build model, like the version_slug and version_name (version.verbose_name). This is to allow us to link back to the PR (the view docs button wouldn’t be available here, or we can save that as well).

With the last sentence there is the additional question… Do we want to delete the docs too? Currently, we don’t delete the docs, they are kept forever. Should we delete them after x days? This probably something we want to do in .com, as we shouldn’t keep private docs around forever.

Some ideas:

• Delete the version, but don’t delete the build object, the extra fields should be able to link to the original PR and build docs (docs are kept).
• Don’t delete the version when the PR is closed, but mark it as inactive. Have a task delete all external versions marked as inactive and where the last edited date is greater than x days, but set the relationship of the build to null, so it doesn’t get deleted (here the docs are deleted as well).
• The same as above, but delete the build objects too.
• Keep both, version/builds and docs, but allow users to delete them manually, additionally have an automation rule to delete versions from closed PRs.

[ericholscher]

Edit: Misunderstood

I think we want to keep the docs around, but perhaps for some limited time? Eventually this will start costing us real money, so probably after 3-6 months we delete it? Maybe we keep Versions & Docs around for 3 months, and then delete them together.

[agjohnson]

Yeah, filters are already a part of the new theme on the build list page and version list page. I’d also like to make a special filter eventually which is latest/stable at the top of the queryset filter. I’ll note to add a filter for pull request builds if there isn’t already a filter on version/builds