372 Commits

Author SHA1 Message Date
David Buxton
fc33c9859a Merge branch 'reddit-graphite-check-overhaul' 2015-10-05 18:58:15 +01:00
David Buxton
8eff54c265 Add small test 2015-10-05 18:57:23 +01:00
David Buxton
ebbd626943 Merge branch 'graphite-check-overhaul' of https://github.com/reddit/cabot into reddit-graphite-check-overhaul 2015-10-05 18:52:48 +01:00
David Buxton
80e56089a7 Pin foreman (fixes #196) 2015-10-02 10:47:12 +02:00
David Buxton
4544986150 Update admin 2015-10-02 10:27:15 +02:00
David Buxton
ef8bcc6253 Update example conf files to hipchat 1.7.0 2015-10-02 10:10:35 +02:00
David Buxton
ec964151f5 Add ability to cancel acknowledgements 2015-10-02 10:06:07 +02:00
Neil Williams
14f77c2f49 Overhaul formatting of Graphite check failures.
This gives better detail on which specific series failed the checks
which can help wildcard checks be more informative.

To save space, we strip the nodes that all series have in common from
the front and back of the series targets, so "prefix.a.suffix" and
"prefix.b.suffix" turn into just "a" and "b".
2015-10-01 22:35:05 -07:00
David Buxton
b4ab215ad2 Basic alert acknowledgement mechanism 2015-10-01 21:17:23 -04:00
David Buxton
257bc4a819 Don't throw an error on >, < etc when one or more input series is empty 2015-10-01 19:41:49 -04:00
David Buxton
f587076dda Upgrade requests to 2.7.0 2015-09-22 14:48:36 +01:00
David Buxton
ee34ceee0f Update tests to address #262 2015-09-22 13:19:19 +01:00
David Buxton
88aab44722 Remove debugging print statement 2015-09-15 12:11:37 +01:00
David Buxton
a98ab0deba Add some debugging info for travis 2015-09-15 08:53:48 +01:00
David Buxton
1cb8e4505d Test timing separately 2015-09-15 08:41:41 +01:00
crucial7
8f41505411 If statement wasn't checking for None correctly, which made failures "pass" when they shouldn't have. 2015-09-11 13:24:05 -04:00
David Buxton
befb85b3e0 Test for timing 2015-09-01 19:05:59 -04:00
David Buxton
6c9e70ef34 Update icon 2015-09-01 18:54:37 -04:00
David Buxton
d0c709375d Fix KeyError 2015-09-01 18:50:28 -04:00
David Buxton
cd34b45e27 Return all values from Graphite 2015-09-01 18:42:31 -04:00
David Buxton
e420a3eb41 Fix time taken 2015-09-01 18:39:40 -04:00
David Buxton
b567bde9c0 Add some tests for annoying graphite responses 2015-09-01 18:26:18 -04:00
David Buxton
9855e41124 Small fix 2015-09-01 18:13:14 -04:00
David Buxton
b468c5bf57 Resolve issues with expected metrics 2015-09-01 17:52:56 -04:00
David Buxton
5d45497687 Merge pull request #252 from movermeyer/master
Readding changes proposed by yigitbey, plus support for status checks…
2015-08-26 15:22:48 +01:00
Michael Overmeyer
c117e0021d Using proper casing for default values of calculated_status. 2015-08-25 12:49:45 -04:00
Michael Overmeyer
7ccf6fc721 Readding changes proposed by yigitbey, plus support for status checks on instances. See https://github.com/arachnys/cabot/pull/157 2015-08-25 07:40:46 -04:00
David Buxton
7a2d5d47e0 Merge pull request #250 from movermeyer/master
Increasing the valid size of the email field.
2015-08-24 19:33:18 -04:00
Michael Overmeyer
114f5750b5 Fixing typo.
Issue 251 (https://github.com/arachnys/cabot/issues/251)
2015-08-24 17:20:29 -04:00
michael.overmeyer@EXACTEARTH.COM
f060c29374 Increasing the valid size of the email field.
The longest valid email is 254 bytes:
https://stackoverflow.com/questions/386294/what-is-the-maximum-length-of-a-valid-email-address
http://www.rfc-editor.org/errata_search.php?rfc=3696&eid=1690

However, Django 1.6.8 has a default maximum of 75 (811508b051/django/db/models/fields/__init__.py (L994)). While we could override this limit, but I don't think we should "patch Django in Cabot".

If we ever move to Django 1.8.x, we can increase this to 254.
2015-08-24 16:56:19 -04:00
David Buxton
c54f353a6c Merge pull request #227 from elsmorian/patch-2
Update setup_dependencies.sh
2015-08-13 11:16:41 +01:00
David Buxton
376b26a211 Merge branch 'cieplik-master' 2015-08-13 11:15:43 +01:00
David Buxton
f39709d930 Merge branch 'master' of https://github.com/cieplik/cabot into cieplik-master 2015-08-13 11:15:19 +01:00
David Buxton
6d428dff0c Merge pull request #245 from spladug/configurable-user-agent
HTTP Checks: Make User-Agent configurable.
2015-08-10 09:46:06 +01:00
Neil Williams
8dfaab5641 HTTP Checks: Make User-Agent configurable.
This is helpful so the service being checked can log the source of the
health check requests more informatively.
2015-08-07 11:14:06 -07:00
Chris Elsmore
da8fe97d48 Update setup_dependencies.sh
Rewrite static alias in the HTTPS example Nginx config to be the same location as HTTP, so you don't lose static assetswhen you start only serving from HTTPS!
2015-06-11 14:39:10 +01:00
David Buxton
f5f201ca92 Merge pull request #226 from elsmorian/patch-1
example production env tweak
2015-06-11 10:59:20 +01:00
Chris Elsmore
9165f3b662 example production env tweak
Only including this as it got me for a little while - a reminder that Cabot won't see your graphite server if you leave off the trailing slash.

(It tries to access graphite.example.comevents instead of joining the URL together with a slash if not already present- suggest using urljoin to prevent this?)
2015-06-11 10:56:24 +01:00
David Buxton
a03450507f Merge pull request #221 from miguel250/patch-2
Add support for authentication with LDAP
2015-05-13 22:11:53 +01:00
Dawid Cieplinski
15c51102f8 Convert *_INTERVAL variables to int
ALERT_INTERVAL and NOTIFICATION_INTERVAL are now converted to
numbers. This allows user-defined ALERT_INTERVAL and
NOTIFICATION_INTERVAL env variables to work without throwing
TypeErrors:

    return self.run(*args, **kwargs)
  File "/cabot/cabot/cabotapp/tasks.py", line 68, in update_service
    service.update_status()
  File "/cabot/cabot/cabotapp/models.py", line 243, in update_status
    self.alert()
  File "/cabot/cabot/cabotapp/models.py", line 174, in alert
    if self.last_alert_sent and (timezone.now() - timedelta(minutes=settings.ALERT_INTERVAL)) < self.last_alert_sent:
TypeError: unsupported type for timedelta minutes component: str
2015-05-12 12:07:50 +02:00
Miguel Perez
bed4ace93b Add support for authentication with LDAP
- add necessary dependencies
2015-05-11 17:20:47 -04:00
David Buxton
2708ee7eed Merge pull request #213 from kmwhite/make-backup-task-use-consts
Using defined values in backup task
2015-04-08 17:28:12 -04:00
Kristofer M White
08de4d4a07 Using defined values in backup task
The backup fabric task was using hardcoded values that represented the
defaults for the PG_* values. This change makes use of the variables so
that the task works in different environments
2015-04-08 16:15:26 -05:00
David Buxton
1b323366fd Merge pull request #193 from 0rc/add_alerting_on_expected_metric_values
added alerting on expected metrics values rather than number of hosts re...
2015-04-05 14:58:20 -04:00
David Buxton
a9820f0818 Merge pull request #211 from efagin/master
Deployment improvements
2015-04-05 14:44:54 -04:00
Edward Fagin
fc4f64bc46 Move foreman -e command line param to earlier as it doesn't seem to be getting called properly right now - foreman keeps defaulting to whatever's in the .foreman file otherwise on Ubuntu 14.04. 2015-04-03 11:55:54 -04:00
Edward Fagin
fbaa120653 Add better string escaping to upstart template just in case somebody has an exclamation point in a Django key or something. 2015-04-03 11:47:58 -04:00
David Buxton
76ab3fb66f Merge pull request #206 from xinity/patch-1
adding alert type selection and hackpad support
2015-03-30 10:18:29 +01:00
xinity
c1c78e08fc adding alert type selection to instances 2015-03-30 10:46:55 +02:00
David Buxton
2829eed8e5 Merge pull request #191 from bonniejools/master
Remove 'No URL file' error message and set default versions for plugins.
2015-03-24 15:24:26 +00:00