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

issue-2947: don't backup tablet info in temporary server #2978

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

issue-2947: don't backup tablet info in temporary server #2978

wants to merge 3 commits into from
+31 −12

Conversation

vladstepanyuk
Copy link
Contributor

@vladstepanyuk vladstepanyuk commented Feb 5, 2025 •
edited by SvartMetal
Loading

#2947 показалось странным, что временный сервер тоже бэкапит информацию о таблетках. Плюс это может помешать во время рестарта при прогреве соедений в задачке #2947

#2100

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Feb 5, 2025

Hi! Thank you for contributing!
The tests on this PR will run after a maintainer adds an ok-to-test label to this PR manually. Thank you for your patience!

@komarevtsev-d komarevtsev-d added blockstore Add this label to run only cloud/blockstore build and tests on PR large-tests Launch large tests for PR ok-to-test Label to approve test launch for external members labels Feb 5, 2025
@github-actions github-actions bot removed the ok-to-test Label to approve test launch for external members label Feb 5, 2025
@komarevtsev-d komarevtsev-d added the ok-to-test Label to approve test launch for external members label Feb 5, 2025
@github-actions github-actions bot removed the ok-to-test Label to approve test launch for external members label Feb 5, 2025
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Feb 5, 2025

Note

This is an automated comment that will be appended during run.

🔴 linux-x86_64-relwithdebinfo: some tests FAILED for commit d4e7069.

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
3810 3809 0 1 0 0

@komarevtsev-d komarevtsev-d added the ok-to-test Label to approve test launch for external members label Feb 6, 2025
@github-actions github-actions bot removed the ok-to-test Label to approve test launch for external members label Feb 6, 2025
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Feb 6, 2025

Note

This is an automated comment that will be appended during run.

🔴 linux-x86_64-relwithdebinfo: some tests FAILED for commit 31811e6.

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
3810 3809 0 1 0 0

@komarevtsev-d komarevtsev-d added the ok-to-test Label to approve test launch for external members label Feb 6, 2025
@github-actions github-actions bot removed the ok-to-test Label to approve test launch for external members label Feb 6, 2025
komarevtsev-d
komarevtsev-d previously approved these changes Feb 6, 2025
View reviewed changes
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Feb 6, 2025

Note

This is an automated comment that will be appended during run.

🔴 linux-x86_64-relwithdebinfo: some tests FAILED for commit ac3eceb.

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
3810 3809 0 1 0 0

@vladstepanyuk vladstepanyuk force-pushed the issue-2947-add-temp-server-flag branch from ac3eceb to 32bc59e Compare February 11, 2025 09:13
@komarevtsev-d komarevtsev-d added the ok-to-test Label to approve test launch for external members label Feb 11, 2025
@github-actions github-actions bot removed the ok-to-test Label to approve test launch for external members label Feb 11, 2025
@github-actions GitHub Actions
Copy link
Contributor

Note

This is an automated comment that will be appended during run.

🔴 linux-x86_64-relwithdebinfo: some tests FAILED for commit 32bc59e.

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
3818 3817 0 1 0 0

@komarevtsev-d komarevtsev-d added the ok-to-test Label to approve test launch for external members label Feb 12, 2025
@github-actions github-actions bot removed the ok-to-test Label to approve test launch for external members label Feb 12, 2025
@vladstepanyuk vladstepanyuk force-pushed the issue-2947-add-temp-server-flag branch from 32bc59e to b1e769f Compare February 12, 2025 11:41
@komarevtsev-d komarevtsev-d added the ok-to-test Label to approve test launch for external members label Feb 12, 2025
@github-actions github-actions bot removed the ok-to-test Label to approve test launch for external members label Feb 12, 2025
@github-actions GitHub Actions
Copy link
Contributor

Note

This is an automated comment that will be appended during run.

🟢 linux-x86_64-relwithdebinfo: all tests PASSED for commit b1e769f.

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
3820 3820 0 0 0 0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@SvartMetal SvartMetal Awaiting requested review from SvartMetal

@komarevtsev-d komarevtsev-d Awaiting requested review from komarevtsev-d

At least 2 approving reviews are required to merge this pull request.

Assignees
No one assigned
Labels
blockstore Add this label to run only cloud/blockstore build and tests on PR large-tests Launch large tests for PR
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@vladstepanyuk @SvartMetal @komarevtsev-d