Chart: move crds chart logic after setting image

[defo89]

Proposed Changes

• This makes sure that crds chart also has the updated image values

Summary by CodeRabbit

• Chores
  • Updated Helm chart packaging workflow to generate multiple chart variants—one with CRDs included in the template and another with CRDs removed from the template.

[coderabbitai]

Walkthrough

The CI workflow .github/workflows/publish-chart.yml is modified to reorder and duplicate Helm chart packaging steps. The previously removed "Package Helm chart with crds folder in template" step is reintroduced after the image tagging step, followed by a new "Package Helm chart with removed crds folder from template folder" step, resulting in multiple sequential packaging executions with different CRD configurations.

Changes

Cohort / File(s)	Summary
CI/CD Workflow Configuration .github/workflows/publish-chart.yml	Reordered and duplicated Helm chart packaging steps with CRD-specific variants. Introduces multiple packaging executions with and without CRD folder configurations.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

• Chart: Set container image repository and tag #269: Modifies the same CI workflow file and affects the sequence around "Prepare CRDs" and chart packaging steps.

Suggested labels

size/S

Suggested reviewers

• hardikdr

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name	Status	Explanation	Resolution
Description check	❓ Inconclusive	The description provides context for the change but is incomplete compared to the template structure.	Consider expanding the description to include additional bullet points and link any related issues using 'Fixes #' format if applicable.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately describes the main change: reordering the crds chart logic to run after image setting.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch chart-fix-crds-chart

---

No actionable comments were generated in the recent review. 🎉

Tip

Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}