x/stackalytics
Code Issues Proposed changes

Revert "Add config option default_data_file"

Browse Source 
default_data_uri option handles local file already, path need to be
prefixed with 'file://', for example:
default_data_uri = file:///path/to/default_data.json

This reverts commit 1888bfb60cdab2fec08da417a2a45ec838fa1322.

Change-Id: I558238dbfc633793013451f836c7ce449e94c6c5
Signed-off-by: Pierre-Louis Bonicoli <pierre-louis.bonicoli@libregerbil.fr>
This commit is contained in:
Pierre-Louis Bonicoli 2017-03-09 11:28:45 +01:00
parent 224078d7c5
commit dd4ba31bfd
3 changed files with 1 additions and 14 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
stackalytics/processor/config.py
View File

@ -24,9 +24,6 @@ CONNECTION_OPTS = [
]
PROCESSOR_OPTS = [
cfg.StrOpt('default-data-file',
help='Default data file. This file is used instead of '
'default-data-uri if specifying.'),
cfg.StrOpt('default-data-uri',
default='https://git.openstack.org/cgit/'
'openstack/stackalytics/plain/etc/default_data.json',

5
stackalytics/processor/main.py
View File

@ -313,10 +313,7 @@ def main():
runtime_storage_inst = runtime_storage.get_runtime_storage(
CONF.runtime_storage_uri)
if CONF.default_data_file:
default_data = utils.read_json_from_file(CONF.default_data_file)
else:
default_data = utils.read_json_from_uri(CONF.default_data_uri)
default_data = utils.read_json_from_uri(CONF.default_data_uri)
if not default_data:
LOG.critical('Unable to load default data')
return not 0

7
stackalytics/processor/utils.py
View File

@ -22,7 +22,6 @@ import re
import time
import iso8601
import json
from oslo_config import cfg
from oslo_log import log as logging
import requests
@ -152,12 +151,6 @@ def read_json_from_uri(uri, session=None):
{'error': e, 'uri': uri})
def read_json_from_file(filename):
with open(filename) as json_data:
data = json.load(json_data)
return data
def read_yaml_from_uri(uri):
try:
return yaml.safe_load(read_uri(uri))