Propagate chart version to regcache dependency in ace-installer#1299
Merged
Conversation
The contents-% target bumps a subchart's version and updates the matching dependency constraint in ace, acaas, and platform-opscenter. But regcache is a dependency only of ace-installer and ace-installer-certified, which were missing from the list. As a result, bumping regcache (via make update-charts) left ace-installer's dependency pinned to the previous version, so 'helm dependency update charts/ace-installer' failed with: can't get a valid version for 1 subchart(s): "regcache" (repository "file://../regcache", version "v2026.6.19") Add ace-installer and ace-installer-certified to the propagation list. Signed-off-by: Tamal Saha <tamal@appscode.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The `v2026.7.10` release (failing job) failed with:
Root cause
The
contents-%target in theMakefilebumps a subchart’s version and propagates it into the matching dependency constraint of parent charts — but onlyace,acaas, andplatform-opscenter:regcacheis a dependency only oface-installerandace-installer-certified(none of the three listed charts use it). So whenmake update-chartsbumpsregcacheto the new release version, theace-installerdependency stays pinned to the previous version, andhelm dependency update charts/ace-installercan no longer find it.Fix
Add
ace-installerandace-installer-certifiedto the propagation list so theirregcachedependency constraint is bumped alongside the chart. (Themap(select(.name == "$*"))is a no-op for charts that do not depend on$*, so the existing entries stay safe.)