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

add force operation to delete all zero compaction stat ranges; add aсtion to write specified data to compaction map table #1690

Merged
merged 4 commits into from
Aug 15, 2024
Merged

add force operation to delete all zero compaction stat ranges; add aсtion to write specified data to compaction map table #1690

merged 4 commits into from
Aug 15, 2024

Conversation

WilyTiger
Copy link
Collaborator

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

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
5530 5523 0 7 0 0

@WilyTiger WilyTiger force-pushed the users/wilytiger/NBSNEBIUS-282-4 branch from 7bf888a to 755d690 Compare July 30, 2024 18:27
@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 755d690.

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
5534 5527 0 7 0 0

@WilyTiger WilyTiger force-pushed the users/wilytiger/NBSNEBIUS-282-4 branch from 755d690 to eca436f Compare July 31, 2024 08:58
@WilyTiger WilyTiger added filestore Add this label to run only cloud/filestore build and tests on PR large-tests Launch large tests for PR labels Jul 31, 2024
@WilyTiger WilyTiger force-pushed the users/wilytiger/NBSNEBIUS-282-4 branch from eca436f to 52b1f03 Compare July 31, 2024 10:30
@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 52b1f03.

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

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Aug 2, 2024

Note

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

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

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

@WilyTiger WilyTiger force-pushed the users/wilytiger/NBSNEBIUS-282-4 branch from 8499c31 to 4af4b4e Compare August 5, 2024 10:21
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Aug 5, 2024

Note

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

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

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

qkrorlqr
qkrorlqr requested changes Aug 5, 2024
View reviewed changes
cloud/filestore/libs/storage/tablet/model/compaction_map.cpp Outdated Show resolved Hide resolved
cloud/filestore/libs/storage/tablet/tablet_actor_delete_zero_compaction_ranges.cpp Outdated Show resolved Hide resolved
cloud/filestore/libs/storage/tablet/tablet_actor_monitoring.cpp Outdated Show resolved Hide resolved
cloud/filestore/libs/storage/tablet/tablet_actor_monitoring.cpp Outdated Show resolved Hide resolved
cloud/filestore/libs/storage/tablet/tablet_actor_write_compactionmap.cpp Outdated Show resolved Hide resolved
cloud/filestore/libs/storage/tablet/tablet_ut_data.cpp Outdated Show resolved Hide resolved
cloud/filestore/libs/storage/tablet/tablet_ut_data.cpp Outdated Show resolved Hide resolved
cloud/filestore/libs/storage/tablet/tablet_ut_data.cpp Outdated Show resolved Hide resolved
cloud/filestore/libs/storage/tablet/tablet_ut_data.cpp Outdated Show resolved Hide resolved
cloud/filestore/libs/storage/tablet/tablet_ut_data.cpp Outdated Show resolved Hide resolved
@WilyTiger WilyTiger force-pushed the users/wilytiger/NBSNEBIUS-282-4 branch from 3a85502 to 37f6eae Compare August 7, 2024 15:34
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Aug 7, 2024

Note

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

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

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

@WilyTiger WilyTiger force-pushed the users/wilytiger/NBSNEBIUS-282-4 branch from 37f6eae to 18d394b Compare August 7, 2024 17:05
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Aug 7, 2024

Note

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

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

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

@WilyTiger WilyTiger force-pushed the users/wilytiger/NBSNEBIUS-282-4 branch from ea174cf to 59494cf Compare August 13, 2024 15:38
@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 59494cf.

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

qkrorlqr
qkrorlqr previously approved these changes Aug 14, 2024
View reviewed changes
yegorskii
yegorskii previously approved these changes Aug 14, 2024
View reviewed changes
@WilyTiger WilyTiger dismissed stale reviews from yegorskii and qkrorlqr via 51c955c August 15, 2024 09:39
@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 51c955c.

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

@WilyTiger WilyTiger merged commit edc5a54 into main Aug 15, 2024
11 of 14 checks passed
@WilyTiger WilyTiger deleted the users/wilytiger/NBSNEBIUS-282-4 branch August 15, 2024 11:57
WilyTiger added a commit that referenced this pull request Aug 16, 2024
@WilyTiger
add force operation to delete all zero compaction stat ranges; add aс…
2ef2a0d 
…tion to write specified data to compaction map table (#1690)

* add force operation to delete all zero compaction stat ranges; add action to write specified data to compaction map table

* fix-issues

* fix issues

* fix
debnatkh pushed a commit that referenced this pull request Aug 18, 2024
@WilyTiger @debnatkh
add force operation to delete all zero compaction stat ranges; add aс…
a43ea21 
…tion to write specified data to compaction map table (#1690)

* add force operation to delete all zero compaction stat ranges; add action to write specified data to compaction map table

* fix-issues

* fix issues

* fix
debnatkh added a commit that referenced this pull request Aug 19, 2024
@debnatkh @WilyTiger @qkrorlqr
[Filestore, merge to stable] add force operation to delete all zero c…
1fbce29 
…ompaction stat ranges; add action to write specified data to compaction map table, nfs-client.txt, support aliases for filestore names (#1817)

* add force operation to delete all zero compaction stat ranges; add aсtion to write specified data to compaction map table (#1690)

* add force operation to delete all zero compaction stat ranges; add action to write specified data to compaction map table

* fix-issues

* fix issues

* fix

* [Filestore] add `nfs-client.txt` config for filestore analogous to the `nbs-client.txt` one (#1687)

add nfs-client.txt config for filestore analogous to the nbs-client one

* issue-1794: support aliases for filestore names (#1774)

* issue-1794: support aliases for filestore names

* fix cmakelists

* issue-1794: fixed use-after-free in fs search by alias, not making a new THashMap upon each alias search, removed incorrect TABLET_VERIFY in TIndexTabletActor::CompleteTx_CreateSession (#1816)

* issue-1794: fixed use-after-free in fs search by alias, not making a new THashMap upon each alias search, removed incorrect TABLET_VERIFY in TIndexTabletActor::CompleteTx_CreateSession

* issue-1794: fixed use-after-free in fs search by alias, not making a new THashMap upon each alias search, removed incorrect TABLET_VERIFY in TIndexTabletActor::CompleteTx_CreateSession - fixed comment

* fix build: exclude TCertificate from config.cpp

* fix build

* fix build

* fix build

---------

Co-authored-by: Darya Frolova <47457802+WilyTiger@users.noreply.github.com>
Co-authored-by: Darya Frolova <wilytiger@nebius.com>
Co-authored-by: Andrei Strelkovskii <thinkingwhat@yandex.ru>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@qkrorlqr qkrorlqr qkrorlqr approved these changes

@yegorskii yegorskii yegorskii 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
@WilyTiger @qkrorlqr @yegorskii