Skip to content

Revert "feat: Update openapi contracts to include onConflict parameter for restore (#655)" - #656

Open
devanbenz wants to merge 1 commit into
masterfrom
db/27578/revert-restore-conflict
Open

Revert "feat: Update openapi contracts to include onConflict parameter for restore (#655)"#656
devanbenz wants to merge 1 commit into
masterfrom
db/27578/revert-restore-conflict

Conversation

@devanbenz

@devanbenz devanbenz commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

Reverts #655.

Bucket conflict handling for influx restore --on-conflict is now implemented entirely client-side in the CLI (see influxdata/influx-cli#583): the CLI checks bucket existence itself and resolves error/skip/replace without any server involvement. The server never receives an onConflict parameter, so this spec addition is unused and can be removed.

…r for restore (#655)"

This reverts commit d90ba20.

Bucket conflict handling for `influx restore --on-conflict` is now
implemented entirely client-side in the CLI (influxdata/influx-cli#583),
so the server never receives an onConflict parameter and the spec change
is no longer needed.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@devanbenz
devanbenz requested a review from a team as a code owner August 11, 2026 20:46
@devanbenz
devanbenz requested review from jstirnaman and removed request for a team August 11, 2026 20:46

@gwossum gwossum left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

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