Merge branch 'develop' into bug/namespace-for-server-endpoints

Author: Shakeel Mohamed

examples/analytics/README.md

@@ -19,7 +19,7 @@ meant to manage input and output of data into Splunk.
 ### AnalyticsTracker
 
 The `input.py` file defines the "input" side of the Analytics service. If you 
-wanted to log some analytics data in your app, you would the `AnalyticsTracker`
+wanted to log some analytics data in your app, you would have the `AnalyticsTracker`
 class defined in this file in order to do so.
 
 The `AnalyticsTracker` class encapsulates all the information required to log
@@ -73,7 +73,7 @@ Similarly to `AnalyticsTracker`, the `output.py` file defines the "output" side
 of the Analytics service. If you want to extract the events you logged in using
 `AnalyticsTracker`, you'd use the `AnalyticsRetriever` class.
 
-Creating an `AnalyticsTracker` instance is identical to the `AnalyticsTracker`:
+Creating an `AnalyticsRetriever` instance is identical to the `AnalyticsTracker`:
 
 ```python
 from analytics.output import AnalyticsRetriever
@@ -138,7 +138,7 @@ you can see a graph of events over time, properties, etc.
 
 We make use of the excellent open source
 [flot](http://code.google.com/p/flot/) graphing library to render
-our Javascript graphs. We also use the [`bottle.py`](bottlepy.org)
+our Javascript graphs. We also use the [`bottle.py`](http://bottlepy.org)
 micro-web framework.
 
 ## Running the Sample

examples/analytics/input.py

@@ -59,7 +59,7 @@ def __init__(self, application_name, splunk_info, index = ANALYTICS_INDEX_NAME):
 
     @staticmethod
     def encode(props):
-        encoded = " ";
+        encoded = " "
         for k,v in props.iteritems():
             # We disallow dictionaries - it doesn't quite make sense.
             assert(not isinstance(v, dict))

examples/explorer/server.py

@@ -114,7 +114,7 @@ def make_request(self, url, method, data, headers):
             response_file = StringIO.StringIO(response_text)
 
             # On errors, we also log the response text
-            self.log_message("Response: %s", response_text);
+            self.log_message("Response: %s", response_text)
 
             # Send the error response code
             self.send_response(e.code, message=e.msg)

examples/export/export.py

@@ -328,7 +328,7 @@ def main():
     service = connect(**options.kwargs)
 
     if path.exists(options.kwargs['output']):
-        if options.kwargs['recover'] == False:
+        if not options.kwargs['recover']:
             print "Export file %s exists, and recover option nor specified" % \
                   options.kwargs['output']
             sys.exit(1)

examples/github_forks/github_forks.py

@@ -50,6 +50,7 @@ def get_scheme(self):
         scheme.use_single_instance = True
 
         owner_argument = Argument("owner")
+        owner_argument.title = "Owner"
         owner_argument.data_type = Argument.data_type_string
         owner_argument.description = "Github user or organization that created the repository."
         owner_argument.required_on_create = True
@@ -59,6 +60,7 @@ def get_scheme(self):
         scheme.add_argument(owner_argument)
 
         repo_name_argument = Argument("repo_name")
+        repo_name_argument.title = "Repo Name"
         repo_name_argument.data_type = Argument.data_type_string
         repo_name_argument.description = "Name of the Github repository."
         repo_name_argument.required_on_create = True

examples/random_numbers/random_numbers.py

@@ -50,6 +50,7 @@ def get_scheme(self):
         scheme.use_single_instance = True
 
         min_argument = Argument("min")
+        min_argument.title = "Minimum"
         min_argument.data_type = Argument.data_type_number
         min_argument.description = "Minimum random number to be produced by this input."
         min_argument.required_on_create = True
@@ -59,6 +60,7 @@ def get_scheme(self):
         scheme.add_argument(min_argument)
 
         max_argument = Argument("max")
+        max_argument.title = "Maximum"
         max_argument.data_type = Argument.data_type_number
         max_argument.description = "Maximum random number to be produced by this input."
         max_argument.required_on_create = True

examples/saved_search/README.md

@@ -8,7 +8,7 @@ saved search actions.
 `list` and `delete` requires the `--name` argument to either list the contents 
 of a specific saved search or delete a specific saved search.
 
-`create` requires the `--name` argument, as well as a littany of other arguments 
+`create` requires the `--name` argument, as well as a list of any other arguments 
 to establish a saved search. The help output is seen below.
 
 Of special note is the events that can perform actions (`--actions` and 

examples/saved_search/saved_search.py

@@ -174,15 +174,15 @@ def main(argv):
             if key == "operation":
                 operation = opts.kwargs[key]
             else:
-                kwargs[key] = urllib.quote(opts.kwargs[key])
+                kwargs[key] = opts.kwargs[key]
 
     # no operation? if name present, default to list, otherwise list-all
     if not operation:
         if kwargs.has_key('name'):
             operation = 'list'
         else:
             operation = 'list-all'
-
+    
     # pre-sanitize
     if (operation != "list" and operation != "create" 
                             and operation != "delete"

examples/searchcommands_app/bin/simulate.py

@@ -86,7 +86,7 @@ class SimulateCommand(GeneratingCommand):
 
     def generate(self):
         """ Yields one random record at a time for the duration of `duration` """
-        self.logger.debug('SimulateCommand: %s' % self)  # log command line
+        self.logger.debug('SimulateCommand: %s', self)  # log command line
         if not self.records:
             if self.seed is not None:
                 random.seed(self.seed)

setup.py

@@ -168,7 +168,7 @@ def run(self):
         sdk_dir = os.path.abspath('.')
 
         def exclude(path):
-            # TODO: Replace with filter function because exclude is deprecated
+            # TODO: DVPL-5866 - Replace with filter function because exclude is deprecated
             basename = os.path.basename(path)
             for pattern in ['.DS_Store', '.idea', '*.log', '*.py[co]']:
                 if fnmatch(basename, pattern):