feat(chart): Add ability to change security and resources for the Helm chart

[treydock]

Description

Add several new configuration options for Helm.

• Ensure all Job and Deployment resources can support securityContexts
• Ensure Jobs and Deployments can have resources defined

Checklist

Note: Ensure your PR title follows the Conventional Commits format (e.g., feat(scheduler): add new feature)

• Self-reviewed
• Added/updated tests (if needed)
• Updated documentation (if needed)

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 76d41eb8-3aad-43c0-a20e-b989030e32ae

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

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.}

[enoodle]

@treydock Why add resources configuration to those helm jobs and not just set some constant? I am not sure that it is really helpful to play with this value and maybe we should keep it a little bit more simple by just setting a value.

[treydock]

Why add resources configuration to those helm jobs and not just set some constant? I am not sure that it is really helpful to play with this value and maybe we should keep it a little bit more simple by just setting a value.

Setting some value is good but I'd imagine it's possible the value set would need to be overridden. I figured leaving empty leaves it undefined like before but allows sites that want resource limits to be able to set them. We use Kyverno to ensure things we deploy are setting resource requests and limits so having some value set is necessary but also can't assume a hardcoded default is always sufficient I'd imagine.