Actions
Issue #8711
closed
Database access is needed during collectstatic
Start date:
Due date:
Estimated time:
Severity:
2. Medium
Version:
Platform Release:
OS:
Triaged:
No
Groomed:
No
Sprint Candidate:
No
Tags:
Sprint:
Quarter:
Description
Since #7487 (9a9a06f1e37f5367e36a24b9aa3aca6ca78dab53) database access always happens. It now does gracefully fall back, but it's no longer possible to run:
pulpcore-manager collectstatic pulpcore-manager migrate
My understanding is that collectstatic never needs the database so it should skip these checks.
Related issues
Updated by pulpbot over 3 years ago
- Status changed from NEW to POST
Added by Ewoud Kohl van Wijngaarden over 3 years ago
Updated by Anonymous over 3 years ago
- Status changed from POST to MODIFIED
Applied in changeset pulpcore|3669a65d07278a3c455050015893517fbb6f9871.
Updated by ekohl over 3 years ago
- Related to Backport #8712: Backport "Database access is needed during collectstatic" to 3.11 added
Updated by pulpbot over 3 years ago
Updated by pulpbot over 3 years ago
- Status changed from MODIFIED to CLOSED - CURRENTRELEASE
Actions
.
Skip allowed content checks on collectstatic
Since 9a9a06f1e37f5367e36a24b9aa3aca6ca78dab53 DB access is always needed. In later commits this was extended with additional checks.
However, collectstatic should be a fully offline operation that doesn't access the network. As such, content checking is not needed.
Doing this allows the Foreman/Katello installer to run it more freely. The alternative is to add additional ordering. However, I'd argue that this is better and also slightly faster.
Fixes #8711