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

[Backport to 5.15] NS FS Non Contanarized fixes #7771

Merged
merged 7 commits into from
Jan 30, 2024

Conversation

nimrod-becker
Copy link
Contributor

Explain the changes

  1. NS FS Non Contanarized fixes

jackyalbo and others added 7 commits January 30, 2024 11:48
Signed-off-by: jackyalbo <jacky.albo@gmail.com>
(cherry picked from commit 664180e)
Signed-off-by: jackyalbo <jacky.albo@gmail.com>
(cherry picked from commit a24e769)
Signed-off-by: Utkarsh Srivastava <srivastavautkarsh8097@gmail.com>

rename the config variable to "DENY_UPLOAD_TO_STORAGE_CLASS_STANDARD"

Signed-off-by: Utkarsh Srivastava <srivastavautkarsh8097@gmail.com>
(cherry picked from commit 8e67135)
1. Add details to the error message when invalid schema.
2. Update the tests to check the basic message (without the additional info).
3. Add JSDoc on the functions in the file nsfs_schema_utils.

Signed-off-by: shirady <57721533+shirady@users.noreply.github.com>
(cherry picked from commit bd39c93)
Signed-off-by: Utkarsh Srivastava <srivastavautkarsh8097@gmail.com>
(cherry picked from commit 2235975)
1. Added a structure with the types of each option.
2. Removed InvalidAccountUID and InvalidAccountGID since they will be checked when the values are passed.
3. In the file test_nc_nsfs_cli.js separate the function exec_manage_cli to allow passing flags without value.

Signed-off-by: shirady <57721533+shirady@users.noreply.github.com>
(cherry picked from commit a94ebec)
Signed-off-by: Romy <35330373+romayalon@users.noreply.github.com>
(cherry picked from commit 2efb107)
@nimrod-becker nimrod-becker merged commit 02dd033 into noobaa:5.15 Jan 30, 2024
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants