nextcloud / nextcloud/docker

Adding support for DBINDEX redis config value

Open
#2,593 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Shell
Stars
7.4k
Forks
1.9k
PR merge metrics
No merged PRs in 30d

Description

As discussed in #2288, #2236 and #1286. It would be usefull to add the possibility to set dbindex parameter if needed.
Even if it is not the most usefull parameter of redis, it is still supported by redis and users may want to use it.
The default value of dbindex that is already used silently is 0.
I suggest to add this possibilty while keeping the default at 0 in order to prevent current users from any changes and let the possibility to those who want it to customize this parameter via an environment variable.

As the #2288 is a little old I will propose a new PR.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by reviewing the repository's Docker image configuration and how Redis environment variables are handled, then read discussions #2288, #2236, and #1286 for the expected behavior. Done means users can set the Redis database index through an environment variable while the default remains 0 and existing users are unaffected.

Written by the indexing model from the issue text.

Assessment

Tech stack
docker, redis, shell
Domain
devops, infrastructure
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Quiet
Clarity
Mostly clear
Newbie friendliness
62/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.