Skip to content

Fix/destroy state - #509

Merged
MohcineTor merged 7 commits into
mainfrom
fix/destroy_state
Aug 7, 2026
Merged

Fix/destroy state#509
MohcineTor merged 7 commits into
mainfrom
fix/destroy_state

Conversation

@MohcineTor

Copy link
Copy Markdown
Collaborator

@MohcineTor
MohcineTor requested a review from sellisd July 31, 2026 11:03
Comment thread Babylon/utils/environment.py Dismissed
Comment thread Babylon/utils/kubernetes_state.py Dismissed
Comment thread Babylon/utils/kubernetes_state.py Dismissed

@sellisd sellisd left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Seems ok to me, this will fix the current issue. For a more long term solution how about moving out the destroy.py some functionality to a dedicated state command? e.g. to enable a user to sync, show or delete states and their contents?

@MohcineTor
MohcineTor merged commit 593cafb into main Aug 7, 2026
7 checks passed
@MohcineTor
MohcineTor deleted the fix/destroy_state branch August 7, 2026 17:10
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.

3 participants