Skip to content

refactor(hub): pin hub version to 0.8.43#130

Open
maxju wants to merge 2 commits into
masterfrom
refactor-pin-hub
Open

refactor(hub): pin hub version to 0.8.43#130
maxju wants to merge 2 commits into
masterfrom
refactor-pin-hub

Conversation

@maxju
Copy link
Copy Markdown
Member

@maxju maxju commented May 21, 2026

testing flux pr previews

Summary by CodeRabbit

  • Chores
    • Updated container image version to 0.8.44.

Review Change Stack

@maxju maxju added the deploy/preview Deploy 1 hub and 2 nodes to max dev preview cluster label May 21, 2026
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 21, 2026

📝 Walkthrough

Walkthrough

Updates the Helm chart for flame-hub to pin the container image version from latest to 0.8.44, replacing a floating tag reference with a fixed release version in the values configuration.

Changes

Image Version Update

Layer / File(s) Summary
Flame Hub container image version pinning
charts/flame-hub/values.yaml
The image.tag field is updated from latest to the fixed version 0.8.44, ensuring reproducible deployments with a pinned container version.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Suggested reviewers

  • tada5hi

Poem

🐰 From latest into certain days,
The flame-hub now shows clearer ways,
Version pinned at zero-eight,
No surprises left to fate! 🔥

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Title check ⚠️ Warning The PR title states version 0.8.43, but the actual change in values.yaml updates the image tag to 0.8.44, creating a discrepancy. Update the PR title to 'refactor(hub): pin hub version to 0.8.44' to match the actual version change in the code.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

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

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch refactor-pin-hub

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.

Copy link
Copy Markdown

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🧹 Nitpick comments (1)
charts/flame-hub/values.yaml (1)

176-176: ⚡ Quick win

Consider pinning the authup version for consistency.

While the main hub image is now pinned to 0.8.44, the authup subchart still uses tag: "latest" (line 176). For consistency and to ensure fully reproducible deployments across all components, consider pinning this version as well.

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@charts/flame-hub/values.yaml` at line 176, The authup subchart currently uses
a floating image tag ("tag: \"latest\"") which prevents reproducible
deployments; update the authup subchart's image tag to a fixed version string
(match the hub's pinned style, e.g., "0.8.44" or the appropriate authup release)
by replacing the value for the key tag under the authup block in values.yaml so
the authup image is pinned rather than "latest".
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@charts/flame-hub/values.yaml`:
- Line 59: The values.yaml currently pins the Flame Hub image tag to 0.8.44
which appears missing on GHCR (image ghcr.io/privateaim/hub:0.8.44); either
publish the missing 0.8.44 tag to GHCR or update charts/flame-hub/values.yaml to
a published tag (or a digest) so deployments won't 404 on pull, and while here
consider pinning authup's image (currently using latest) to a specific tag for
consistency.

---

Nitpick comments:
In `@charts/flame-hub/values.yaml`:
- Line 176: The authup subchart currently uses a floating image tag ("tag:
\"latest\"") which prevents reproducible deployments; update the authup
subchart's image tag to a fixed version string (match the hub's pinned style,
e.g., "0.8.44" or the appropriate authup release) by replacing the value for the
key tag under the authup block in values.yaml so the authup image is pinned
rather than "latest".
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: a0be1d32-a192-45bd-894f-3e88fbb34356

📥 Commits

Reviewing files that changed from the base of the PR and between c6ae564 and ed6f280.

📒 Files selected for processing (1)
  • charts/flame-hub/values.yaml

Comment thread charts/flame-hub/values.yaml Outdated
@maxju maxju changed the title refactor(hub): pin hub version to 0.8.44 refactor(hub): pin hub version to 0.8.43 May 21, 2026
@maxju maxju added deploy/preview Deploy 1 hub and 2 nodes to max dev preview cluster and removed deploy/preview Deploy 1 hub and 2 nodes to max dev preview cluster labels May 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

deploy/preview Deploy 1 hub and 2 nodes to max dev preview cluster

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant