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 monit integration #872

Closed
wants to merge 2 commits into from
Closed

Add monit integration #872

wants to merge 2 commits into from

Conversation

DBezemer
Copy link
Contributor

Integrate a monit view into the admin console.

@tan-tan-kanarek
Copy link

Please implement it as admin console plugin, we won’t want it on SaaS.

From: David Bezemer [mailto:notifications@github.com]
Sent: Monday, February 17, 2014 9:41 PM
To: kaltura/server
Subject: [server] Add monit integration (#872)

Integrate a monit view into the admin console.


You can merge this Pull Request by running

git pull https://github.com/DBezemer/server IX-9.11.0

Or view, comment on, or merge it at:

#872

Commit Summary

  • Add monit integration

File Changes

Patch Links:


Reply to this email directly or view it on GitHubhttps://github.com//pull/872.

@zoharbabin
Copy link
Contributor

@DBezemer - To clarify, not every deployment has/want monit, in addition, the general practice is to implement things as plugins so that it's configureable and easy to enable/disable according to specific deployment needs.

Take a look at -
plugins/admin_console/gallery
plugins/admin_console/kaltura_internal_tools

This extension should follow the same practice and be implemented as -
plugins/monitor/monit

Thanks!

@DBezemer
Copy link
Contributor Author

considering there is absolutely no documentation on this, let alone a description on how to add a plugin once written, I'll take a pass on this one.

When documentation is added I'll see if it's still worth contributing this as a plugin, until that time I'll take it from either the RPM's or add it to the admin console myself.

@DBezemer DBezemer closed this Feb 25, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants