Skip to content

Add schedulerName and runtimeClassName support to component workloads - #1461

Merged
benbz merged 2 commits into
element-hq:mainfrom
somaz94:add-scheduler-runtimeclass
Jul 20, 2026
Merged

Add schedulerName and runtimeClassName support to component workloads#1461
benbz merged 2 commits into
element-hq:mainfrom
somaz94:add-scheduler-runtimeclass

Conversation

@somaz94

@somaz94 somaz94 commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

Adds schedulerName and runtimeClassName as pod-scheduling knobs on every
component workload, plus matching global values applied to all components
(a per-component value overrides the global one).

This mirrors the existing priorityClassName (#1438) and affinity (#1442)
support, filling out the remaining common pod-spec scheduling fields. Both are
plain string scalars rendered via the same coalesce(component, global) pattern
in _pods.tpl.

Validation (local):

  • Regenerated values.yaml + values.schema.json from source fragments
    (scripts/assemble_helm_charts_from_fragments.sh); git diff clean.
  • New manifest tests test_pod_schedulerName.py + test_pod_runtimeClassName.py
    pass (full pytest run: 144 passed).
  • reuse lint: 656/656 files compliant.

A newsfragment will be added once the PR number is assigned.

somaz94 added a commit to somaz94/somaz94 that referenced this pull request Jul 20, 2026
@github-actions

Copy link
Copy Markdown

dyff of changes in rendered templates of CI manifests

No changes in rendered templates

@somaz94
somaz94 marked this pull request as ready for review July 20, 2026 08:25
@somaz94
somaz94 requested a review from a team as a code owner July 20, 2026 08:25

@benbz benbz left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for implementing these

@benbz
benbz merged commit 2fa8112 into element-hq:main Jul 20, 2026
97 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