fixing the syntax of the unassigned shards check to conform to pep8 syntax guidelines

Author: stack72

check_es_unassigned_shards.py

@@ -9,41 +9,46 @@
 except ImportError:
     import simplejson as json
 
-class ESShardsCheck(NagiosCheck):
 
-  def __init__(self):
+class ESShardsCheck(NagiosCheck):
 
-    NagiosCheck.__init__(self)
+    def __init__(self):
 
-    self.add_option('H','host', 'host', 'The cluster to check')
-    self.add_option('P','port', 'port', 'The ES port - defaults to 9200')
+        NagiosCheck.__init__(self)
 
+        self.add_option('H', 'host', 'host', 'The cluster to check')
+        self.add_option('P', 'port', 'port', 'The ES port - defaults to 9200')
 
-  def check(self, opts, args):
-    host = opts.host
-    port = int(opts.port or '9200')
+    def check(self, opts, args):
+        host = opts.host
+        port = int(opts.port or '9200')
 
-    try:
-      response = urllib2.urlopen(r'http://%s:%d/_cluster/health' %(host, port))
-    except urllib2.HTTPError, e:
-      raise Status('unknown', ("API failure", None, "API failure:\n\n%s" % str(e)))
-    except urllib2.URLError, e:
-      raise Status('critical', (e.reason))
+        try:
+            response = urllib2.urlopen(r'http://%s:%d/_cluster/health'
+                                       % (host, port))
+        except urllib2.HTTPError, e:
+            raise Status('unknown', ("API failure", None,
+                         "API failure:\n\n%s" % str(e)))
+        except urllib2.URLError, e:
+            raise Status('critical', (e.reason))
 
-    response_body = response.read()
+        response_body = response.read()
 
-    try:
-      es_cluster_health = json.loads(response_body)
-    except ValueError:
-      raise Status('unknown', ("API returned nonsense",))
+        try:
+            es_cluster_health = json.loads(response_body)
+        except ValueError:
+            raise Status('unknown', ("API returned nonsense",))
 
-    unassigned_shards = es_cluster_health['unassigned_shards']
+        unassigned_shards = es_cluster_health['unassigned_shards']
 
-    if es_cluster_health['unassigned_shards'] != unassigned_shards:
-      raise Status('CRITICAL', "There are '%s' unassigned shards in the cluster" %unassigned_shards)
-    else:
-      raise Status('OK', "All shards in the cluster are currently assigned")
+        if es_cluster_health['unassigned_shards'] != unassigned_shards:
+            raise Status('CRITICAL',
+                         "There are '%s' unassigned shards in the cluster"
+                         % (unassigned_shards))
+        else:
+            raise Status('OK',
+                         "All shards in the cluster are currently assigned")
 
 
 if __name__ == "__main__":
-  ESShardsCheck().run()
+    ESShardsCheck().run()