Fixes bug 853031 - Updated configuration of processors to accept new ela... #1148

Merged
merged 1 commit into from Mar 25, 2013

Conversation

Projects
None yet
2 participants
Contributor

adngdb commented Mar 21, 2013

...sticsearch keys.

I am unable to test this locally, mainly because of the lack of a local HBase, but the config should load fine (I had some errors that I fixed), so I'd like to test this on stage asap.

scripts/startProcessor.py
- # doc: a url to submit crash_ids for Elastic Search (use %s in place of the
- # crash_id) (leave blank to disable)
+ # name: destination.storage2.elasticsearch_urls
+ # doc: the urls to the elasticsearch instances (leave blank to disable)
@lonnen

lonnen Mar 22, 2013

Contributor

the urls for the elasticsearch instances

scripts/config/processorconfig.py.dist
-#elasticSearchOoidSubmissionUrl.default = 'http://node14.generic.metrics.sjc1.mozilla.com:9999/queue/tasks/%s'
-elasticSearchOoidSubmissionUrl.default = ''
+elasticsearch_urls = cm.Option()
+elasticsearch_urls.doc = 'the urls to the elasticsearch instances (leave blank to disable)'
@lonnen

lonnen Mar 22, 2013

Contributor

urls for the elasticsearch

Contributor

lonnen commented Mar 22, 2013

Fix those nits and merge at will. ㊙️

adngdb added a commit that referenced this pull request Mar 25, 2013

Merge pull request #1148 from AdrianGaudebert/853031-processor-config
Fixes bug 853031 - Updated configuration of processors to accept new ela...

@adngdb adngdb merged commit f2a5693 into mozilla-services:master Mar 25, 2013

1 check passed

default Jenkins build 'socorro-github' #613 has succeeded
Details

@adngdb adngdb deleted the adngdb:853031-processor-config branch Mar 25, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment