chore(deps): update ghcr.io/doublewordai/control-layer docker tag to v8#65
chore(deps): update ghcr.io/doublewordai/control-layer docker tag to v8#65renovate[bot] wants to merge 1 commit into
Conversation
fd7f700 to
0436b20
Compare
5eb4e20 to
c141f58
Compare
2bd8040 to
d5ed62a
Compare
cf184bf to
f073e2c
Compare
cfa92e1 to
f933869
Compare
47c8508 to
11612fd
Compare
5f753ae to
b3af825
Compare
b3af825 to
ba43472
Compare
There was a problem hiding this comment.
Summary
This PR is an automated dependency update from Renovate that bumps the appVersion in Chart.yaml from 0.25.0 to 8.46.0. This is a major version upgrade (0.x → 8.x) for the underlying ghcr.io/doublewordai/control-layer application image. The change affects the default image tag used in deployments since the template references .Chart.AppVersion when no explicit tag is provided.
Findings
-
Non-blocking
Chart.yaml:6: Major version upgrade (0.25.0 → 8.46.0) without accompanying documentation about breaking changes or migration requirements. Consider adding a note in the PR description or updating CHANGELOG.md with a link to the upstream application's release notes. -
Non-blocking
CHANGELOG.md: The changelog tracks chart version history but does not document application version updates. Consider adding an entry noting the appVersion bump to help users track what application version each chart release deploys. -
Nit
tests/control_layer_deployment_test.yaml:7-23: The test "should create control-layer deployment with default appVersion" only validates the image format regex but doesn't assert the specific version. Adding a version assertion would catch unintended appVersion changes during review. -
Nit
values.yaml:52: Theimage.tagis set to""(empty string), which relies on thedefault .Chart.AppVersionbehavior in the template. This is correct but could be clearer with a comment noting this intentional defaulting behavior.
Suggested next steps
This is a straightforward automated dependency update. Before merging:
- Verify the upstream control-layer v8 release notes for any breaking changes that might affect deployments (e.g., configuration schema changes, required environment variables, or database migration requirements).
- Optionally update CHANGELOG.md to document this appVersion update for user visibility.
- Ensure any downstream environments are prepared for the major version upgrade if they rely on the default image tag.
ba43472 to
18c0a58
Compare
There was a problem hiding this comment.
Summary
This PR updates the appVersion in Chart.yaml from 0.25.0 to 8.46.1, which controls the default Docker image tag for the control-layer deployment. This is an automated Renovate dependency update.
Verdict: The chart change itself is correct and follows the established pattern. However, this is a major version jump (0.x → 8.x) in the application version, which warrants verification that the new image version is compatible with the existing chart templates and default values before merging.
Research Notes
- Fetched renovate.json configuration confirming the appVersion is automatically updated from the Docker registry (
datasourceTemplate: docker) - Verified in
templates/deployment.yaml:42thatChart.AppVersionis used as the default image tag whenimage.tagis not explicitly set - Checked existing tests in
tests/control_layer_deployment_test.yaml- they validate image format but not specific version compatibility - Reviewed CHANGELOG.md showing historical appVersion updates have been routine
- Attempted to fetch available Docker tags from GHCR but authentication was required
Suggested Next Steps
-
Review control-layer application changelog (if available) for breaking changes between v0.25.0 and v8.46.1 - particularly around:
- Configuration file format changes
- Environment variable changes
- Health check endpoint changes (
/healthz) - Database schema migrations
-
Test the new image in a non-production environment to verify:
- Pod starts successfully with current default values
- Health probes pass
- Secret mounting works correctly
- ConfigMap configuration is compatible
-
Consider pinning
image.tagin production values files rather than relying on appVersion defaults if you need more control over rollout timing.
General Findings
No issues found with the actual code change. The single-line modification to Chart.yaml is the correct approach for updating the default container image version in a Helm chart.
39fac03 to
1e4d1f6
Compare
There was a problem hiding this comment.
Summary
This PR updates the appVersion in Chart.yaml from 0.25.0 to 8.46.3, representing a major version jump (v0 → v8) of the control-layer Docker image. The change is automatically generated by Renovate.
Verdict: Needs changes - While the dependency update itself is straightforward, there are several concerns with this major version bump that should be addressed before merging.
Research notes
- Fetched https://github.com/doublewordai/control-layer - confirms v8.46.3 was released today (May 7, 2026)
- Fetched https://docs.doubleword.ai/control-layer/ - Control Layer is an AI model gateway written in Rust
- The repository uses Renovate with Docker versioning for appVersion tracking (renovate.json lines 9-14)
- Image tag resolution in deployment templates uses
.Values.image.tag | default .Chart.AppVersion
Suggested next steps
- Blocking: Increment the chart
versionin Chart.yaml alongside this major appVersion change (Helm best practice) - Non-blocking: Review control-layer v8 release notes for breaking changes that might affect deployments
- Non-blocking: Consider testing the rendered Helm templates with the new version before deploying to production
General findings
Major version jump without changelog review
The jump from 0.25.0 to 8.46.3 crosses multiple major versions. Per semantic versioning, major version increments can include breaking changes. The PR description doesn't include any migration notes or changelog references. Teams deploying this should verify compatibility with their existing configurations, especially:
- Database schema migrations (control-layer uses PostgreSQL)
- Configuration file format changes
- API endpoint compatibility
- Authentication/session handling changes
Chart version should be incremented
Per Helm chart best practices, when updating appVersion (especially across major versions), the chart version should also be incremented. This allows users to distinguish between chart releases and ensures proper version tracking in Helm repositories.
b232907 to
9c5217d
Compare
There was a problem hiding this comment.
Summary
This PR updates the appVersion in Chart.yaml from 0.25.0 to 8.49.0, which controls the default Docker image tag for the control-layer service. This is a major version jump (0.x → 8.x) managed by Renovate bot.
The change is mechanically correct - only the single version field is modified, and the Helm templates will correctly use this new version as the default image tag. No code changes required; ready to merge pending operational validation.
Research notes
- Fetched the GitHub repository page to confirm this is a Helm chart for deploying the Doubleword control layer service
- Verified
templates/deployment.yaml:42uses.Chart.AppVersionas the default image tag whenimage.tagis not explicitly set - Confirmed
renovate.jsonis configured with regexManager to track Docker image versions via theappVersionfield - Test suite (
tests/control_layer_deployment_test.yaml) validates image format but not specific version compatibility
Suggested next steps
- Verify upstream changelog: Review release notes for control-layer versions 0.25.0 → 8.49.0 to identify any breaking changes that might require configuration updates (environment variables, config schema, database migrations, etc.)
- Test deployment in non-production environment: Deploy with the new image version and verify:
- Application starts successfully
- Health checks pass (
/healthzendpoint) - Database migrations run correctly (if applicable)
- Existing configuration remains compatible
- Check resource requirements: Major version jumps may change CPU/memory requirements - verify current resource limits are still appropriate
General findings
Non-blocking consideration: The test at tests/control_layer_deployment_test.yaml:7 is named "should create control-layer deployment with default appVersion" but only validates that the image matches the pattern ^ghcr\.io/doublewordai/control-layer:.*$ - it doesn't assert the specific version. This is acceptable for a generic test, but teams relying on tests to catch version mismatches may want to add a test that explicitly validates the Chart.yaml appVersion field propagates correctly to the deployment image.
This PR contains the following updates:
0.25.0→8.50.0Configuration
📅 Schedule: (UTC)
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
This PR was generated by Mend Renovate. View the repository job log.