usnistgov / SpectrumBrowser

ITL
12 stars 11 forks source link

New Disk Monitor Service. #209

Closed ranganathanm closed 9 years ago

ranganathanm commented 9 years ago

The disk monitoring service needs to be co-resident with the database and periodically report utilization of the disk by writing the information into mongodb. This will allow the resource monitor to function regardless of where the db is installed.

ranganathanm commented 9 years ago

Assigned to Kheri because it is an ITS specific requirement that the db be located in its own virtual machine.

ranganathanm commented 9 years ago

Commit is not complete. When I run fab deploy it does not install the service. Please fix the problem so that the service is installed on the db VM. The deployment script should do this - not the Makefile.

klhicks commented 9 years ago

Can we close this one now? @ranganathanm

ranganathanm commented 9 years ago

There are errors in the fabric script. The fabric script must deploy the dbmonitor service on the database machine. After this is done (and tested) we can close.


From: klhicks notifications@github.com Sent: Monday, August 24, 2015 1:29 PM To: usnistgov/SpectrumBrowser Cc: Ranganathan, Mudumbai Subject: Re: [SpectrumBrowser] New Disk Monitor Service. (#209)

Can we close this one now? @ranganathanmhttps://github.com/ranganathanm

Reply to this email directly or view it on GitHubhttps://github.com/usnistgov/SpectrumBrowser/issues/209#issuecomment-134308652.

ranganathanm commented 9 years ago

The disk utilization script is returning 0. This needs to be fixed. A unit-test case has to be written for this feature.

ranganathanm commented 9 years ago

unit-test committed.

klhicks commented 9 years ago

No errors were returned.

ranganathanm commented 9 years ago

Unit test does not check for things like whether the disk utilization returned is 0. If it does not return error that just means that all expected fields are present. Please check if the disk utilization returned is the same as what you see when you go to the Database machine and run

df /spectrumdb

The unit test is meant to serve as a pattern for future features you may add.

klhicks commented 9 years ago

It does not. Working on fixing the issue where if the df value changes it creates a new ID so the command we use now only finds the first value inserted, which is the wrong value.

klhicks commented 9 years ago

The correct value is now being displayed and the mongo entry issue has been fixed.

ranganathanm commented 9 years ago

Verified and closing. Thanks.