Skip to content

Prevent e2e Helm install from re-owning pre-applied CRDs#42

Merged
Breee merged 2 commits into
mainfrom
copilot/fix-ci-e2e-job-failure
May 29, 2026
Merged

Prevent e2e Helm install from re-owning pre-applied CRDs#42
Breee merged 2 commits into
mainfrom
copilot/fix-ci-e2e-job-failure

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 29, 2026

The ci / e2e workflow failed during operator install because CRDs were applied via kubectl earlier in the job, then rendered again by the drop chart, triggering Helm ownership validation errors.

  • Root cause

    • E2E job mixed two CRD lifecycle paths in one run:
      • manual apply: kubectl apply -f config/crd/bases/
      • chart-managed CRDs from charts/drop templates
    • Helm install aborted on existing CRDs lacking release ownership metadata.
  • Change

    • Updated the e2e workflow Helm install command to explicitly disable CRD rendering from the drop chart in this path.
    • This aligns install behavior with the existing pre-apply CRD step and avoids ownership conflicts.
  • Workflow delta

    helm install drop charts/drop \
      --namespace drop-system \
      --create-namespace \
      --skip-crds \
      --set crds.install=false \
      --set image.repository=controller \
      --set image.tag=ci

Copilot AI changed the title [WIP] Fix failing GitHub Actions job ci / e2e Prevent e2e Helm install from re-owning pre-applied CRDs May 29, 2026
Copilot finished work on behalf of Breee May 29, 2026 21:20
Copilot AI requested a review from Breee May 29, 2026 21:20
@Breee Breee marked this pull request as ready for review May 29, 2026 21:49
@Breee Breee merged commit b4a345f into main May 29, 2026
6 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