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

[Filestore] Implement a tool which replays filestore-vhost's profile log #2124

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

[Filestore] Implement a tool which replays filestore-vhost's profile log #2124

wants to merge 83 commits into from
+1,389 −46

Conversation

proller
Copy link
Collaborator

@proller proller commented Sep 25, 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 a79bd29.

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

@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 a297662.

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
5805 5799 0 6 0 0

@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 ff0574b.

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

@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 823e1b7.

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
5810 5804 0 6 0 0

@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 d3d1f67.

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

@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 472f048.

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

@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 1af310a.

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

@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 c2e7c5c.

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

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Nov 2, 2024

Note

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

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

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

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Nov 5, 2024

Note

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

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

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

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Nov 7, 2024

Note

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

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

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

debnatkh
debnatkh reviewed Nov 7, 2024
View reviewed changes
cloud/filestore/tools/testing/loadtest/lib/request_replay.h Outdated Show resolved Hide resolved
cloud/filestore/tools/testing/loadtest/lib/request_replay.h Outdated Show resolved Hide resolved
cloud/filestore/tools/testing/loadtest/lib/request_replay_fs.cpp Outdated Show resolved Hide resolved
cloud/filestore/tools/testing/loadtest/lib/request_replay_fs.cpp Outdated Show resolved Hide resolved
cloud/filestore/tools/testing/loadtest/lib/request_replay_fs.cpp Show resolved Hide resolved
cloud/filestore/tools/testing/loadtest/lib/request_replay_fs.cpp Outdated Show resolved Hide resolved
cloud/filestore/tools/testing/loadtest/lib/request_replay_fs.cpp Outdated Show resolved Hide resolved
cloud/filestore/tools/testing/loadtest/lib/request_replay_fs.cpp Outdated Show resolved Hide resolved
cloud/filestore/tools/testing/loadtest/lib/request_replay_fs.cpp Outdated Show resolved Hide resolved
cloud/filestore/tools/testing/loadtest/lib/request_replay.cpp Outdated Show resolved Hide resolved
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Nov 7, 2024

Note

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

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

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
6095 6093 0 2 0 0

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Nov 8, 2024

Note

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

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

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

@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 bc92193.

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

debnatkh
debnatkh requested changes Nov 11, 2024
View reviewed changes
cloud/filestore/tools/testing/loadtest/lib/request_replay_fs.cpp
std::move(spec),
std::move(logging),
std::move(session),
std::move(filesystemId),
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nbs/cloud/filestore/libs/client/durable.cpp

Lines 119 to 129 in 79223f7

TDurableClientBase(
ILoggingServicePtr logging,
ITimerPtr timer,
ISchedulerPtr scheduler,
IRetryPolicyPtr retryPolicy,
std::shared_ptr<TClient> client)
: Logging(std::move(logging))
, Timer(std::move(timer))
, Scheduler(std::move(scheduler))
, RetryPolicy(std::move(retryPolicy))
, Client(std::move(client))

cloud/filestore/tools/testing/loadtest/lib/request_replay_fs.cpp Outdated Show resolved Hide resolved
cloud/filestore/tools/testing/loadtest/lib/request_replay_fs.cpp Show resolved Hide resolved
cloud/filestore/tools/testing/loadtest/lib/request_replay_fs.cpp Outdated Show resolved Hide resolved
@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 c70c01f.

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

@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 a552ec1.

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

@debnatkh
Copy link
Collaborator

В другом пр договорились добавить тесты с канонизацией результатов и с походами по grpc api

@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 2131ab8.

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
6108 6107 0 1 0 0

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

@debnatkh debnatkh debnatkh approved these changes

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

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@proller @debnatkh