Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

List backup fails when a partition has same backup taken by multiple nodes #12622

Closed
deepthidevaki opened this issue May 2, 2023 · 0 comments · Fixed by #12624
Closed

List backup fails when a partition has same backup taken by multiple nodes #12622

deepthidevaki opened this issue May 2, 2023 · 0 comments · Fixed by #12624
Assignees
@deepthidevaki
Labels
kind/bug Categorizes an issue or PR as a bug version:8.2.4 Marks an issue as being completely or in parts released in 8.2.4 version:8.3.0-alpha1 Marks an issue as being completely or in parts released in 8.3.0-alpha1 version:8.3.0-alpha2 Marks an issue as being completely or in parts released in 8.3.0-alpha2 version:8.3.0 Marks an issue as being completely or in parts released in 8.3.0

Comments

@deepthidevaki
Copy link
Contributor

deepthidevaki commented May 2, 2023 •
edited
Loading

Describe the bug

Sometime, a partition may attempt to take a backup again when a backup for the same id already exists. This case is rare, but can happen sometimes if there is a leader change while taking a backup. If this happens, list backup fails with an error message:

{
    "message": "Duplicate key 1 (attempted merging values BackupStatus[backupId=1, partitionId=1, status=COMPLETED, failureReason=, brokerVersion=8.2.2, createdAt=...])"
}

Expected behavior
List backup should be able to handle duplicate backup ids for a partition.
@deepthidevaki deepthidevaki added the kind/bug Categorizes an issue or PR as a bug label May 2, 2023
@deepthidevaki deepthidevaki self-assigned this May 2, 2023
@deepthidevaki deepthidevaki mentioned this issue May 2, 2023
Merged
14 tasks
@ghost ghost closed this as completed in fdf22bd May 2, 2023
This was referenced May 2, 2023
Merged
Merged
ghost pushed a commit that referenced this issue May 2, 2023
@deepthidevaki
merge: #12627
550c216 
12627: [Backport stable/8.2] fix: list backup can handle duplicate backupids of a partition r=deepthidevaki a=backport-action

# Description
Backport of #12624 to `stable/8.2`.

relates to #12622

Co-authored-by: Deepthi Devaki Akkoorath <[email protected]>
ghost pushed a commit that referenced this issue May 2, 2023
@deepthidevaki
merge: #12628
8c79faf 
12628: [Backport release-8.3.0-alpha1] fix: list backup can handle duplicate backupids of a partition r=deepthidevaki a=backport-action

# Description
Backport of #12624 to `release-8.3.0-alpha1`.

relates to #12622

Co-authored-by: Deepthi Devaki Akkoorath <[email protected]>
@remcowesterhoud remcowesterhoud added version:8.3.0-alpha1 Marks an issue as being completely or in parts released in 8.3.0-alpha1 version:8.2.4 Marks an issue as being completely or in parts released in 8.2.4 and removed version:8.3.0-alpha1 Marks an issue as being completely or in parts released in 8.3.0-alpha1 labels May 3, 2023
@lenaschoenburg lenaschoenburg added the version:8.3.0-alpha2 Marks an issue as being completely or in parts released in 8.3.0-alpha2 label Jun 7, 2023
@megglos megglos added the version:8.3.0 Marks an issue as being completely or in parts released in 8.3.0 label Oct 5, 2023
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@deepthidevaki deepthidevaki

Labels
kind/bug Categorizes an issue or PR as a bug version:8.2.4 Marks an issue as being completely or in parts released in 8.2.4 version:8.3.0-alpha1 Marks an issue as being completely or in parts released in 8.3.0-alpha1 version:8.3.0-alpha2 Marks an issue as being completely or in parts released in 8.3.0-alpha2 version:8.3.0 Marks an issue as being completely or in parts released in 8.3.0
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: list backup can handle duplicate backupids of a partition camunda/camunda
4 participants
@megglos @lenaschoenburg @deepthidevaki @remcowesterhoud