Skip to content

Fix/helm provenance first layer - #2205

Open
rg3122 wants to merge 3 commits into
fluxcd:mainfrom
rg3122:fix/helm-provenance-first-layer
Open

rg3122 wants to merge 3 commits into
fluxcd:mainfrom
rg3122:fix/helm-provenance-first-layer

Conversation

@rg3122

@rg3122 rg3122 commented Oct 23, 2026

Copy link
Copy Markdown

Summary

Fixes Helm OCI chart extraction when the provenance file appears before the chart content layer.

When spec.layerSelector is not configured, the controller now checks whether the first layer is a Helm provenance layer. If it is, the controller selects the Helm chart content layer when available. Other OCI artifacts and existing chart-first artifacts retain their current behavior, and an explicit layer selector continues to take precedence.

  • Adds a regression test for provenance-first layer ordering.
  • Adds a regression test confirming chart-first layer ordering remains unchanged.
  • Limits the behavior change to the default layer-selection path.

Fixes #2198.

Testing

  • Added unit tests using in-memory OCI image layers.
  • Verified the implementation and regression tests are saved in the fork.
  • The Go test suite has not been run in this environment because repository cloning and dependency downloads were unavailable.
  • CI should run the controller tests before merging.

Signed-off-by: Riya <riyagoel064@gmail.com>
Signed-off-by: Riya <riyagoel064@gmail.com>
@rg3122
rg3122 force-pushed the fix/helm-provenance-first-layer branch from f570602 to 60ff205 Compare October 24, 2026 06:48
Signed-off-by: Riya <rriya_be20@thapar.edu>
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.

OCIRepository: pick the Helm chart layer automatically when the first layer is a provenance file

1 participant