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

Show that the overall assigned server diskspace exceeds physical disksize #10240

Open
newhinton opened this issue Jul 14, 2018 · 18 comments
Open
Labels
1. to develop Accepted and waiting to be taken care of enhancement feature: users and groups

Comments

@newhinton
Copy link
Contributor

Currently there is no indicator how much disk-space the current nextcoud-instance is using. Add an indicator to show this.

See #10127

@newhinton newhinton changed the title [Feature] Show that the overall assignedserver diskspace exceeds physical disksize [Feature] Show that the overall assigned server diskspace exceeds physical disksize Jul 14, 2018
@nextcloud-bot
Copy link
Member

GitMate.io thinks possibly related issues are #5201 (Feature Request: Show Shared Folder Name), #220 (Feature Request: Prevent duplicate uploads server-side), #10230 (Feature/shares overview), #5502 (Feature request - Assign nextcloud tags to events), and #9005 (c).

@stratege1401
Copy link

stratege1401 commented Jul 16, 2018

if you are talking about the nextcloud data folder usage, there is some apps available like the "dashboard apps".

Usually, in order to avoid chrashing your server due to available hdd space, you may want to create specific partition:

  • log --> so, if your log goes mad, they just stop growing and your server still "works"
  • home --> might be usefull to seperated the home from the "run" and the "data"
  • data --> where you write data, like the nextcloud data files, or the rdd files, or the whatever app files( in order to avoid chrasing your server due to available hdd space )
  • run : the /etc /www /sbin /...

In my case, i have 2 disk:

  • a 32 GO ssd for the Debian/www/nextcloud running files
    -- 3 partitions mounted as /run /home /log
  • a raid5 array for all the data files
    --- 1 partiton mounted as /mnt/raid6/nextcloud
    I personnaly give quoto to users, i let them take care of their usage but i help them with the dashboard app

@newhinton
Copy link
Contributor Author

newhinton commented Jul 16, 2018

@stratege1401 Its not about crashing my server, its about the visual information that your total assigned quota of all user exceeds the remaining free diskspace. While it's true that there are apps that add this function, it could be a corefunctionality since this is a vital information for every admin needs. Also this was a proposal for a feature, and more or less open for discussion ;)

@MorrisJobke I think such an indicator (total assigned space/physical dispace) could find its place directly above the useradministration. Its a place where you assign the quota, and therefore you need this information there. What do you think?

@MorrisJobke
Copy link
Member

@MorrisJobke I think such an indicator (total assigned space/physical dispace) could find its place directly above the useradministration. Its a place where you assign the quota, and therefore you need this information there. What do you think?

Makes sense 👍

cc @nextcloud/designers for more input

@jancborchardt
Copy link
Member

I think such an indicator (total assigned space/physical dispace) could find its place directly above the useradministration. Its a place where you assign the quota, and therefore you need this information there. What do you think?

Totally agree, really good place to put it. :) That’s where the individual quota and usage of everyone is shown, and hence we should put the whole usage there too.

@MorrisJobke MorrisJobke changed the title [Feature] Show that the overall assigned server diskspace exceeds physical disksize Show that the overall assigned server diskspace exceeds physical disksize Jul 17, 2018
@newhinton
Copy link
Contributor Author

@jancborchardt I had the old useradministration in mind when i made the proposition. The new design gets rid of the first row, which i would need to reintroduce. This may, or may not be good, i am not sure.

Also i want this bar bigger, at least twice that big, but i havent found out how to do this, yet :D

2018-07-18 20_44_02-window

@jancborchardt
Copy link
Member

@newhinton good mockup, yes! Looking at it I’m actually thinking it might be nicer to do it just like in the Files app, where it is on the bottom left in the navigation:
screenshot from 2018-07-18 23-02-39

That way it’s consistent and basically standardized. :) What do you think @nextcloud/designers?

@newhinton
Copy link
Contributor Author

newhinton commented Jul 18, 2018

I have thought about that too, but it also could be confusing, since this changes from overall to user usage, and this is not directly clear. We would need to make a clear distinction between those two elements, and i have no good idea how to achieve this

@newhinton
Copy link
Contributor Author

Just a quick idea, what if we move it down there, but make it a pie chart? dont know if it would look good ^^

@juliusknorr
Copy link
Member

I have thought about that too, but it also could be confusing, since this changes from overall to user usage, and this is not directly clear. We would need to make a clear distinction between those two elements, and i have no good idea how to achieve this

It should have a clear indicator that it is about the system storage quota, but then it is the best position to put it, i think.

Just a quick idea, what if we move it down there, but make it a pie chart? dont know if it would look good ^^

Not sure, but the usual look seems already fine to me as it is not to bloated and we have consistency with the files app then. But feel free to do a mockup on how you think this could look like. That makes it easier to discuss.

@newhinton
Copy link
Contributor Author

newhinton commented Jul 22, 2018

@juliushaertl I have decided that in the first version i am using the "old" way, like in the filesapp. For that i moved the css regarding the quota from the files app to apps.scss in the core. Is that okay?

@jancborchardt I am currently showing only the used to assigned ratio, the information if assigned exceeds the overall storage is missing. For that i would use an idea from stack-overflow which uses three overlapping div's to emulate a tiered bar. It looks the same, but works differently. I rewrote like 95% of the stackoverflow-code, but i would request advice here, if it is okay to use that code.

If i can use the code, than we wouldn't need to move the quota-css, instead we would introduce new classes in the core ;)

nextcloud:/feature/10240/show-overall-server-usage-in-useradministration

@newhinton
Copy link
Contributor Author

@juliushaertl @jancborchardt can you give me feedback here?

@juliusknorr
Copy link
Member

@juliushaertl I have decided that in the first version i am using the "old" way, like in the filesapp. For that i moved the css regarding the quota from the files app to apps.scss in the core. Is that okay?

Yes, that should be fine.

Regarding the exceeding overal storage, how about just showing an error icon next to the quota bar with a tooltip that gives some more details on the error?

@newhinton
Copy link
Contributor Author

@juliushaertl That depends on what we want to show
If we only show something if the overall used storage exceeds the physical space, then a warning would be fine. But my plan is to show overassignment when it occurs, whether it is used or not. In that case, it would not be an error, maybe even desired. In that case an warning would be over the top

@skjnldsv skjnldsv added the 1. to develop Accepted and waiting to be taken care of label Jun 17, 2019
@szaimen

This comment has been minimized.

@Gobytego

This comment has been minimized.

@skjnldsv

This comment has been minimized.

@joshtrichards
Copy link
Member

The serverinfo app might also be another good place for this. It also has similar info.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1. to develop Accepted and waiting to be taken care of enhancement feature: users and groups
Projects
None yet
Development

No branches or pull requests

10 participants