Project

Profile

Help

Issue #8711

closed

Database access is needed during collectstatic

Added by ekohl over 3 years ago. Updated over 3 years ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Assignee:
-
Category:
-
Sprint/Milestone:
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

Related to Pulp - Backport #8712: Backport "Database access is needed during collectstatic" to 3.11CLOSED - CURRENTRELEASEbmbouter

Actions
Actions #1

Updated by pulpbot over 3 years ago

  • Status changed from NEW to POST

Added by Ewoud Kohl van Wijngaarden over 3 years ago

Revision 3669a65d | View on GitHub

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

Actions #2

Updated by Anonymous over 3 years ago

  • Status changed from POST to MODIFIED
Actions #3

Updated by ekohl over 3 years ago

  • Related to Backport #8712: Backport "Database access is needed during collectstatic" to 3.11 added
Actions #5

Updated by dalley over 3 years ago

  • Sprint/Milestone set to 3.13.0
Actions #6

Updated by pulpbot over 3 years ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Also available in: Atom PDF