sangoma / pysensu

Python Sensu Client

Sensu client for python

This is a client to interact with the Sensu API

Features

• Includes methods for some entrypoints (clients, events, checks, stashes) of the Sensu API (0.24) IMPORTANT: from 0.6.0 release we are supporting Sensu API 0.24+ .. backward compatibility with previous versions of Sensu API might be broken.
• Includes methods for handle subscriptions info (nodes subscribed to a specific channel)

Trivial Example

from pysensu.api import SensuAPI

url = 'http://localhost:4567'
my_sensu = SensuAPI(url, username=username, password=password)
print(my_sensu.get_info())

DEBUG

To enable debug import logging module and set the debug level

import logging
logging.getLogger('pysensu.api').setLevel(logging.DEBUG)

Exceptions

You can handle an exception using a code like this. Note: Sensu API give you a 404 code if a client does not exists.

import pysensu.api

url = 'http://localhost:4567'
my_sensu = pysensu.api.SensuAPI(url)
client = "testclient"

try:
    print(my_sensu.get_client_data(client))
except pysensu.api.SensuAPIException as e:
    if "404" in str(e):
        print("{} does not exists on sensu api".format(client))
    else:
        raise e

pysensu's Issues

documentation wrong

there is no pysensu.api and there is no pysensu.api.SensuAPI

Logging output in case of exception

The logger output strings in case of exception, so also if you handle the exception, you have output in your console.
To reproduce try to query a client that does not exists and handle the exception, you will find a 404 in the output.

The problem is related to logger.warning and logger.error, we could set logger to debug, so who want to debug the problem need only to set debug in his own script.

Bug on bad request status

Hi,
I tried to make a query with a client that does not exists and I have this error in
API pysensu/api.py", line 59, in _request elif resp.status_code.startswith('400'): AttributeError: 'int' object has no attribute 'startswith'

This is because the resp.status.code is an integer, so or we change it to str and use starswith
elif str(resp.status_code).startswith('400'):
or we can use
elif resp.status_code == 400:

Just to understand: why you distinguish 400 from other errors?
Maybe, do you want a different exception messages for all the 40* errors?
If so we need the startswith but with only 40 and not 400:
elif str(resp.status_code).startswith('40'):

If we decide how we want to proceed and manage 40* errors, I could write the related code and send a merge requests .