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

Added statfs for block-cache #1470

Merged
merged 48 commits into from
Jan 15, 2025
Merged

Added statfs for block-cache #1470

merged 48 commits into from
Jan 15, 2025

Conversation

ashruti-msft
Copy link
Collaborator

@ashruti-msft ashruti-msft commented Jul 24, 2024

Added statfs for block_cache which will show the memory available if disk is not configured otherwise the disk space. Added corresponding tests for this and fixed a few unit changes in file_cache.

Used a VM with an attached disk of size 5G for the follwoing outputs:

When file cache/block cache uses the attached disk for disk caching
Screenshot 2024-11-22 153142

When config has block_cache component without disk cache it will show the memory available:
Screenshot 2024-11-22 153029

Output when file cache/block cache uses the same VM as disk cache
Screenshot 2024-11-22 152615

@vibhansa-msft vibhansa-msft modified the milestones: V2-2.3.1, v2-2.3.2 Jul 26, 2024
Ubuntu and others added 2 commits August 27, 2024 05:48
@jainakanksha-msft
Copy link
Collaborator

@ashruti-msft Please add the output in the PR description and details of the change.

@jainakanksha-msft
Copy link
Collaborator

@vibhansa-msft
This PR changes will represent temp disk details with the notation of root Filesystem.
I think that can confuse the users.
Should we change the FileSystem representation also for this somehow if that is feasible?

@vibhansa-msft vibhansa-msft modified the milestones: v2-2.4.0, v2-2.4.1 Nov 12, 2024
@ashruti-msft
Copy link
Collaborator Author

@ashruti-msft Please add the output in the PR description and details of the change.

@jainakanksha-msft I have added the output please have a look

@vibhansa-msft vibhansa-msft changed the base branch from main to blobfuse/2.4.1 November 29, 2024 07:58
@vibhansa-msft vibhansa-msft merged commit c16d3d2 into blobfuse/2.4.1 Jan 15, 2025
8 checks passed
@vibhansa-msft vibhansa-msft deleted the ashruti/disk branch January 15, 2025 09:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants