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

NC | Backports - 5.18.z #8881

Merged
merged 8 commits into from
Mar 24, 2025
Merged

Conversation

romayalon
Copy link
Contributor

@romayalon romayalon commented Mar 19, 2025

@romayalon romayalon force-pushed the romy-backports-5.18 branch from 56045cb to 68e7fd2 Compare March 24, 2025 07:48
Copy link
Contributor

@shirady shirady left a comment

Choose a reason for hiding this comment

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

I reviewed commits:
dbc461e
68e7fd2

LGTM

@romayalon romayalon force-pushed the romy-backports-5.18 branch from 68e7fd2 to 5353490 Compare March 24, 2025 08:42
romayalon and others added 8 commits March 24, 2025 11:00
Signed-off-by: Romy <[email protected]>
(cherry picked from commit 9071a54)
Signed-off-by: Romy <[email protected]>
(cherry picked from commit 82cd296)
Signed-off-by: nadav mizrahi <[email protected]>
(cherry picked from commit 48a908e)
Signed-off-by: nadav mizrahi <[email protected]>
(cherry picked from commit 2f27896)
…eration

1. Replace the function stat_ignore_eacces with stat_if_exists and add ignore of ENOENT and ENOTDIR (in addition to EACCES with a flag of config.EACCES_IGNORE_ENTRY).
2. Call stat on the entry_path one time and save is instead of calling it in concurrency so that we can be sure that the value was not deleted from the result array. Move the stat call before results.push(r) and results.splice(pos, 0, r) as they both add a result to the array.
3. Add a comment before the check_access, see also in PR description of Added check_access() call to list objects noobaa#6576.

This change is continuing PR noobaa#8751 by adding a case to ignore a stat failure entry.

Signed-off-by: shirady <[email protected]>
(cherry picked from commit fe4115c)
@romayalon romayalon force-pushed the romy-backports-5.18 branch from 5353490 to 625cbe6 Compare March 24, 2025 09:00
@nimrod-becker nimrod-becker merged commit 6c8a935 into noobaa:5.18 Mar 24, 2025
11 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.

5 participants