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

Handling of low disk space #29

Open
michellutz opened this issue Aug 28, 2018 · 4 comments
Open

Handling of low disk space #29

michellutz opened this issue Aug 28, 2018 · 4 comments
Labels
EIP-draft Announcement. Will not be discussed further as long as the necessary information is available.

Comments

@michellutz
Copy link

michellutz commented Aug 28, 2018

Background and Motivation

To improve handling of low disk space the user could be notified about the status of disk. In addition,
some old test object data should be deleted after a while.

This issue is related to etf-validator/etf-webapp#13.

Proposed change

  • Warn user on low disk space in Status View
  • Delete old test object data (based on a max lifetime property) e.g. db index files

Alternatives

  • Adding a label in home.html that shows the usable disk space.
  • Improve the class "TestResultCleaner" to clean also the old test object data.

Funding

TBD

@michellutz michellutz added the EIP-draft Announcement. Will not be discussed further as long as the necessary information is available. label Aug 28, 2018
@jenriquesoriano
Copy link

It could be useful to show in the status more information regarding the ETF status:
% free memory (free/total)
% free disk space (free/total)
CPU load

@michellutz michellutz added EIP Improvement Proposal. Put up for discussion. and removed EIP-draft Announcement. Will not be discussed further as long as the necessary information is available. labels Sep 26, 2018
@jonherrmann
Copy link
Contributor

I think these are information that are relevant only for the Admin and should not be shown in the standard user interface. Meanwhile, I even think that the Loaded components should not be displayed and the status view should be renamed to Running tests or something like that.

For users, it might be more useful to show a classified status, based on the resource details. For example: "System is under heavy load, tests may take a while", "No space left on disk for new test data. New tests can not be started at this time. Wait for old test data to be cleaned up.". I think these information should be displayed when the user attempts to start a new test or visits the Running tests view.

@michellutz michellutz added EIP-draft Announcement. Will not be discussed further as long as the necessary information is available. and removed EIP Improvement Proposal. Put up for discussion. labels Oct 2, 2018
@jenriquesoriano
Copy link

To do:

  • Check with the INSPIRE community the actual use of the status tab of the ETF
  • Check with the INSPIRE community the kind of messages regarding the resources consumption that might be more useful for the users
  • Investigate how to get some additional user navigation info from the ETF/Tomcat logs

@carlospzurita
Copy link

As @jonherrmann said, the information on the ETF Status page is not very relevant to a common user of the application. Maybe the messages to display could be:

  • Low work load, service ready for use
  • Heavy work load, tests run will be queued (In case of low memory or threads availables)
  • Performing data clean up, service disabled (In case of low disk space)

Another option could be to check the test objects that the user is providing, to see if the size of the data could be hosted on the system. Also, apply some heuristics to evaluate the space in memory needed for the test run, and how much time it will take; this way some quick test runs could be allowed under heavy work load.

On a related note, it would be interesting to check the usage of the status page. Right now, the page is accessed via Javascript, and no log is generated when displaying the status tab. A tool like Matomo (formerly named Piwik) could be used to capture the client side interactions, and get usage statistics.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
EIP-draft Announcement. Will not be discussed further as long as the necessary information is available.
Projects
None yet
Development

No branches or pull requests

4 participants