Skip to content

fix: clear cluster dependencies#644

Draft
ApexWorm wants to merge 1 commit into
peteromallet:mainfrom
ApexWorm:fix/clear-cluster-dependencies
Draft

fix: clear cluster dependencies#644
ApexWorm wants to merge 1 commit into
peteromallet:mainfrom
ApexWorm:fix/clear-cluster-dependencies

Conversation

@ApexWorm

Copy link
Copy Markdown

Problem

desloppify plan cluster update --depends-on requires at least one cluster, so a stale dependency edge cannot be removed through the CLI. Passing an empty value becomes an invalid cluster name instead of clearing the list.

Fix

Add an explicit mutually exclusive --clear-depends-on flag. The update flow maps it to an empty dependency list, persists that list, and reports that dependencies were cleared. Existing manually constructed argument namespaces remain compatible.

Validation

  • python3 -m pytest -q desloppify/tests/commands/test_parser_groups_option_sections_direct.py desloppify/tests/commands/plan/test_cluster_ops_direct.py (15 passed)
  • python3 -m ruff check on changed files
  • Full suite: 5,811 passed, 3 skipped; two pre-existing review-prompt assertions fail identically on clean origin/main.

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.

1 participant