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 disk space to output? #46

Closed
nickvergessen opened this issue Sep 2, 2016 · 20 comments
Closed

Add disk space to output? #46

nickvergessen opened this issue Sep 2, 2016 · 20 comments

Comments

@nickvergessen
Copy link
Member

I think we should display the disk space as well

@hitam4450
Copy link

Well, it has to defined what kind of disc space you are talking about!

The disc space where NC is located (installed) or the disc space where the data is stored? That is not always the same disc space! I reassigned the data folder onto a separate disc form the installation of NC!

regards, hitam4450

@nickvergessen
Copy link
Member Author

Same here, of course data dir is what is of interest ;)

@schiessle
Copy link
Member

I remember that I did a quick test when I started writing the app. The problem I faced was to get the available disc space for /data (remember, it also has to work with external primary storage). Do you have a idea how the get this information with the files view or node api?

@nickvergessen
Copy link
Member Author

Well honestly, better to have a value working for everything except external storage, then one that is usless to everyone? ;)

@BentHaase
Copy link

BentHaase commented Oct 29, 2016

To add up to this, please allow an option to show total used storage and storage used by user. (for example in a splitted bar graph or pi-chart)

@nickvergessen nickvergessen modified the milestone: Nextcloud 11.0 Jan 18, 2017
@GitHubUser4234
Copy link

To add up to this, please allow an option to show total used storage and storage used by user. (for example in a splitted bar graph or pi-chart)

Absolutely agree to have an option to see the storage used by users, whereas a bar graph would be the preferable way of presentation.

@brantje
Copy link
Member

brantje commented Jul 13, 2017

Would be nice to see various disk spaces.
Eg:

  • From the location where nextcloud is installed.
  • As above, used disk space by users.

@TheMrApostel
Copy link

I agree.
It's not abvious which users storage is used up by checking the log file.
The log file just gives back a 507 Insufficient Storage.

@brantje
Copy link
Member

brantje commented Jul 22, 2017

Anyway i could add an bounty to fix this issue?

@Schmuuu
Copy link

Schmuuu commented Aug 4, 2017

There is an update coming for the "User Settings" administrators page. The users used space will be graphically shown in the quota column. So you will see a bar indicating a rough percentage of used space and when mouseover there will be exact values.

I think when this feature will be out (probably with 12.1) there is no need for a user based disc usage in the serverinfo area.
However I would still like to see the overall disc usage.

@TheMrApostel
Copy link

@Schmuuu
That would be great! I think you mean 12.0.1? This is going to be released today.
I will upgrade my version to that at the end of the week then.

@Schmuuu
Copy link

Schmuuu commented Aug 7, 2017

@TheMrApostel
Yeah I meant 12.0.1, sorry :)
I upgraded my test server to 12.0.1 RC5 yesterday and wanted to review the feature I described. While I couldn't see the feature implemented I searched the Pull Request again and noticed, that it is actually NC 13 that should ship it. Sorry for the misleading.

But maybe it works to change the files accordingly and make it work in 12.0.1 already. Here is the pull request with some screenshots:
nextcloud/server#5342

@brantje
Copy link
Member

brantje commented Aug 11, 2017

They made a backport to 12.0.1.

However I would still like to see the overall disc usage.

This would be nice, maybe we can use the output of df -h?

@c0fe
Copy link

c0fe commented Mar 7, 2018

Any updates on this?

@c0fe
Copy link

c0fe commented Mar 8, 2018

@nickvergessen

@nickvergessen
Copy link
Member Author

Well anyone can pick up the task and add the value to the UI ;)

@c0fe
Copy link

c0fe commented Mar 8, 2018

@nickvergessen is there any design guidelines that must be followed?

@nickvergessen
Copy link
Member Author

well it should fit into the existing code/ui

@JamesBream
Copy link

JamesBream commented May 14, 2018

@nickvergessen This should be pretty easy given that <freespace> already appears in the external monitoring API, right?

I'd be up for taking a crack at this in a few weeks time if no-one else gets to it before then.

@patschi
Copy link
Member

patschi commented Sep 4, 2018

The <freespace> value is now also being displayed in the Storage section on the serverinfo page, as done in PR #127.

@patschi patschi closed this as completed Sep 4, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests