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-2841: add lock on StartEndpointImpl method call #2842

Merged
merged 10 commits into from
Jan 21, 2025
Merged

issue-2841: add lock on StartEndpointImpl method call #2842

merged 10 commits into from
Jan 21, 2025

Conversation

vladstepanyuk
Copy link
Contributor

@github-actions GitHub Actions
Copy link
Contributor

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 large-tests Launch large tests for PR blockstore Add this label to run only cloud/blockstore build and tests on PR ok-to-test Label to approve test launch for external members labels Jan 13, 2025
@github-actions github-actions bot removed the ok-to-test Label to approve test launch for external members label Jan 13, 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 c18e19a.

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
3760 3758 0 2 0 0

@komarevtsev-d komarevtsev-d added the ok-to-test Label to approve test launch for external members label Jan 13, 2025
@github-actions github-actions bot removed the ok-to-test Label to approve test launch for external members label Jan 13, 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 f945c4c.

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

komarevtsev-d
komarevtsev-d previously approved these changes Jan 16, 2025
View reviewed changes
@komarevtsev-d komarevtsev-d added the ok-to-test Label to approve test launch for external members label Jan 16, 2025
@github-actions github-actions bot removed the ok-to-test Label to approve test launch for external members label Jan 16, 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 a3b13e1.

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

@komarevtsev-d komarevtsev-d added the ok-to-test Label to approve test launch for external members label Jan 16, 2025
@komarevtsev-d komarevtsev-d added the ok-to-test Label to approve test launch for external members label Jan 17, 2025
@github-actions github-actions bot removed the ok-to-test Label to approve test launch for external members label Jan 17, 2025
komarevtsev-d
komarevtsev-d previously approved these changes Jan 17, 2025
View reviewed changes
@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 73c9b4c.

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

@komarevtsev-d komarevtsev-d added the ok-to-test Label to approve test launch for external members label Jan 19, 2025
@github-actions github-actions bot removed the ok-to-test Label to approve test launch for external members label Jan 19, 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 420a80c.

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
3761 3760 0 1 0 0

@komarevtsev-d komarevtsev-d added the ok-to-test Label to approve test launch for external members label Jan 20, 2025
@github-actions github-actions bot removed the ok-to-test Label to approve test launch for external members label Jan 20, 2025
komarevtsev-d
komarevtsev-d previously approved these changes Jan 20, 2025
View reviewed changes
@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 4507bbf.

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
3761 3756 0 5 0 0

@komarevtsev-d komarevtsev-d self-requested a review January 20, 2025 06:12
@vladstepanyuk vladstepanyuk requested a review from budevg January 21, 2025 07:29
@komarevtsev-d komarevtsev-d added the ok-to-test Label to approve test launch for external members label Jan 21, 2025
@github-actions github-actions bot removed the ok-to-test Label to approve test launch for external members label Jan 21, 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 0c4360d.

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

@komarevtsev-d komarevtsev-d merged commit 0dd142c into ydb-platform:main Jan 21, 2025
17 of 21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@komarevtsev-d komarevtsev-d komarevtsev-d approved these changes

@budevg budevg budevg approved these changes

@drbasic drbasic Awaiting requested review from drbasic

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 @budevg @komarevtsev-d