[PR #5751] [CLOSED] fix: add missing password when using redis in sentinel mode #13052

Closed
opened 2025-12-29 22:25:09 +01:00 by adam · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/netbox-community/netbox/pull/5751
Author: @nerzhul
Created: 2/4/2021
Status: Closed

Base: developHead: patch-1


📝 Commits (1)

  • be1b6b6 fix: add missing password when using redis in sentinel mode

📊 Changes

1 file changed (+1 additions, -0 deletions)

View changed files

📝 netbox/netbox/settings.py (+1 -0)

📄 Description

Fixes: N/A

If you use redis with sentinel and authentication, cacheops has no information about the password, whereas without sentinel it's fine.

Here is the generated runtime issue

/opt/netbox/venv/lib/python3.8/site-packages/cacheops/redis.py:19: RuntimeWarning: The cacheops cache is unreachable! Error: Authentication required.
  warnings.warn("The cacheops cache is unreachable! Error: %s" % e, RuntimeWarning)

publishing the password field solve the issue


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/netbox-community/netbox/pull/5751 **Author:** [@nerzhul](https://github.com/nerzhul) **Created:** 2/4/2021 **Status:** ❌ Closed **Base:** `develop` ← **Head:** `patch-1` --- ### 📝 Commits (1) - [`be1b6b6`](https://github.com/netbox-community/netbox/commit/be1b6b6aa3c0ad0880f72a69da70a8c31e9bc295) fix: add missing password when using redis in sentinel mode ### 📊 Changes **1 file changed** (+1 additions, -0 deletions) <details> <summary>View changed files</summary> 📝 `netbox/netbox/settings.py` (+1 -0) </details> ### 📄 Description <!-- Thank you for your interest in contributing to NetBox! Please note that our contribution policy requires that a feature request or bug report be opened for approval prior to filing a pull request. This helps avoid wasting time and effort on something that we might not be able to accept. Please indicate the relevant feature request or bug report below. IF YOUR PULL REQUEST DOES NOT REFERENCE AN ACCEPTED BUG REPORT OR FEATURE REQUEST, IT WILL BE MARKED AS INVALID AND CLOSED. --> ### Fixes: N/A <!-- Please include a summary of the proposed changes below. --> If you use redis with sentinel and authentication, cacheops has no information about the password, whereas without sentinel it's fine. Here is the generated runtime issue ``` /opt/netbox/venv/lib/python3.8/site-packages/cacheops/redis.py:19: RuntimeWarning: The cacheops cache is unreachable! Error: Authentication required. warnings.warn("The cacheops cache is unreachable! Error: %s" % e, RuntimeWarning) ``` publishing the password field solve the issue --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
adam added the pull-request label 2025-12-29 22:25:09 +01:00
adam closed this issue 2025-12-29 22:25:09 +01:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/netbox#13052