From fc6a0f36c4f05ee233d24db0220b587a5b09c54e Mon Sep 17 00:00:00 2001 From: nico Date: Wed, 26 Jun 2019 14:49:04 +0200 Subject: code optimizations * memory optimization * runtime reduction * code cleanup --- nextcloud_apps.py | 58 +++++----- nextcloud_dbsize.py | 57 +++++----- nextcloud_files.py | 57 +++++----- nextcloud_multi.py | 298 +++++++++++++++++++++++++-------------------------- nextcloud_shares.py | 109 +++++++++---------- nextcloud_storage.py | 78 +++++++------- nextcloud_users.py | 80 +++++++------- 7 files changed, 358 insertions(+), 379 deletions(-) diff --git a/nextcloud_apps.py b/nextcloud_apps.py index 69ab4ae..b398a1e 100755 --- a/nextcloud_apps.py +++ b/nextcloud_apps.py @@ -18,36 +18,29 @@ import os class NextcloudApps: - def config(self): - config = { - 'apps': [ - 'graph_title Nextcloud available App updates', - 'graph_args --base 1000 -l 0', - 'graph_vlabel updates available', - 'graph_info graph showing the number of available app updates', - 'graph_category nextcloud', - 'num_updates_available.label available app updates', - 'num_updates_available.info number of available app updates', - 'num_updates_available.min 0', - 'num_updates_available.warning 1' - ] - } - - return config - - def get_data(self, api_response): - data = { - 'nextcloud_available_updates': [] - } - + def __init__(self): + self.config = [ + # available_updates + 'graph_title Nextcloud available App updates', + 'graph_args --base 1000 -l 0', + 'graph_printf %.0lf', + 'graph_vlabel updates available', + 'graph_info graph showing the number of available app updates', + 'graph_category nextcloud', + 'num_updates_available.label available app updates', + 'num_updates_available.info number of available app updates', + 'num_updates_available.min 0', + 'num_updates_available.warning 1' + ] + self.result = list() + + def parse_data(self, api_response): # precaution for Nextcloud versions prior to version 14 version = api_response['ocs']['data']['nextcloud']['system']['version'].split(sep=".") if int(version[0]) >= 14: num_updates_available = api_response['ocs']['data']['nextcloud']['system']['apps']['num_updates_available'] - data['nextcloud_available_updates'].append('num_updates_available.value %s' % num_updates_available) - - return data + self.result.append('num_updates_available.value %s' % num_updates_available) def run(self): # init request session with specific header and credentials @@ -63,10 +56,11 @@ class NextcloudApps: # if status code is successful continue if r.status_code == 200: - result = self.get_data(r.json()) + self.parse_data(r.json()) - # for key in results print every entry in dict - [print('\n'.join(result[key])) for key in result.keys()] + # output results to stdout + for el in self.result: + print(el, file=sys.stdout) elif r.status_code == 996: print('server error') @@ -82,10 +76,14 @@ class NextcloudApps: if sys.argv.__len__() >= 2: # check if first argument is config or autoconf if not fetch data if sys.argv[1] == "config": - # for key in config().keys() print every entry in dict - [print('\n'.join(self.config()[key])) for key in self.config().keys()] + # output config list to stdout + for el in self.config: + print(el, file=sys.stdout) + + # if DIRTYCONFIG true also return the corresponding values if os.environ.get('MUNIN_CAP_DIRTYCONFIG') == '1': self.run() + elif sys.argv[1] == 'autoconf': if None in [os.environ.get('username'), os.environ.get('password')]: print('env variables are missing') diff --git a/nextcloud_dbsize.py b/nextcloud_dbsize.py index b9377a7..f6cc4e1 100755 --- a/nextcloud_dbsize.py +++ b/nextcloud_dbsize.py @@ -18,32 +18,24 @@ import os class NextcloudDB: - def config(self): - config = { - 'dbsize': [ - 'graph_title Nextcloud Database Size', - 'graph_args --base 1024 -l 0', - 'graph_vlabel size in byte', - 'graph_info graph showing the database size in byte', - 'graph_category nextcloud', - 'db_size.label database size in byte', - 'db_size.info users connected in the last 5 minutes', - 'db_size.draw AREA', - 'db_size.min 0' - ] - } - - return config - - def get_data(self, api_response): - data = { - 'nextcloud_dbsize': [], - } - + def __init__(self): + self.config = [ + # dbsize + 'graph_title Nextcloud Database Size', + 'graph_args --base 1024 -l 0', + 'graph_vlabel size in byte', + 'graph_info graph showing the database size in byte', + 'graph_category nextcloud', + 'db_size.label database size in byte', + 'db_size.info users connected in the last 5 minutes', + 'db_size.draw AREA', + 'db_size.min 0' + ] + self.result = list() + + def parse_data(self, api_response): dbsize = api_response['ocs']['data']['server']['database']['size'] - data['nextcloud_dbsize'].append('db_size.value %s' % dbsize) - - return data + self.result.append('db_size.value %s' % dbsize) def run(self): # init request session with specific header and credentials @@ -59,10 +51,11 @@ class NextcloudDB: # if status code is successful continue if r.status_code == 200: - result = self.get_data(r.json()) + self.parse_data(r.json()) - # for key in results print every entry in dict - [print('\n'.join(result[key])) for key in result.keys()] + # output results to stdout + for el in self.result: + print(el, file=sys.stdout) elif r.status_code == 996: print('server error') @@ -78,10 +71,14 @@ class NextcloudDB: if sys.argv.__len__() >= 2: # check if first argument is config or autoconf if not fetch data if sys.argv[1] == "config": - # for key in config().keys() print every entry in dict - [print('\n'.join(self.config()[key])) for key in self.config().keys()] + # output config list to stdout + for el in self.config: + print(el, file=sys.stdout) + + # if DIRTYCONFIG true also return the corresponding values if os.environ.get('MUNIN_CAP_DIRTYCONFIG') == '1': self.run() + elif sys.argv[1] == 'autoconf': if None in [os.environ.get('username'), os.environ.get('password')]: print('env variables are missing') diff --git a/nextcloud_files.py b/nextcloud_files.py index ee12d77..f56860a 100755 --- a/nextcloud_files.py +++ b/nextcloud_files.py @@ -18,32 +18,24 @@ import os class NextcloudStorage: - def config(self): - config = { - 'filecount': [ - 'graph_title Nextcloud Files', - 'graph_args --base 1000 -l 0', - 'graph_vlabel number of files', - 'graph_info graph showing the number of files', - 'graph_category nextcloud', - 'num_files.label number of files', - 'num_files.info current number of files in the repository', - 'num_files.min 0' - ] - } - - return config - - def get_data(self, api_response): - data = { - 'nextcloud_filecount': [], - } - - # append the total number of files present + def __init__(self): + self.config = [ + # filecount + 'graph_title Nextcloud Files', + 'graph_args --base 1000 -l 0', + 'graph_printf %.0lf', + 'graph_vlabel number of files', + 'graph_info graph showing the number of files', + 'graph_category nextcloud', + 'num_files.label number of files', + 'num_files.info current number of files in the repository', + 'num_files.min 0' + ] + self.result = list() + + def parse_data(self, api_response): num_files = api_response['ocs']['data']['nextcloud']['storage']['num_files'] - data['nextcloud_filecount'].append('num_files.value %s' % num_files) - - return data + self.result.append('num_files.value %s' % num_files) def run(self): # init request session with specific header and credentials @@ -59,10 +51,11 @@ class NextcloudStorage: # if status code is successful continue if r.status_code == 200: - result = self.get_data(r.json()) + self.parse_data(r.json()) - # for key in results print every entry in dict - [print('\n'.join(result[key])) for key in result.keys()] + # output results to stdout + for el in self.result: + print(el, file=sys.stdout) elif r.status_code == 996: print('server error') @@ -78,10 +71,14 @@ class NextcloudStorage: if sys.argv.__len__() >= 2: # check if first argument is config or autoconf if not fetch data if sys.argv[1] == "config": - # for key in config().keys() print every entry in dict - [print('\n'.join(self.config()[key])) for key in self.config().keys()] + # output config list to stdout + for el in self.config: + print(el, file=sys.stdout) + + # if DIRTYCONFIG true also return the corresponding values if os.environ.get('MUNIN_CAP_DIRTYCONFIG') == '1': self.run() + elif sys.argv[1] == 'autoconf': if None in [os.environ.get('username'), os.environ.get('password')]: print('env variables are missing') diff --git a/nextcloud_multi.py b/nextcloud_multi.py index a4c02d2..f40ec9b 100755 --- a/nextcloud_multi.py +++ b/nextcloud_multi.py @@ -22,157 +22,151 @@ import os class NextcloudMultiGraph: - def config(self): - config = { - 'users': [ - 'multigraph nextcloud_users', - 'graph_title Nextcloud User Activity', - 'graph_args --base 1000 -l 0', - 'graph_printf %.0lf', - 'graph_vlabel connected users', - 'graph_info graph showing the number of connected user', - 'graph_category nextcloud', - 'last5minutes.label last 5 minutes', - 'last5minutes.info users connected in the last 5 minutes', - 'last5minutes.min 0', - 'last1hour.label last hour', - 'last1hour.info users connected in the last hour', - 'last1hour.min 0', - 'last24hours.label last 24 hours', - 'last24hours.info users connected in the last 24 hours', - 'last24hours.min 0', - 'num_users.label number of users', - 'num_users.info total number of users', - 'num_users.min 0' - ], - 'shares': [ - 'multigraph nextcloud_shares', - 'graph_title Nextcloud Shares', - 'graph_args --base 1000 -l 0', - 'graph_vlabel number of shares', - 'graph_info graph showing the number of shares', - 'graph_category nextcloud', - 'num_shares.label total number of shares', - 'num_shares.info current over all total of shares', - 'num_shares.min 0', - 'num_shares_user.label user shares', - 'num_shares_user.info current total of user shares', - 'num_shares_user.min 0', - 'num_shares_groups.label group shares', - 'num_shares_groups.info current total of group shares', - 'num_shares_groups.min 0', - 'num_shares_link.label link shares', - 'num_shares_link.info current total of shares through a link', - 'num_shares_link.min 0', - 'num_shares_mail.label mail shares', - 'num_shares_mail.info current total of mail shares', - 'num_shares_mail.min 0', - 'num_shares_room.label room shares', - 'num_shares_room.info current total of room shares', - 'num_shares_room.min 0', - 'num_shares_link_no_password.label link shares without a password', - 'num_shares_link_no_password.info current total of shares through a link without a password protection', - 'num_shares_link_no_password.min 0', - 'num_fed_shares_sent.label federated shares sent', - 'num_fed_shares_sent.info current total of federated shares sent', - 'num_fed_shares_sent.min 0', - 'num_fed_shares_received.label federated shares recieved', - 'num_fed_shares_received.info current total of federated shares recieved', - 'num_fed_shares_received.min 0' - ], - 'dbsize': [ - 'multigraph nextcloud_dbsize', - 'graph_title Nextcloud Database Size', - 'graph_args --base 1024 -l 0', - 'graph_vlabel size in byte', - 'graph_info graph showing the database size in byte', - 'graph_category nextcloud', - 'db_size.label database size in byte', - 'db_size.info users connected in the last 5 minutes', - 'db_size.draw AREA', - 'db_size.min 0' - ], - 'available_updates': [ - 'multigraph nextcloud_available_updates', - 'graph_title Nextcloud available App updates', - 'graph_args --base 1000 -l 0', - 'graph_vlabel updates available', - 'graph_info graph showing the number of available app updates', - 'graph_category nextcloud', - 'num_updates_available.label available app updates', - 'num_updates_available.info number of available app updates', - 'num_updates_available.min 0', - 'num_updates_available.warning 1' - ], - 'storages': [ - 'multigraph nextcloud_storages', - 'graph_title Nextcloud Storages', - 'graph_args --base 1000 -l 0', - 'graph_vlabel number', - 'graph_info graph showing the number of storages', - 'graph_category nextcloud', - 'num_storages.label total number of storages', - 'num_storages.info current over all total of storages', - 'num_storages.min 0', - 'num_storages_local.label number of local storages', - 'num_storages_local.info current over all total of storage', - 'num_storages_local.min 0', - 'num_storages_home.label number of home storages', - 'num_storages_home.info current over all total of storage', - 'num_storages_home.min 0', - 'num_storages_other.label number of other storages', - 'num_storages_other.info current over all total of storage', - 'num_storages_other.min 0' - ], - "filecount": [ - 'multigraph nextcloud_filecount', - 'graph_title Nextcloud Files', - 'graph_args --base 1000 -l 0', - 'graph_vlabel number of files', - 'graph_info graph showing the number of files', - 'graph_category nextcloud', - 'num_files.label number of files', - 'num_files.info current number of files in the repository', - 'num_files.min 0' - ] - } - - return config - - def get_data(self, api_response): - data = { - 'nextcloud_users': [], - 'nextcloud_shares': [], - 'nextcloud_dbsize': [], - 'nextcloud_available_updates': [], - 'nextcloud_storages': [], - 'nextcloud_filecount': [] - } - + def __init__(self): + self.config = [ + # users + 'multigraph nextcloud_users', + 'graph_title Nextcloud User Activity', + 'graph_args --base 1000 -l 0', + 'graph_printf %.0lf', + 'graph_vlabel connected users', + 'graph_info graph showing the number of connected user', + 'graph_category nextcloud', + 'last5minutes.label last 5 minutes', + 'last5minutes.info users connected in the last 5 minutes', + 'last5minutes.min 0', + 'last1hour.label last hour', + 'last1hour.info users connected in the last hour', + 'last1hour.min 0', + 'last24hours.label last 24 hours', + 'last24hours.info users connected in the last 24 hours', + 'last24hours.min 0', + 'num_users.label number of users', + 'num_users.info total number of users', + 'num_users.min 0', + + # shares + 'multigraph nextcloud_shares', + 'graph_title Nextcloud Shares', + 'graph_args --base 1000 -l 0', + 'graph_printf %.0lf', + 'graph_vlabel number of shares', + 'graph_info graph showing the number of shares', + 'graph_category nextcloud', + 'num_shares.label total number of shares', + 'num_shares.info current over all total of shares', + 'num_shares.min 0', + 'num_shares_user.label user shares', + 'num_shares_user.info current total of user shares', + 'num_shares_user.min 0', + 'num_shares_groups.label group shares', + 'num_shares_groups.info current total of group shares', + 'num_shares_groups.min 0', + 'num_shares_link.label link shares', + 'num_shares_link.info current total of shares through a link', + 'num_shares_link.min 0', + 'num_shares_mail.label mail shares', + 'num_shares_mail.info current total of mail shares', + 'num_shares_mail.min 0', + 'num_shares_room.label room shares', + 'num_shares_room.info current total of room shares', + 'num_shares_room.min 0', + 'num_shares_link_no_password.label link shares without a password', + 'num_shares_link_no_password.info current total of shares through a link without a password protection', + 'num_shares_link_no_password.min 0', + 'num_fed_shares_sent.label federated shares sent', + 'num_fed_shares_sent.info current total of federated shares sent', + 'num_fed_shares_sent.min 0', + 'num_fed_shares_received.label federated shares recieved', + 'num_fed_shares_received.info current total of federated shares recieved', + 'num_fed_shares_received.min 0', + + # dbsize + 'multigraph nextcloud_dbsize', + 'graph_title Nextcloud Database Size', + 'graph_args --base 1024 -l 0', + 'graph_vlabel size in byte', + 'graph_info graph showing the database size in byte', + 'graph_category nextcloud', + 'db_size.label database size in byte', + 'db_size.info users connected in the last 5 minutes', + 'db_size.draw AREA', + 'db_size.min 0', + + # available_updates + 'multigraph nextcloud_available_updates', + 'graph_title Nextcloud available App updates', + 'graph_args --base 1000 -l 0', + 'graph_printf %.0lf', + 'graph_vlabel updates available', + 'graph_info graph showing the number of available app updates', + 'graph_category nextcloud', + 'num_updates_available.label available app updates', + 'num_updates_available.info number of available app updates', + 'num_updates_available.min 0', + 'num_updates_available.warning 1', + + # storages + 'multigraph nextcloud_storages', + 'graph_title Nextcloud Storages', + 'graph_args --base 1000 -l 0', + 'graph_printf %.0lf', + 'graph_vlabel number', + 'graph_info graph showing the number of storages', + 'graph_category nextcloud', + 'num_storages.label total number of storages', + 'num_storages.info current over all total of storages', + 'num_storages.min 0', + 'num_storages_local.label number of local storages', + 'num_storages_local.info current over all total of storage', + 'num_storages_local.min 0', + 'num_storages_home.label number of home storages', + 'num_storages_home.info current over all total of storage', + 'num_storages_home.min 0', + 'num_storages_other.label number of other storages', + 'num_storages_other.info current over all total of storage', + 'num_storages_other.min 0', + + # filecount + 'multigraph nextcloud_filecount', + 'graph_title Nextcloud Files', + 'graph_args --base 1000 -l 0', + 'graph_printf %.0lf', + 'graph_vlabel number of files', + 'graph_info graph showing the number of files', + 'graph_category nextcloud', + 'num_files.label number of files', + 'num_files.info current number of files in the repository', + 'num_files.min 0' + ] + self.result = list() + + def parse_data(self, api_response): # users users = api_response['ocs']['data']['activeUsers'] num_users = api_response['ocs']['data']['nextcloud']['storage']['num_users'] - data['nextcloud_users'].append('multigraph nextcloud_users') + self.result.append('multigraph nextcloud_users') # append for every key in users the key and the value to the results - [data['nextcloud_users'].append(str(key) + ".value " + str(users[key])) - for key in users.keys()] + for key, value in users.items(): + self.result.append('{k}.value {v}'.format(k=key, v=value)) # append total number of users - data['nextcloud_users'].append('num_users.value %s' % num_users) + self.result.append('num_users.value %s' % num_users) # shares shares = api_response['ocs']['data']['nextcloud']['shares'] - data['nextcloud_shares'].append('multigraph nextcloud_shares') + self.result.append('multigraph nextcloud_shares') # append for every key in shares the key and the value if the key starts with "num" - [data['nextcloud_shares'].append(str(key) + ".value " + str(shares[key])) - for key in shares if key.startswith('num')] + for key, value in shares.items(): + if key.startswith('num'): + self.result.append('{k}.value {v}'.format(k=key, v=value)) # dbsize dbsize = api_response['ocs']['data']['server']['database']['size'] - data['nextcloud_dbsize'].append('multigraph nextcloud_dbsize') - data['nextcloud_dbsize'].append('db_size.value %s' % dbsize) + self.result.append('multigraph nextcloud_dbsize') + self.result.append('db_size.value %s' % dbsize) # app updates # precaution for Nextcloud versions prior to version 14 @@ -180,23 +174,22 @@ class NextcloudMultiGraph: if int(version[0]) >= 14: num_updates_available = api_response['ocs']['data']['nextcloud']['system']['apps']['num_updates_available'] - data['nextcloud_available_updates'].append('multigraph nextcloud_available_updates') - data['nextcloud_available_updates'].append('num_updates_available.value %s' % num_updates_available) + self.result.append('multigraph nextcloud_available_updates') + self.result.append('num_updates_available.value %s' % num_updates_available) # storage storage = api_response['ocs']['data']['nextcloud']['storage'] - data['nextcloud_storages'].append('multigraph nextcloud_storages') + self.result.append('multigraph nextcloud_storages') # append for every key in storage the key and the value if the key starts with "num" - [data['nextcloud_storages'].append(str(key) + ".value " + str(storage[key])) - for key in storage if key.startswith('num_storages')] + for key, value in storage.items(): + if key.startswith('num_storages'): + self.result.append('{k}.value {v}'.format(k=key, v=value)) # filecount num_files = api_response['ocs']['data']['nextcloud']['storage']['num_files'] - data['nextcloud_filecount'].append('multigraph nextcloud_filecount') - data['nextcloud_filecount'].append('num_files.value %s' % num_files) - - return data + self.result.append('multigraph nextcloud_filecount') + self.result.append('num_files.value %s' % num_files) def run(self): # init request session with specific header and credentials @@ -212,10 +205,11 @@ class NextcloudMultiGraph: # if status code is successful continue if r.status_code == 200: - result = self.get_data(r.json()) + self.parse_data(r.json()) - # for key in results print every entry in dict - [print('\n'.join(result[key])) for key in result.keys()] + # output results to stdout + for el in self.result: + print(el, file=sys.stdout) elif r.status_code == 996: print('server error') @@ -231,10 +225,14 @@ class NextcloudMultiGraph: if sys.argv.__len__() >= 2: # check if first argument is config or autoconf if not fetch data if sys.argv[1] == "config": - # for key in config().keys() print every entry in dict - [print('\n'.join(self.config()[key])) for key in self.config().keys()] + # output config list to stdout + for el in self.config: + print(el, file=sys.stdout) + + # if DIRTYCONFIG true also return the corresponding values if os.environ.get('MUNIN_CAP_DIRTYCONFIG') == '1': self.run() + elif sys.argv[1] == 'autoconf': if None in [os.environ.get('username'), os.environ.get('password')]: print('env variables are missing') diff --git a/nextcloud_shares.py b/nextcloud_shares.py index 4c70d06..5bfa28f 100755 --- a/nextcloud_shares.py +++ b/nextcloud_shares.py @@ -18,60 +18,52 @@ import os class NextcloudShares: - def config(self): - config = { - 'shares': [ - 'graph_title Nextcloud Shares', - 'graph_args --base 1000 -l 0', - 'graph_vlabel number of shares', - 'graph_info graph showing the number of shares', - 'graph_category nextcloud', - 'num_shares.label total number of shares', - 'num_shares.info current over all total of shares', - 'num_shares.min 0', - 'num_shares_user.label user shares', - 'num_shares_user.info current total of user shares', - 'num_shares_user.min 0', - 'num_shares_groups.label group shares', - 'num_shares_groups.info current total of group shares', - 'num_shares_groups.min 0', - 'num_shares_link.label link shares', - 'num_shares_link.info current total of shares through a link', - 'num_shares_link.min 0', - 'num_shares_mail.label mail shares', - 'num_shares_mail.info current total of mail shares', - 'num_shares_mail.min 0', - 'num_shares_room.label room shares', - 'num_shares_room.info current total of room shares', - 'num_shares_room.min 0', - 'num_shares_link_no_password.label link shares without a password', - 'num_shares_link_no_password.info current total of shares through a link without a password protection', - 'num_shares_link_no_password.min 0', - 'num_fed_shares_sent.label federated shares sent', - 'num_fed_shares_sent.info current total of federated shares sent', - 'num_fed_shares_sent.min 0', - 'num_fed_shares_received.label federated shares recieved', - 'num_fed_shares_received.info current total of federated shares recieved', - 'num_fed_shares_received.min 0' - ] - } - - return config - - def get_data(self, api_response): - data = { - 'nextcloud_shares': [], - } - - # shares + def __init__(self): + self.config = [ + # shares + 'graph_title Nextcloud Shares', + 'graph_args --base 1000 -l 0', + 'graph_printf %.0lf', + 'graph_vlabel number of shares', + 'graph_info graph showing the number of shares', + 'graph_category nextcloud', + 'num_shares.label total number of shares', + 'num_shares.info current over all total of shares', + 'num_shares.min 0', + 'num_shares_user.label user shares', + 'num_shares_user.info current total of user shares', + 'num_shares_user.min 0', + 'num_shares_groups.label group shares', + 'num_shares_groups.info current total of group shares', + 'num_shares_groups.min 0', + 'num_shares_link.label link shares', + 'num_shares_link.info current total of shares through a link', + 'num_shares_link.min 0', + 'num_shares_mail.label mail shares', + 'num_shares_mail.info current total of mail shares', + 'num_shares_mail.min 0', + 'num_shares_room.label room shares', + 'num_shares_room.info current total of room shares', + 'num_shares_room.min 0', + 'num_shares_link_no_password.label link shares without a password', + 'num_shares_link_no_password.info current total of shares through a link without a password protection', + 'num_shares_link_no_password.min 0', + 'num_fed_shares_sent.label federated shares sent', + 'num_fed_shares_sent.info current total of federated shares sent', + 'num_fed_shares_sent.min 0', + 'num_fed_shares_received.label federated shares recieved', + 'num_fed_shares_received.info current total of federated shares recieved', + 'num_fed_shares_received.min 0' + ] + self.result = list() + + def parse_data(self, api_response): shares = api_response['ocs']['data']['nextcloud']['shares'] - data['nextcloud_shares'].append('multigraph nextcloud_shares') # append for every key in shares the key and the value if the key starts with "num" - [data['nextcloud_shares'].append(str(key) + ".value " + str(shares[key])) - for key in shares if key.startswith('num')] - - return data + for key, value in shares.items(): + if key.startswith('num'): + self.result.append('{k}.value {v}'.format(k=key, v=value)) def run(self): # init request session with specific header and credentials @@ -87,10 +79,11 @@ class NextcloudShares: # if status code is successful continue if r.status_code == 200: - result = self.get_data(r.json()) + self.parse_data(r.json()) - # for key in results print every entry in dict - [print('\n'.join(result[key])) for key in result.keys()] + # output results to stdout + for el in self.result: + print(el, file=sys.stdout) elif r.status_code == 996: print('server error') @@ -106,10 +99,14 @@ class NextcloudShares: if sys.argv.__len__() >= 2: # check if first argument is config or autoconf if not fetch data if sys.argv[1] == "config": - # for key in config().keys() print every entry in dict - [print('\n'.join(self.config()[key])) for key in self.config().keys()] + # output config list to stdout + for el in self.config: + print(el, file=sys.stdout) + + # if DIRTYCONFIG true also return the corresponding values if os.environ.get('MUNIN_CAP_DIRTYCONFIG') == '1': self.run() + elif sys.argv[1] == 'autoconf': if None in [os.environ.get('username'), os.environ.get('password')]: print('env variables are missing') diff --git a/nextcloud_storage.py b/nextcloud_storage.py index ee3d7d2..7a2e86a 100755 --- a/nextcloud_storage.py +++ b/nextcloud_storage.py @@ -18,44 +18,37 @@ import os class NextcloudStorage: - def config(self): - config = { - 'storage': [ - 'graph_title Nextcloud Storages', - 'graph_args --base 1000 -l 0', - 'graph_vlabel number of storage', - 'graph_info graph showing the number of storages', - 'graph_category nextcloud', - 'num_storages.label total number of storages', - 'num_storages.info current over all total of storages', - 'num_storages.min 0', - 'num_storages_local.label number of local storages', - 'num_storages_local.info current over all total of storage', - 'num_storages_local.min 0', - 'num_storages_home.label number of home storages', - 'num_storages_home.info current over all total of storage', - 'num_storages_home.min 0', - 'num_storages_other.label number of other storages', - 'num_storages_other.info current over all total of storage', - 'num_storages_other.min 0' - ] - } - - return config - - def get_data(self, api_response): - data = { - 'nextcloud_storages': [], - } - - # storage + def __init__(self): + self.config = [ + # storages + 'graph_title Nextcloud Storages', + 'graph_args --base 1000 -l 0', + 'graph_printf %.0lf', + 'graph_vlabel number', + 'graph_info graph showing the number of storages', + 'graph_category nextcloud', + 'num_storages.label total number of storages', + 'num_storages.info current over all total of storages', + 'num_storages.min 0', + 'num_storages_local.label number of local storages', + 'num_storages_local.info current over all total of storage', + 'num_storages_local.min 0', + 'num_storages_home.label number of home storages', + 'num_storages_home.info current over all total of storage', + 'num_storages_home.min 0', + 'num_storages_other.label number of other storages', + 'num_storages_other.info current over all total of storage', + 'num_storages_other.min 0' + ] + self.result = list() + + def parse_data(self, api_response): storage = api_response['ocs']['data']['nextcloud']['storage'] # append for every key in storage the key and the value if the key starts with "num" - [data['nextcloud_storages'].append(str(key) + ".value " + str(storage[key])) - for key in storage if key.startswith('num_storages')] - - return data + for key, value in storage.items(): + if key.startswith('num_storages'): + self.result.append('{k}.value {v}'.format(k=key, v=value)) def run(self): # init request session with specific header and credentials @@ -71,10 +64,11 @@ class NextcloudStorage: # if status code is successful continue if r.status_code == 200: - result = self.get_data(r.json()) + self.parse_data(r.json()) - # for key in results print every entry in dict - [print('\n'.join(result[key])) for key in result.keys()] + # output results to stdout + for el in self.result: + print(el, file=sys.stdout) elif r.status_code == 996: print('server error') @@ -90,10 +84,14 @@ class NextcloudStorage: if sys.argv.__len__() >= 2: # check if first argument is config or autoconf if not fetch data if sys.argv[1] == "config": - # for key in config().keys() print every entry in dict - [print('\n'.join(self.config()[key])) for key in self.config().keys()] + # output config list to stdout + for el in self.config: + print(el, file=sys.stdout) + + # if DIRTYCONFIG true also return the corresponding values if os.environ.get('MUNIN_CAP_DIRTYCONFIG') == '1': self.run() + elif sys.argv[1] == 'autoconf': if None in [os.environ.get('username'), os.environ.get('password')]: print('env variables are missing') diff --git a/nextcloud_users.py b/nextcloud_users.py index b2c651d..cfaae72 100755 --- a/nextcloud_users.py +++ b/nextcloud_users.py @@ -18,48 +18,37 @@ import os class NextcloudUsers: - def config(self): - config = { - 'users': [ - 'graph_title Nextcloud User Activity', - 'graph_args --base 1000 -l 0', - 'graph_printf %.0lf', - 'graph_vlabel connected users', - 'graph_info graph showing the number of connected user', - 'graph_category nextcloud', - 'last5minutes.label last 5 minutes', - 'last5minutes.info users connected in the last 5 minutes', - 'last5minutes.min 0', - 'last1hour.label last hour', - 'last1hour.info users connected in the last hour', - 'last1hour.min 0', - 'last24hours.label last 24 hours', - 'last24hours.info users connected in the last 24 hours', - 'last24hours.min 0', - 'num_users.label number of users', - 'num_users.info total number of users', - 'num_users.min 0' - ] - } - - return config - - def get_data(self, api_response): - data = { - 'nextcloud_users': [], - } - # users + def __init__(self): + self.config = [ + # users + 'graph_title Nextcloud User Activity', + 'graph_args --base 1000 -l 0', + 'graph_printf %.0lf', + 'graph_vlabel connected users', + 'graph_info graph showing the number of connected user', + 'graph_category nextcloud', + 'last5minutes.label last 5 minutes', + 'last5minutes.info users connected in the last 5 minutes', + 'last5minutes.min 0', + 'last1hour.label last hour', + 'last1hour.info users connected in the last hour', + 'last1hour.min 0', + 'last24hours.label last 24 hours', + 'last24hours.info users connected in the last 24 hours', + 'last24hours.min 0', + 'num_users.label number of users', + 'num_users.info total number of users', + 'num_users.min 0' + ] + self.result = list() + + def parse_data(self, api_response): users = api_response['ocs']['data']['activeUsers'] num_users = api_response['ocs']['data']['nextcloud']['storage']['num_users'] # append for every key in users the key and the value to the results - [data['nextcloud_users'].append(str(key) + ".value " + str(users[key])) - for key in users.keys()] - - # append total number of users - data['nextcloud_users'].append('num_users.value %s' % num_users) - - return data + for key, value in users.items(): + self.result.append('{k}.value {v}'.format(k=key, v=value)) def run(self): # init request session with specific header and credentials @@ -75,10 +64,11 @@ class NextcloudUsers: # if status code is successful continue if r.status_code == 200: - result = self.get_data(r.json()) + self.parse_data(r.json()) - # for key in results print every entry in dict - [print('\n'.join(result[key])) for key in result.keys()] + # output results to stdout + for el in self.result: + print(el, file=sys.stdout) elif r.status_code == 996: print('server error') @@ -94,10 +84,14 @@ class NextcloudUsers: if sys.argv.__len__() >= 2: # check if first argument is config or autoconf if not fetch data if sys.argv[1] == "config": - # for key in config().keys() print every entry in dict - [print('\n'.join(self.config()[key])) for key in self.config().keys()] + # output config list to stdout + for el in self.config: + print(el, file=sys.stdout) + + # if DIRTYCONFIG true also return the corresponding values if os.environ.get('MUNIN_CAP_DIRTYCONFIG') == '1': self.run() + elif sys.argv[1] == 'autoconf': if None in [os.environ.get('username'), os.environ.get('password')]: print('env variables are missing') -- cgit v1.2.3-18-g5258