Skip to content

Conversation

@AshishKingdom
Copy link

@AshishKingdom AshishKingdom commented Nov 3, 2025

The tag 3.7.1 does not exist https://hub.docker.com/r/signoz/zookeeper/tags
So, a version bump is needed. Currently image pulling is failing.

Summary by CodeRabbit

  • Chores
    • Updated ZooKeeper to version 3.9.3.

@AshishKingdom AshishKingdom requested a review from a team as a code owner November 3, 2025 09:03
@coderabbitai
Copy link

coderabbitai bot commented Nov 3, 2025

Walkthrough

ZooKeeper image tag updated from version 3.7.1 to 3.9.3 in the Helm chart values configuration file. No other configuration fields or metadata were modified.

Changes

Cohort / File(s) Change Summary
ZooKeeper Helm Chart
charts/zookeeper/values.yaml
Updated image tag from 3.7.1 to 3.9.3

Estimated Code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

  • Verify that ZooKeeper version 3.9.3 is a valid release and compatible with the current deployment setup
  • Confirm the image digest (if pinned) aligns with the new tag version, or validate that digest-less pull is acceptable

Poem

🐰 A hop, a skip, the version flew,
From 3.7.1 to 3.9, it's new!
ZooKeeper climbs the Helm chart tree,
Minor bumps to compatibility! 🎉

Pre-merge checks and finishing touches

✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check ✅ Passed The pull request title "Update Zookeeper image tag to 3.9.3" directly and accurately reflects the main change in the changeset, which is updating the ZooKeeper image tag from 3.7.1 to 3.9.3 in charts/zookeeper/values.yaml. The title is concise, specific, and avoids vague or generic language. A teammate scanning the repository history would immediately understand that this PR addresses a version update for the ZooKeeper image dependency.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 9ee2387 and b1f9692.

📒 Files selected for processing (1)
  • charts/zookeeper/values.yaml (1 hunks)
🔇 Additional comments (1)
charts/zookeeper/values.yaml (1)

79-79: LGTM! Fix resolves missing image tag issue with proper version upgrade.

ZooKeeper 3.9.x is forward-compatible with 3.7.x clients and servers, and no special upgrade procedure is needed. ZooKeeper 3.9.3 includes security fixes (CVE-2024-6763 and dependency upgrades for Netty). The change is minimal and safe—only the image tag is updated; all other image configuration remains unchanged.

Recommend deploying this to a test environment to verify the StatefulSet initializes successfully with the new tag.


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.

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.

1 participant