-
Notifications
You must be signed in to change notification settings - Fork 119
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
Cleanup disk quota options #528
Conversation
@Kontakter has imported your pull request. If you are a Yandex employee, you can view this diff. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There are compilation errors:
$(SOURCE_ROOT)/yt/yt/server/node/exec_node/job.cpp:2729:38: error: use of undeclared identifier 'Config_'
options.EnableSandboxDiskQuota = Config_->SlotManager->EnableDiskQuota;
...
$(SOURCE_ROOT)/yt/yt/server/node/data_node/unittests/volume_manager_ut.cpp:352:10: error: field designator 'HasRootFSQuota' does not refer to any field in type 'TUserSandboxOptions'
.HasRootFSQuota = true,
^
$(SOURCE_ROOT)/yt/yt/server/node/data_node/unittests/volume_manager_ut.cpp:353:10: error: field designator 'EnableDiskQuota' does not refer to any field in type 'TUserSandboxOptions'
.EnableDiskQuota = false
4de2ca6
to
fff1429
Compare
fff1429
to
8e719e8
Compare
@Kontakter has imported your pull request. If you are a Yandex employee, you can view this diff. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There are still some build errors:
yt/yt/server/node/data_node/unittests/volume_manager_ut.cpp:352:10: error: field designator 'HasRootFSQuota' does not refer to any field in type 'TUserSandboxOptions'
.HasRootFSQuota = true,
^
yt/yt/server/node/data_node/unittests/volume_manager_ut.cpp:353:10: error: field designator 'EnableDiskQuota' does not refer to any field in type 'TUserSandboxOptions'
.EnableDiskQuota = false
^
2 errors generated.
Add slot location option for enabling disk quota for sandbox directories. It is implemented as user disk quota or as porto (project) quota volume. Root volume quota works independently but enforces the same limits. It is implemented only for porto volume.
This file not in OS |
8e719e8
to
9fc1f9f
Compare
@Kontakter has imported your pull request. If you are a Yandex employee, you can view this diff. |
✅ This pull request is being closed because it has been successfully merged. |
Add separate option for enabling disk quota for (slot) sandbox directory. It is implemented as user quota or as porto (project) quota volume. Root volume quota works independently but enforces the same limits. It is implemented only for porto volume. --- ec6078e4ebf548666ea038b666a07e58ca36d2a4 Pull Request resolved: #528
Add separate option for enabling disk quota for (slot) sandbox directory. It is implemented as user quota or as porto (project) quota volume. Root volume quota works independently but enforces the same limits. It is implemented only for porto volume. --- 2ed74abcbc5912ed092f6ce6e5ecdc7ca0f4d791 Pull Request resolved: #528
Add separate option for enabling disk quota for (slot) sandbox directory. It is implemented as user quota or as porto (project) quota volume. Root volume quota works independently but enforces the same limits. It is implemented only for porto volume. --- 46ed6938b6ac2e446938d84ef8b8158b39b2c8d2 Pull Request resolved: #528
Add slot location option for enabling disk quota for sandbox directories.
It is implemented as user disk quota or as porto (project) quota volume.
Root volume quota works independently but enforces the same limits.
It is implemented only for porto volume.