Skip to content

fix(ci): create the chart index workspace before running cr index - #20

Merged
pfernandes21 merged 1 commit into
masterfrom
devin/fix-chart-releaser-index-dir
Aug 6, 2026
Merged

pfernandes21 merged 1 commit into
masterfrom
devin/fix-chart-releaser-index-dir

Conversation

@Vervious

@Vervious Vervious commented Aug 6, 2026

Copy link
Copy Markdown

Tracking issue

Related to #19

Why are the changes needed?

The index regeneration step added in #19 fails on master:

Updating index .cr-index/index.yaml
Error: open .cr-index/index.yaml288038684: no such file or directory

The step does rm -rf .cr-index .cr-release-packages but only recreates .cr-release-packages. cr index writes its output via a temporary suffixed file next to --index-path and does not create that parent directory itself, so the write fails.

Everything before that point worked: cr v1.6.1 installed, the origin/gh-pages worktree was prepared, and all seven release packages were downloaded and inspected including flyte-binary-v0.1.11.tgz. So flyte-binary v0.1.11 is still missing from the published index.

What changes were proposed in this pull request?

-          mkdir -p .cr-release-packages
+          mkdir -p .cr-index .cr-release-packages

How was this patch tested?

Release Charts only runs on push to master, and #19's local verification passed only because it ran in a working tree that already had .cr-index — so this time the workflow's own shell was tested rather than an adaptation of it.

Both run: blocks were extracted verbatim from the committed workflow and executed in separate clean clones, with only the Actions environment stubbed (GITHUB_REPOSITORY, GITHUB_REPOSITORY_OWNER, RUNNER_TEMP, GITHUB_PATH, CR_TOKEN/GH_TOKEN, CHART_RELEASER_VERSION), $GITHUB_PATH applied between steps, and the final --push flag dropped so gh-pages is untouched.

At origin/master the harness reproduces the CI failure exactly:

before_status=1
Found flyte-binary-v0.1.11.tgz
Updating index .cr-index/index.yaml
Error: open .cr-index/index.yaml288038684: no such file or directory

At this branch it succeeds:

after_status=0
Found flyte-binary-v0.1.11.tgz
Updating index .cr-index/index.yaml

The clean clones had no origin/gh-pages ref, so the step's explicit fetch path was exercised too. Walking the rest of the step in that harness surfaced no other dependency on pre-existing local state.

Labels

fixed

Related PRs

Link to Devin session: https://app.devin.ai/sessions/137e980b425d43668660198d80b6e21d
Requested by: @Vervious

Assisted-by: Devin:claude-sonnet-4.5
Co-Authored-By: benchan <ben@vervious.com>
@Vervious Vervious self-assigned this Aug 6, 2026
@devin-ai-integration

Copy link
Copy Markdown

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment, CI, and merge conflict monitoring

@pfernandes21
pfernandes21 merged commit 6bac6f6 into master Aug 6, 2026
45 of 46 checks passed
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.

2 participants