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

Add option to choose the scan_tree() implementation in the NetApp backend #1975

Open
achimnol opened this issue Mar 29, 2024 · 0 comments
Open
Assignees
Labels
area:upstream Issues with root cause from upstream 3rd parties comp:storage-proxy Related to Storage proxy component effort:easy Need to understand only a specific region of codes (good first issue, easy). type:enhance Enhance component, behavior, internals without user-facing features urgency:3 Must be finished within a certain time frame.
Milestone

Comments

@achimnol
Copy link
Member

achimnol commented Mar 29, 2024

In some customer setups, running the NetApp's XCP executable is too slow, incurring 2-3 seconds of delays whenever storage-proxy tries to scan the directory contents (list of files) regardless of how many files are there.

Since directory listing is a frequently invoked operation in the storage folder management UI, it should be fast.

Unfortunately it seems not easy to reduce the XCP invocation latency. Let's add an opt-out flag (e.g., netapp_xcp_exclude_scan_tree) to the NetApp storage plugin's configuration about whether to use XCP-based scan_tree() implementation for XCPFSOpModel or the vanilla scan_tree() implementation from the super-class, BaseFSOpModel.

@achimnol achimnol added area:upstream Issues with root cause from upstream 3rd parties comp:storage-proxy Related to Storage proxy component type:enhance Enhance component, behavior, internals without user-facing features urgency:3 Must be finished within a certain time frame. labels Mar 29, 2024
@achimnol achimnol added this to the 24.03 milestone Mar 29, 2024
@achimnol achimnol added the effort:easy Need to understand only a specific region of codes (good first issue, easy). label Mar 29, 2024
@achimnol achimnol changed the title Add option to choose the scandir() implementation in the NetApp backend Add option to choose the scan_tree() implementation in the NetApp backend Mar 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:upstream Issues with root cause from upstream 3rd parties comp:storage-proxy Related to Storage proxy component effort:easy Need to understand only a specific region of codes (good first issue, easy). type:enhance Enhance component, behavior, internals without user-facing features urgency:3 Must be finished within a certain time frame.
Projects
None yet
Development

No branches or pull requests

2 participants