search

eurica / splunk_app_pagerduty

PagerDuty Alert - Trigger PagerDuty Incidents from Splunk Alerts.
http://splunk-base.splunk.com/apps/55908/splunk-app-for-pagerduty
Other
8 stars 7 forks source link

fixed a mismatch in config section name between setup and pagerduty.py #1

Closed gerirgaudi closed 11 years ago

gerirgaudi commented 12 years ago

When running pagerduty.py, the following error was produced:

Traceback (most recent call last):
  File "/opt/splunk/bin/scripts/pagerduty.py", line 159, in <module>
     main()
  File "/opt/splunk/bin/scripts/pagerduty.py", line 141, in main
    service_api_key = get_service_api_key(config_file)
  File "/opt/splunk/bin/scripts/pagerduty.py", line 129, in get_service_api_key
    return config.get('service_api', 'service_api_key')
  File "/usr/lib/python2.6/ConfigParser.py", line 532, in get
    raise NoSectionError(section)
ConfigParser.NoSectionError: No section: 'service_api'
ampledata commented 11 years ago

Thanks for this PR, and sorry about the delay in responding. I'll integrate this and other namespace fixes in version 1.1.0 (and I'm sorry Splunk's App namespace handling is so confusing...)