Skip to content

Add ActiveCheck initialization ownership switch#2757

Open
fabrizio2210 wants to merge 3 commits into
mainfrom
agent/sched-1967-classify-initialization-checks
Open

Add ActiveCheck initialization ownership switch#2757
fabrizio2210 wants to merge 3 commits into
mainfrom
agent/sched-1967-classify-initialization-checks

Conversation

@fabrizio2210

@fabrizio2210 fabrizio2210 commented Jul 17, 2026

Copy link
Copy Markdown
Collaborator

Context

SCHED-1967 moves mandatory cluster initialization into the Slurm-cluster HelmRelease before ActiveChecks diagnostics are decoupled from Terraform apply. This compatibility-first PR prepares the ActiveChecks chart while preserving existing behavior by default.

Changes

  • add clusterInitialization.managedBySlurmCluster, disabled by default
  • suppress legacy initialization ActiveChecks and their dependencies when enabled
  • retain the force and dry-run jail maintenance checks
  • add Helm rendering coverage for both ownership modes

Validation

  • helm unittest helm/*/ (356 tests passed)
  • default rendered manifests are unchanged from HEAD
  • git diff --check

@fabrizio2210
fabrizio2210 marked this pull request as ready for review July 17, 2026 11:17
@fabrizio2210 fabrizio2210 added terraform Changes in Terraform recipe and removed terraform Changes in Terraform recipe labels Jul 17, 2026
@fabrizio2210 fabrizio2210 added the helm Functional changes in Helm charts label Jul 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fix helm Functional changes in Helm charts

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant