Skip to content

Added support for better webhooks and victorops for saved searches notifications#217

Merged
rrossetti-splunk merged 5 commits intosplunk:masterfrom
pjayala:better_webhooks
Jan 7, 2026
Merged

Added support for better webhooks and victorops for saved searches notifications#217
rrossetti-splunk merged 5 commits intosplunk:masterfrom
pjayala:better_webhooks

Conversation

@pjayala
Copy link
Contributor

@pjayala pjayala commented Nov 25, 2025

No description provided.

@pjayala pjayala changed the title Added support for better webhooks for saved searches notifications Added support for better webhooks and victorops for saved searches notifications Nov 28, 2025
@rrossetti-splunk rrossetti-splunk self-assigned this Dec 12, 2025
@rrossetti-splunk
Copy link
Collaborator

@pjayala Can you fix the lint error?

Error: splunk/resource_splunk_saved_searches_test.go:517:13: undefined: newSavedSearchesVictorops (typecheck)

@pjayala
Copy link
Contributor Author

pjayala commented Dec 15, 2025

@pjayala Can you fix the lint error?

Error: splunk/resource_splunk_saved_searches_test.go:517:13: undefined: newSavedSearchesVictorops (typecheck)

Done!

@rrossetti-splunk
Copy link
Collaborator

@pjayala Thanks for fixing Lint error, but a test is failing:

RUN TestAccSplunkSavedSearches
testing.go:684: Step 7 error: Check failed: Check 11/22 error: splunk_saved_searches.test: Attribute 'action_victorops_param_inactive_polls' expected "10", got ""
--- FAIL: TestAccSplunkSavedSearches (1.36s)

@pjayala
Copy link
Contributor Author

pjayala commented Dec 18, 2025

@pjayala Thanks for fixing Lint error, but a test is failing:

RUN TestAccSplunkSavedSearches
testing.go:684: Step 7 error: Check failed: Check 11/22 error: splunk_saved_searches.test: Attribute 'action_victorops_param_inactive_polls' expected "10", got ""
--- FAIL: TestAccSplunkSavedSearches (1.36s)

Thanks, I have fixed tests for victorops

@rrossetti-splunk
Copy link
Collaborator

@pjayala Thanks for fixing Lint error, but a test is failing:

RUN TestAccSplunkSavedSearches
testing.go:684: Step 7 error: Check failed: Check 11/22 error: splunk_saved_searches.test: Attribute 'action_victorops_param_inactive_polls' expected "10", got ""
--- FAIL: TestAccSplunkSavedSearches (1.36s)

Thanks, I have fixed tests for victorops

Looks like there is a typo in the test "TestAccSplunkSavedSearches"

RUN TestAccSplunkSavedSearches
testing.go:684: Step 8 error: config is invalid: 2 problems:

    - Unsupported argument: An argument named "action_better_webhook_param_credetials" is not expected here. Did you mean "action_better_webhook_param_credential"?

@pjayala
Copy link
Contributor Author

pjayala commented Jan 7, 2026

@rrossetti-splunk Thanks, I fixed the typo

@rrossetti-splunk rrossetti-splunk merged commit 2011744 into splunk:master Jan 7, 2026
2 checks passed
@rrossetti-splunk
Copy link
Collaborator

rrossetti-splunk commented Jan 7, 2026

@rrossetti-splunk Thanks, I fixed the typo

New release including the change is out - 1.4.33
Thanks for your help and patience with this!
https://registry.terraform.io/providers/splunk/splunk/latest

@pjayala pjayala deleted the better_webhooks branch January 8, 2026 10:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants