From d49de4ede216ff24085c4e58dfc1ea970390cb6f Mon Sep 17 00:00:00 2001 From: chenjiankun Date: Fri, 15 Sep 2017 03:17:50 +0000 Subject: Read user & password from yardstick.conf in Grafana configuration JIRA: YARDSTICK-812 Currently grafana data source configuration is hardcoding . It is a risk. so I read it from yardstick.conf. Change-Id: I8a9c8afbce6c4534fc43a0bfb5c56d67a8b59db0 Signed-off-by: chenjiankun --- api/resources/v1/env.py | 14 ++++---------- 1 file changed, 4 insertions(+), 10 deletions(-) (limited to 'api/resources/v1') diff --git a/api/resources/v1/env.py b/api/resources/v1/env.py index 47ea91643..f3c01152b 100644 --- a/api/resources/v1/env.py +++ b/api/resources/v1/env.py @@ -101,21 +101,15 @@ class V1Env(ApiResource): def _create_data_source(self, ip): url = 'http://admin:admin@{}:{}/api/datasources'.format(ip, consts.GRAFANA_PORT) - influx_conf = utils.parse_ini_file(consts.CONF_FILE) - - try: - influx_url = influx_conf['dispatcher_influxdb']['target'] - except KeyError: - LOG.exception('influxdb url not set in yardstick.conf') - raise + influx_conf = utils.parse_ini_file(consts.CONF_FILE).get('dispatcher_influxdb', {}) data = { "name": "yardstick", "type": "influxdb", "access": "proxy", - "url": influx_url, - "password": "root", - "user": "root", + "url": influx_conf.get('target', ''), + "password": influx_conf.get('password', ''), + "user": influx_conf.get('username', ''), "database": "yardstick", "basicAuth": True, "basicAuthUser": "admin", -- cgit 1.2.3-korg