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-2434: DescribeSessions method should return OrphanSessions as well as active sessions to prevent filesystem destruction right after tablet reboot #2484

Merged
merged 3 commits into from
Nov 14, 2024
Merged

issue-2434: DescribeSessions method should return OrphanSessions as well as active sessions to prevent filesystem destruction right after tablet reboot #2484

merged 3 commits into from
Nov 14, 2024
+186 −48

Conversation

qkrorlqr
Copy link
Collaborator

@qkrorlqr
issue-2434: DescribeSessions method should return OrphanSessions as w…
596570d 
…ell as active sessions to prevent filesystem destruction right after tablet reboot
@qkrorlqr qkrorlqr added large-tests Launch large tests for PR filestore Add this label to run only cloud/filestore build and tests on PR labels Nov 12, 2024
yegorskii
yegorskii previously approved these changes Nov 12, 2024
View reviewed changes
debnatkh
debnatkh previously approved these changes Nov 12, 2024
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 596570d.

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
1953 1952 0 1 0 0

@qkrorlqr qkrorlqr dismissed stale reviews from debnatkh and yegorskii via fa215d4 November 13, 2024 11:22
debnatkh
debnatkh previously approved these changes Nov 13, 2024
View reviewed changes
@qkrorlqr qkrorlqr added the recheck Add this label to relaunch checks, it will be automatically removed label Nov 13, 2024
@github-actions github-actions bot removed the recheck Add this label to relaunch checks, it will be automatically removed label Nov 13, 2024
@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 fa215d4.

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
1954 1951 0 2 1 0

@qkrorlqr qkrorlqr added the recheck Add this label to relaunch checks, it will be automatically removed label Nov 13, 2024
@github-actions github-actions bot removed the recheck Add this label to relaunch checks, it will be automatically removed label Nov 13, 2024
@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 fa215d4.

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
1954 1952 0 2 0 0

@qkrorlqr qkrorlqr added the recheck Add this label to relaunch checks, it will be automatically removed label Nov 13, 2024
@github-actions github-actions bot removed the recheck Add this label to relaunch checks, it will be automatically removed label Nov 13, 2024
@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 fa215d4.

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

@qkrorlqr
issue-2434: DestroySessionRequest should be sent from ServiceActor to…
0bfc5a8 
… IndexTabletActor even if Session can't be found in ServiceState (e.g. after filestore-server reboot)
@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 0bfc5a8.

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

@qkrorlqr qkrorlqr merged commit 94423e4 into main Nov 14, 2024
11 of 13 checks passed
@qkrorlqr qkrorlqr deleted the users/qkrorlqr/issue-2434 branch November 14, 2024 09:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yegorskii yegorskii yegorskii approved these changes

@debnatkh debnatkh debnatkh approved these changes

Assignees
No one assigned
Labels
filestore Add this label to run only cloud/filestore 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
@qkrorlqr @debnatkh @yegorskii