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.

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

Users/yegorskii/merge 23 3 #1807

Merged
merged 14 commits into from
Aug 19, 2024
Merged

Users/yegorskii/merge 23 3 #1807

merged 14 commits into from
Aug 19, 2024

Conversation

yegorskii
Copy link
Collaborator

No description provided.

Copy link
Contributor

Note

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

🟢 default-linux-x86-64-relwithdebinfo: all tests PASSED for commit 94f0904.

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

@yegorskii yegorskii added the large-tests Launch large tests for PR label Aug 18, 2024
Copy link
Contributor

Note

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

🟢 default-linux-x86-64-relwithdebinfo: all tests PASSED for commit 94f0904.

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

qkrorlqr
qkrorlqr previously approved these changes Aug 19, 2024
qkrorlqr
qkrorlqr previously approved these changes Aug 19, 2024
yegorskii and others added 14 commits August 19, 2024 09:25
* add private api call to get storage service config

* fix python style checks

* update
* use new ydb regitration api

* fix headers order

* update

* fix tests

* update

* update

* update

* update

* fix include order

* better unit testing

* update
* refactor node registration

* update

* update

* update
* use new ydb registration api in filestore + tests

* remove blank lines

* remove getting configs from cms

* update

* remove some includes

* moved registration parameters to storage config

* update

* update

* update

* update

* update

---------

Co-authored-by: yegorskii <[email protected]>
* set items count limit for TTrimBytes transaction

* update

* update

* update

* update

* update

* removed blank lines

* add STORAGE_VERIFY to FinishCleanup

* update

* add log tag to tfreshbytes

* more verify macroses

* add newline

---------

Co-authored-by: yegorskii <[email protected]>
Copy link
Contributor

Note

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

🟢 default-linux-x86-64-relwithdebinfo: all tests PASSED for commit f799682.

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

@yegorskii yegorskii merged commit e125081 into stable-23-3 Aug 19, 2024
6 of 8 checks passed
@yegorskii yegorskii deleted the users/yegorskii/merge-23-3 branch August 19, 2024 11:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
large-tests Launch large tests for PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants