Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Use PATHs to call clear_artifacts #4296

Merged
merged 1 commit into from Jun 25, 2018
Merged

Use PATHs to call clear_artifacts #4296

merged 1 commit into from Jun 25, 2018

Commits on Jun 25, 2018

  1. Use PATHs to call clear_artifacts 

    Use a list of PATHs Instead of using a ``Version.pk`` when calling
`clear_artifacts`. This is because in the ``Version.delete`` method,
the Version object is removed immediately after broadcasting the
``clear_artifacts`` task and that could produce a race condition.

With this change, we calculate all the PATHs needed _before_
broadcasting the task and _before_ deleting the object from the
database.
    @humitos
    humitos committed Jun 25, 2018
    Copy the full SHA
    8282885 View commit details
    Browse the repository at this point in the history