-
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
[Alerting][Connectors] Refactor IBM Resilient: Generic Implementation (phase one) #74357
[Alerting][Connectors] Refactor IBM Resilient: Generic Implementation (phase one) #74357
Conversation
3b6bb94
to
a99a0da
Compare
a99a0da
to
0a1b5bb
Compare
0a1b5bb
to
dd5afb2
Compare
da1d165
to
5d65f72
Compare
Pinging @elastic/kibana-alerting-services (Team:Alerting Services) |
Pinging @elastic/siem (Team:SIEM) |
@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.
Great work! LGTM in overall, but requested the same changes as for a Jira PR, because we want to keep the code cleaner and planning to work on the actions functionality improvements and connectors and actions responsibility splitting. cc @mikecote
x-pack/plugins/actions/server/builtin_action_types/resilient/case_schema.ts
Outdated
Show resolved
Hide resolved
x-pack/plugins/actions/server/builtin_action_types/resilient/translations.ts
Outdated
Show resolved
Hide resolved
x-pack/plugins/actions/server/builtin_action_types/resilient/validators.ts
Outdated
Show resolved
Hide resolved
...rs_actions_ui/public/application/components/builtin_action_types/email/email_params.test.tsx
Outdated
Show resolved
Hide resolved
...tions_ui/public/application/components/builtin_action_types/resilient/case_mappings/utils.ts
Outdated
Show resolved
Hide resolved
f52c027
to
2fb4fe0
Compare
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! Just a few nits about white spaces
...rs_actions_ui/public/application/components/builtin_action_types/email/email_params.test.tsx
Show resolved
Hide resolved
...ions_ui/public/application/components/builtin_action_types/es_index/es_index_params.test.tsx
Show resolved
Hide resolved
...ns_ui/public/application/components/builtin_action_types/pagerduty/pagerduty_params.test.tsx
Show resolved
Hide resolved
..._ui/public/application/components/builtin_action_types/servicenow/servicenow_params.test.tsx
Show resolved
Hide resolved
...rs_actions_ui/public/application/components/builtin_action_types/slack/slack_params.test.tsx
Show resolved
Hide resolved
...ctions_ui/public/application/components/builtin_action_types/webhook/webhook_params.test.tsx
Show resolved
Hide resolved
@elasticmachine merge upstream |
💚 Build SucceededBuild metrics@kbn/optimizer bundle module count
async chunks size
page load bundle size
distributable file count
History
To update your PR or re-run it, just comment with: |
… (phase one) (elastic#74357) Co-authored-by: Elastic Machine <[email protected]>
…tation (phase one) (#74357) (#77264) Co-authored-by: Elastic Machine <[email protected]> Co-authored-by: Elastic Machine <[email protected]>
…s-for-710 * 'master' of github.com:elastic/kibana: (65 commits) Separate url forwarding logic and legacy services (elastic#76892) Bump yargs-parser to v13.1.2+ (elastic#77009) [Ingest Manager] Shared Fleet agent policy action (elastic#76013) [Search] Re-add support for aborting when a connection is closed (elastic#76470) [Search] Remove long-running query pop-up (elastic#75385) [Monitoring] Fix UI error when alerting is not available (elastic#77179) do not log plugin id format warning in dist mode (elastic#77134) [ML] Improving client side error handling (elastic#76743) [Alerting][Connectors] Refactor IBM Resilient: Generic Implementation (phase one) (elastic#74357) [Docs] some basic searchsource api docs (elastic#77038) add cGroupOverrides to the legacy config (elastic#77180) Change saved object bulkUpdate to work across multiple namespaces (elastic#75478) [Security Solution][Resolver] Replace Selectable popover with badges (elastic#76997) Removing ml-state index from archive (elastic#77143) [Security Solution] Add unit tests for histograms (elastic#77081) [Lens] Filters aggregation (elastic#75635) [Enterprise Search] Update WS Overview logic to use new config data (elastic#77122) Cleanup type output before building new types (elastic#77211) [Security Solution] Use safe type in resolver backend (elastic#76969) Use proper lodash syntax (elastic#77105) ... # Conflicts: # x-pack/plugins/index_lifecycle_management/public/application/sections/edit_policy/components/node_allocation.tsx
API changes for creating an
|
@cnasikas Do you mean "API changes for creating a IBM Resilient connector"? |
@gchaps Yes. Sorry, my mistake. |
@cnasikas as this PR is an API change, I think this should have the label |
@gchaps Ok! What is the difference? |
Pinging @elastic/security-solution (Team: SecuritySolution) |
Summary
Summary
This PR refactors the Jira connector to be generic. Specifically:
incidentTypes
andseverity
When #73778 is merged
x-pack/plugins/actions/server/builtin_action_types/case
folder will be removed in this PR.UI:
API
Push incident:
Pushes an incident to Jira
Endpoint:
api/actions/action/<action_id>/_execute
Method:
POST
Payload:
Response
Get issue types:
Get all incident types in IBM Resilient.
Endpoint:
api/actions/action/<action_id>/_execute
Method:
POST
Payload:
Response
Get severity:
Get all severity codes in IBM Resilient.
Endpoint:
api/actions/action/<action_id>/_execute
Method:
POST
Payload:
Response
Checklist
Delete any items that are not applicable to this PR.
For maintainers