-
Notifications
You must be signed in to change notification settings - Fork 8.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ServiceNow action improvements #60052
Conversation
Pinging @elastic/siem (Team:SIEM) |
Pinging @elastic/kibana-alerting-services (Team:Alerting Services) |
af07139
to
b8f5a72
Compare
x-pack/plugins/actions/server/builtin_action_types/servicenow/action_handlers.test.ts
Show resolved
Hide resolved
x-pack/plugins/actions/server/builtin_action_types/servicenow/helpers.ts
Show resolved
Hide resolved
x-pack/plugins/actions/server/builtin_action_types/servicenow/index.test.ts
Outdated
Show resolved
Hide resolved
x-pack/plugins/actions/server/builtin_action_types/servicenow/index.test.ts
Outdated
Show resolved
Hide resolved
x-pack/plugins/actions/server/builtin_action_types/servicenow/schema.ts
Outdated
Show resolved
Hide resolved
x-pack/plugins/actions/server/builtin_action_types/servicenow/transformers.ts
Outdated
Show resolved
Hide resolved
@elasticmachine merge upstream |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
From an alerting/actions point of view, this code LGTM - made a comment about a possible restructuring of config-schema definitions, not a big deal.
x-pack/plugins/actions/server/builtin_action_types/servicenow/schema.ts
Outdated
Show resolved
Hide resolved
b9e7742
to
2a831fc
Compare
@elasticmachine merge upstream |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@elasticmachine merge upstream |
x-pack/plugins/actions/server/builtin_action_types/servicenow/helpers.ts
Outdated
Show resolved
Hide resolved
💛 Build succeeded, but was flaky
Test FailuresKibana Pipeline / kibana-oss-agent / Accessibility Tests.test/accessibility/apps/management·ts.Management Create Index pattern wizardStandard Out
Stack Trace
History
To update your PR or re-run it, just comment with: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 🚀
* Apply action types to fields * Add information to each field * Do not create or update comments when actionType is set to nothing * Improve helpers tests * Improve tests * Refactor: Use transformers and pipes * Better types * Refactor tests to new changes * Better error messages * Improve field formatting and display * Improve integration tests * Make username mandatory field * Translate transformers * Refactor schema * Translate appendInformationToField helper * Improve intergration tests Co-authored-by: Elastic Machine <[email protected]>
* Apply action types to fields * Add information to each field * Do not create or update comments when actionType is set to nothing * Improve helpers tests * Improve tests * Refactor: Use transformers and pipes * Better types * Refactor tests to new changes * Better error messages * Improve field formatting and display * Improve integration tests * Make username mandatory field * Translate transformers * Refactor schema * Translate appendInformationToField helper * Improve intergration tests Co-authored-by: Elastic Machine <[email protected]> Co-authored-by: Elastic Machine <[email protected]>
* master: (35 commits) [Alerting] Adds navigation by consumer and alert type to alerting (elastic#58997) Introduce search interceptor (elastic#60523) [ML] Add functional tests for file data visualizer (elastic#60413) [APM] Optimize service map query (elastic#60412) [SIEM][Detection Engine] Adds lists feature flag and list values to the REST interfaces Enhancement/update esdocs datasource (elastic#59512) [junit] only include stdout in report for failures (elastic#60530) Update dependency nock to v12 (elastic#60422) upgrade execa to get stdout/stderr in error messages (elastic#60537) skip flaky suite (elastic#60471) [Ingest] Agent Config Details - Data sources list ui (elastic#60429) [SIEM] Create ML Rules (elastic#58053) skip flaky suite (elastic#60559) fix agent type (elastic#60554) Fixed default message for index threshold includes both threshold values (elastic#60545) [Ingest] Add support for `yaml` field types (elastic#60440) Solved the issue for a GROUP BY expression validation (elastic#60558) [Maps] Mark instance state as readonly (elastic#60557) Move ui/indices into es_ui_shared plugin. (elastic#60186) ServiceNow action improvements (elastic#60052) ...
* master: (64 commits) [Alerting] Adds navigation by consumer and alert type to alerting (elastic#58997) Introduce search interceptor (elastic#60523) [ML] Add functional tests for file data visualizer (elastic#60413) [APM] Optimize service map query (elastic#60412) [SIEM][Detection Engine] Adds lists feature flag and list values to the REST interfaces Enhancement/update esdocs datasource (elastic#59512) [junit] only include stdout in report for failures (elastic#60530) Update dependency nock to v12 (elastic#60422) upgrade execa to get stdout/stderr in error messages (elastic#60537) skip flaky suite (elastic#60471) [Ingest] Agent Config Details - Data sources list ui (elastic#60429) [SIEM] Create ML Rules (elastic#58053) skip flaky suite (elastic#60559) fix agent type (elastic#60554) Fixed default message for index threshold includes both threshold values (elastic#60545) [Ingest] Add support for `yaml` field types (elastic#60440) Solved the issue for a GROUP BY expression validation (elastic#60558) [Maps] Mark instance state as readonly (elastic#60557) Move ui/indices into es_ui_shared plugin. (elastic#60186) ServiceNow action improvements (elastic#60052) ...
Summary
This PR improves ServiceNow action. Specifically implements the functionality of the following action types:
nothing
,overwrite
, andappend
.\r\n
API (new changes)
Create an incident:
Crate an incident to ServiceNow. When the
incidentId
attribute is not inparams
the executor will create the incident.Endpoint:
api/action/<action_id>/_execute
Method:
POST
Payload:
Response
Update an incident:
Update an incident to ServiceNow. When the
incidentId
attribute is inparams
the executor will update the incident.Endpoint:
api/action/<action_id>/_execute
Method:
POST
Payload:
Response
Checklist
Delete any items that are not applicable to this PR.
For maintainers