Why
make release aborts while packaging the openhands chart with Error: Chart.yaml file is missing whenever a hollow directory exists under charts/ — one that contains no Chart.yaml, only gitignored leftovers. This happens naturally after switching branches: on main, automation and runtime-api are top-level charts, but on older release branches they are embedded in charts/openhands/charts/, so checking out such a branch removes the tracked files while the gitignored vendored charts/*.tgz files keep the directories alive.
The Makefile's dependency-rewrite loop only checks that the sibling directory exists before pointing a dependency at it:
if [ -d $(CHARTDIR)/$$dep ]; then \
yq -i "(.dependencies[] | select(.name == \"$$dep\")).repository = \"file://../$$dep\"" ...so the empty-repository embedded-subchart deps get rewritten to file://../automation / file://../runtime-api, and helm package -u fails resolving them. The check should be [ -f $(CHARTDIR)/$$dep/Chart.yaml ], which is the actual precondition for a usable local chart.
Validation
- Reproduced on a
cloud-1.40.x backport branch: with hollow charts/automation and charts/runtime-api present (only gitignored .tgz leftovers inside), make build/openhands-0.7.85.tgz fails with Error: Chart.yaml file is missing; after deleting the two directories the same target packages successfully.
This issue was drafted by an AI agent on behalf of the user.
Why
make releaseaborts while packaging the openhands chart withError: Chart.yaml file is missingwhenever a hollow directory exists undercharts/— one that contains noChart.yaml, only gitignored leftovers. This happens naturally after switching branches: onmain,automationandruntime-apiare top-level charts, but on older release branches they are embedded incharts/openhands/charts/, so checking out such a branch removes the tracked files while the gitignored vendoredcharts/*.tgzfiles keep the directories alive.The Makefile's dependency-rewrite loop only checks that the sibling directory exists before pointing a dependency at it:
if [ -d $(CHARTDIR)/$$dep ]; then \ yq -i "(.dependencies[] | select(.name == \"$$dep\")).repository = \"file://../$$dep\"" ...so the empty-
repositoryembedded-subchart deps get rewritten tofile://../automation/file://../runtime-api, andhelm package -ufails resolving them. The check should be[ -f $(CHARTDIR)/$$dep/Chart.yaml ], which is the actual precondition for a usable local chart.Validation
cloud-1.40.xbackport branch: with hollowcharts/automationandcharts/runtime-apipresent (only gitignored.tgzleftovers inside),make build/openhands-0.7.85.tgzfails withError: Chart.yaml file is missing; after deleting the two directories the same target packages successfully.This issue was drafted by an AI agent on behalf of the user.