Skip to content

fix: handle provider status default cases across providers.#107

Open
garvitssoni wants to merge 3 commits intomainfrom
BREV-3225/deletion-stuck-at-deploying
Open

fix: handle provider status default cases across providers.#107
garvitssoni wants to merge 3 commits intomainfrom
BREV-3225/deletion-stuck-at-deploying

Conversation

@garvitssoni
Copy link
Copy Markdown

@garvitssoni garvitssoni commented Apr 7, 2026

Problem

Across several providers, unknown/transient status values were being coerced into concrete lifecycle states (commonly pending). During deletion this led to incorrect “Starting/Deploying” signals and contributed to UI status regressions.

Root cause

Default-case handling in provider status mappers treated unknown values as known states instead of surfacing “unknown.”

Fix

  • Standardize provider mappers to return “unknown” (empty/unset lifecycle) for unrecognized provider status values, rather than defaulting to pending/failed.

@garvitssoni garvitssoni self-assigned this Apr 7, 2026
@garvitssoni garvitssoni marked this pull request as ready for review April 7, 2026 15:13
@garvitssoni garvitssoni requested a review from a team as a code owner April 7, 2026 15:13
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