Skip to content

Check node for Backup Restore concurrency#48342

Merged
robot-ch-test-poll1 merged 4 commits intomasterfrom
Backup_Restore_concurrency_check_node_2
Apr 5, 2023
Merged

Check node for Backup Restore concurrency#48342
robot-ch-test-poll1 merged 4 commits intomasterfrom
Backup_Restore_concurrency_check_node_2

Conversation

@SmitaRKulkarni
Copy link
Copy Markdown
Member

Changelog category (leave one):

  • Bug-fix

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
Updated to use tryGet instead of get for checking stage of backups/restores in concurrency check

Same change as #47216 , as it was reverted

@SmitaRKulkarni SmitaRKulkarni requested a review from vitlibar April 3, 2023 11:32
@alexey-milovidov alexey-milovidov self-assigned this Apr 3, 2023
@alexey-milovidov
Copy link
Copy Markdown
Member

The change looks good to me - I also looked there and had the same idea, but... it failed.

@tavplubix
Copy link
Copy Markdown
Member

Integration tests (asan) [3/6] - test_backup_restore_on_cluster/test_disallow_concurrency.py::test_concurrent_restores_on_same_node - it's suspicious

@tavplubix tavplubix marked this pull request as draft April 3, 2023 23:54
@SmitaRKulkarni SmitaRKulkarni marked this pull request as ready for review April 5, 2023 07:04
@robot-ch-test-poll1 robot-ch-test-poll1 merged commit 9466cec into master Apr 5, 2023
@robot-ch-test-poll1 robot-ch-test-poll1 deleted the Backup_Restore_concurrency_check_node_2 branch April 5, 2023 21:49
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.

4 participants