openstack/monasca-agent
Code Issues Proposed changes

Merge "Make plugin collection time warning value configurable"

Browse Source
This commit is contained in:
Jenkins
2016-07-05 18:18:44 +00:00
committed by Gerrit Code Review
parent baf61ab6dd 68d6a7a0f2
commit 872fe374bd
3 changed files with 5 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
agent.yaml.template
View File

@@ -80,7 +80,7 @@ Main:
pool_full_max_retries: {args.pool_full_max_retries}
# Threshold value for warning on collection time of each check (in seconds)
sub_collection_warn: 6
sub_collection_warn: {args.plugin_collect_time_warn}
# Collector restart interval (in hours)
collector_restart_interval: 24

1
docs/Agent.md
View File

@@ -98,6 +98,7 @@ All parameters require a '--' before the parameter such as '--verbose'. Run `mon
| check_frequency | How often to run metric collection in seconds | 60 |
| num_collector_threads | Number of threads to use in collector for running checks | 1 |
| pool_full_max_retries | Maximum number of collection cycles where all of the threads in the pool are still running plugins before the collector will exit| 4 |
| plugin_collect_time_warn | Number of seconds a plugin collection time exceeds that causes a warning to be logged for that plugin| 6 |
| keystone_url | This is a required parameter that specifies the url of the keystone api for retrieving tokens. It must be a v3 endpoint. | http://192.168.1.5:35357/v3 |
| dimensions | A comma separated list of key:value pairs to include as dimensions in all submitted metrics| region:a,az:1 |
| service | This is an optional parameter that specifies the name of the service associated with this particular node | nova, cinder, myservice |

3
monasca_setup/main.py
View File

@@ -211,6 +211,9 @@ def parse_arguments(parser):
"in the pool are still running plugins before the " +
"collector will exit and be restart",
type=validate_positive, default=4)
parser.add_argument('--plugin_collect_time_warn', help="Number of seconds a plugin collection time exceeds " +
"that causes a warning to be logged for that plugin",
type=validate_positive, default=6)
parser.add_argument('--dimensions', help="Additional dimensions to set for all metrics. A comma separated list " +
"of name/value pairs, 'name:value,name2:value2'")
parser.add_argument('--ca_file', help="Sets the path to the ca certs file if using certificates. " +