Project

Profile

Help

Issue #9070

Remove Redis from status information if unused

Added by ekohl 2 months ago. Updated 21 days 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

Unless content caching or the RQ tasking system is used, Redis isn't needed. However, it is unconditionally included in the status views.


Related issues

Blocks Pulp - Issue #8997: Ensure that Pulp can function without RedisNEW<a title="Actions" class="icon-only icon-actions js-contextmenu" href="#">Actions</a>
Copied to Pulp - Backport #9085: Backport 9070 "Remove Redis from status information if unused" to 3.14.zCLOSED - CURRENTRELEASE

<a title="Actions" class="icon-only icon-actions js-contextmenu" href="#">Actions</a>

Associated revisions

Revision 8ab8afb4 View on GitHub
Added by Ewoud Kohl van Wijngaarden 2 months ago

Conditionally include Redis connection information

Redis is only used when cache is enabled or RQ tasking is used. In other cases, the connection is not needed. We've changed the Foreman installer to only install Redis if needed, just to stop wasting resources1. However, this leads to warnings.

This PR changes the status view to only include Redis if it's used.

closes #9070 https://pulp.plan.io/issues/9070

History

#1 Updated by ekohl 2 months ago

  • Blocks Issue #8997: Ensure that Pulp can function without Redis added

#2 Updated by pulpbot 2 months ago

  • Status changed from NEW to POST

#3 Updated by Anonymous 2 months ago

  • Status changed from POST to MODIFIED

#4 Updated by dalley 2 months ago

  • Sprint/Milestone set to 3.15.0

#5 Updated by dalley 2 months ago

  • Copied to Backport #9085: Backport 9070 "Remove Redis from status information if unused" to 3.14.z added

#6 Updated by pulpbot 21 days ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Please register to edit this issue

Also available in: Atom PDF