Skip to content

fix: pin redisinsight image to specific version#11625

Open
dsk-dev-ai wants to merge 4 commits intoappwrite:1.9.xfrom
dsk-dev-ai:fix-redisinsight-tag
Open

fix: pin redisinsight image to specific version#11625
dsk-dev-ai wants to merge 4 commits intoappwrite:1.9.xfrom
dsk-dev-ai:fix-redisinsight-tag

Conversation

@dsk-dev-ai
Copy link
Copy Markdown

What does this PR do?

This PR replaces the latest tag for the RedisInsight image with a fixed version.

Why?

Using latest can lead to:

  • Inconsistent environments
  • Unexpected breaking changes
  • Difficult debugging and rollback

Pinning the version ensures reproducibility and stability.

Changes

  • Updated:
    redis/redisinsight:latest → redis/redisinsight:2.54.0

Notes

Version selected from official Docker Hub as a stable release.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Mar 24, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 9f58b6b5-4ea8-41bf-a1d1-31c5240b5be5

📥 Commits

Reviewing files that changed from the base of the PR and between 44326eb and 7e9b8fc.

📒 Files selected for processing (1)
  • docker-compose.override.yml
🚧 Files skipped from review as they are similar to previous changes (1)
  • docker-compose.override.yml

📝 Walkthrough

Walkthrough

The docker-compose.override.yml file is updated to change the redis-insight service image from redis/redisinsight:latest to the pinned version redis/redisinsight:3.2.0. No other redis-insight configuration (restart policy, networks, environment, volumes, Traefik labels, or ports) was modified. This is a single-line image tag substitution.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: pinning the RedisInsight image from latest to a specific version (3.2.0).
Description check ✅ Passed The description is related to the changeset but contains outdated information mentioning version 2.54.0 when the actual commit uses 3.2.0.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps bot commented Mar 24, 2026

Greptile Summary

This PR pins the redis/redisinsight Docker image from latest to 2.54.0 in docker-compose.override.yml, a dev-only override file. This is a good hygiene improvement that ensures reproducible local development environments.

  • The change is confined to a dev-tools-only file (docker-compose.override.yml) and has no production impact.
  • Pinning the version prevents unexpected breakage from upstream image updates, which is best practice.
  • As a minor follow-up consideration, adminer (line 8) and mongo-express (line 22) in the same file also omit explicit version tags (defaulting to latest). Pinning those as well would make the entire file consistent with the principle applied here.

Confidence Score: 5/5

  • This PR is safe to merge — it is a single-line, low-risk change in a dev-only Docker Compose override file.
  • The change is minimal, well-scoped, and improves environment reproducibility. It only affects local development tooling, so there is no production risk. No logic or application code is modified.
  • No files require special attention.

Important Files Changed

Filename Overview
docker-compose.override.yml Pins redis/redisinsight from implicit latest to explicit 2.54.0; straightforward and correct change in a dev-only override file.

Reviews (1): Last reviewed commit: "fix: pin redisinsight image to specific ..." | Re-trigger Greptile

Copy link
Copy Markdown
Author

@dsk-dev-ai dsk-dev-ai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hello Sir, This PR Closes #11612 [As Fix Of That Issue]

@b-beyaz
Copy link
Copy Markdown

b-beyaz commented Mar 25, 2026

@dsk-dev-ai hi, i created #11612 this issue before you. i am following the process. When they assign the issue to me. I am gonna create a PR about the issue.

Also, the 2.54.0 tag is not the right choice for this issue. We must use 3.2.0 tag for the solution. You can check current tags in the https://hub.docker.com/r/redis/redisinsight/tags

@dsk-dev-ai
Copy link
Copy Markdown
Author

@dsk-dev-ai hi, i created #11612 this issue before you. i am following the process. When they assign the issue to me. I am gonna create a PR about the issue.

Also, the 2.54.0 tag is not the right choice for this issue. We must use 3.2.0 tag for the solution. You can check current tags in the https://hub.docker.com/r/redis/redisinsight/tags

Thanks for clearification, now plz suggest what I do ?

@b-beyaz
Copy link
Copy Markdown

b-beyaz commented Mar 26, 2026

In our case, the current tag for RedisInsight is latest, which makes it unstable for consistent usage. Instead, we should use a fixed version tag. According to our Docker registry, the latest stable RedisInsight version is 3.2.0, so we should use this tag to ensure consistency and reliability.

Replaced this: "image: redis/redisinsight:2.54.0"
With this: "image: redis/redisinsight:3.2.0"
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants