Skip to content

feat: allow force for cluster upgrades#168

Draft
amdove wants to merge 3 commits intomainfrom
cluster-maint
Draft

feat: allow force for cluster upgrades#168
amdove wants to merge 3 commits intomainfrom
cluster-maint

Conversation

@amdove
Copy link
Contributor

@amdove amdove commented Mar 9, 2026

Description

  • Add force_maintenance config option for both AWS EKS and Azure AKS clusters
  • AWS: Sets ForceUpdateVersion to override upgrade-blocking readiness checks (EKS Insights)
  • Azure: Sets ForceUpgrade to bypass PodDisruptionBudget constraints during upgrades

Test plan

  • Verify config parsing with force_maintenance: true in ptd.yaml
  • Test Azure AKS upgrade with force_maintenance enabled
  • Test AWS EKS upgrade with force_maintenance enabled

Code Flow

Category of change

  • Bug fix (non-breaking change which fixes an issue)
  • Version upgrade (upgrading the version of a service or product)
  • New feature (non-breaking change which adds functionality)
  • Build: a code change that affects the build system or external dependencies
  • Performance: a code change that improves performance
  • Refactor: a code change that neither fixes a bug nor adds a feature
  • Documentation: documentation changes
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist

  • I have reviewed my own diff and added inline comments on lines I want reviewers to focus on or that I am uncertain about

Copy link
Contributor

@ian-flores ian-flores left a comment

Choose a reason for hiding this comment

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

Tested the AWS EKS path end-to-end on ganso01-staging — force upgrade worked correctly to bypass session PDBs during a 1.32 → 1.33 node group upgrade.

The unified force_maintenance naming and Azure support look good.

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