Project

Profile

Help

Issue #9070

closed

Remove Redis from status information if unused

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

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 if Redis was configured and available when Pulp starts, but has an availability issue while Pulp is runningCLOSED - CURRENTRELEASElmjachkyActions
Copied to Pulp - Backport #9085: Backport 9070 "Remove Redis from status information if unused" to 3.14.zCLOSED - CURRENTRELEASEmdellweg

Actions
Actions #1

Updated by ekohl over 3 years ago

  • Blocks Issue #8997: Ensure that Pulp can function if Redis was configured and available when Pulp starts, but has an availability issue while Pulp is running added
Actions #2

Updated by pulpbot over 3 years ago

  • Status changed from NEW to POST

Added by Ewoud Kohl van Wijngaarden over 3 years ago

Revision 8ab8afb4 | View on GitHub

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

Actions #3

Updated by Anonymous over 3 years ago

  • Status changed from POST to MODIFIED
Actions #4

Updated by dalley over 3 years ago

  • Sprint/Milestone set to 3.15.0
Actions #5

Updated by dalley over 3 years ago

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

Updated by pulpbot over 3 years ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Also available in: Atom PDF