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

Worker Page Updated #19

Merged
merged 3 commits into from
Nov 12, 2013
Merged

Worker Page Updated #19

merged 3 commits into from
Nov 12, 2013

Conversation

cyprusglobe
Copy link
Contributor

Added more worker information like mem usage, cpu usage, load average will tidy up the page within the next couple of days

Added caching for run_often reason for it so you can't spam the refresh button (more earlier on resources this way)
The cache last's for 10 seconds then you can refresh again

New Dependencies (psutil, gocept.cache)

Changed Readme.md to reflect new dependencies

Added caching for run_often reason for it so you can spam the refresh button (more earlier on resources this way)

New Dependencies (psutil, gocept.cache)

Changed Readme.md to reflect new dependencies
@cyprusglobe cyprusglobe mentioned this pull request Nov 7, 2013
@cyprusglobe
Copy link
Contributor Author

Question: wondering if a worker is offline do we even show a status page the reason i ask this is because
as of right now i have it showing N/A on the page if it can't reach the worker

let me know what you guys think about this

@cyprusglobe
Copy link
Contributor Author

also to give me suggestions and i will get this fixed
also working on so when you click the refresh button it doesnt reload the page need to work on this :P

@oenvoyage
Copy link
Contributor

I would keep the page. Later we can show other statistics that do not need
online status. like total number rendered frames, last frame, Crash logs etc
On 8 Nov 2013 18:13, "Kelvin Jones" notifications@github.com wrote:

also to give me suggestions and i will get this fixed


Reply to this email directly or view it on GitHubhttps://github.com//pull/19#issuecomment-28079782
.

Changes: run_info to get_system_load
Changes: Added more detailed information in README.md
Cleanup: Dashboard code organization
Added: Added condition if worker doesn’t exist then show response page instead of error
Small Changes:  Changes for worker page for refresh button / icon
Major Change: Settings page now changes setting and doesn’t error out if you only set one os path
Upgrades: upgraded to bootstrap 3.0.2
@cyprusglobe
Copy link
Contributor Author

more changes for you guys to look at

…oved data to worker

Changes: removed unneeded comment and corrected print statement
Added: Exception for Settings when Setting doesn’t exist in database and corrected debug statements to relate to be more clear
@cyprusglobe
Copy link
Contributor Author

more changes / fixes

@oenvoyage oenvoyage merged commit 60e4b34 into armadillica:master Nov 12, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants