Skip to content
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

[Connectors] Split ServiceNow connector type #141961

Merged
merged 10 commits into from
Oct 5, 2022

Conversation

ymao1
Copy link
Contributor

@ymao1 ymao1 commented Sep 27, 2022

Towards #90931

Summary

The ServiceNow connector type is actually 3 different connector types: ServiceNow ITOM, ServiceNow ITSM and ServiceNow SecOps. Cases should own ITSM and SecOps and E&C should own ITOM, so this PR tries to decouple the UI and server code for these connectors and move them to the appropriate folder.

Also moved Xmatters to stack folder because I incorrectly put it in cases in the initial PR.

To Verify

You should be able to create all three types of ServiceNow connectors and successfully test them in the test tab.

@ymao1 ymao1 force-pushed the connector-types-split-servicenow branch from ca41911 to b556b11 Compare September 27, 2022 16:55
@ymao1 ymao1 changed the title Connector types split servicenow [Connectors] Split ServiceNow connector type Sep 27, 2022
@ymao1 ymao1 self-assigned this Sep 27, 2022
@ymao1 ymao1 added release_note:skip Skip the PR/issue when compiling release notes Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) Feature:Actions/ConnectorTypes Issues related to specific Connector Types on the Actions Framework v8.6.0 labels Sep 27, 2022
@ymao1 ymao1 marked this pull request as ready for review September 27, 2022 20:32
@ymao1 ymao1 requested a review from a team as a code owner September 27, 2022 20:32
@ymao1 ymao1 requested a review from a team September 27, 2022 20:32
@ymao1 ymao1 requested a review from a team as a code owner September 27, 2022 20:33
@elasticmachine
Copy link
Contributor

Pinging @elastic/response-ops (Team:ResponseOps)

@ymao1
Copy link
Contributor Author

ymao1 commented Sep 28, 2022

@elasticmachine merge upstream

@ymao1
Copy link
Contributor Author

ymao1 commented Sep 28, 2022

@elasticmachine merge upstream

@ymao1
Copy link
Contributor Author

ymao1 commented Oct 3, 2022

@elasticmachine merge upstream

Copy link
Member

@pmuellr pmuellr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM; tested ITOM and was able to create; asking around now for info to test the ITSM and SecOps flavors, but I assume they work and will re-review if I find they don't.

Copy link
Member

@cnasikas cnasikas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, @ymao1! I tested ITSM & SecOps without any issues. I did not review the code as @pmuellr did already.

@ymao1
Copy link
Contributor Author

ymao1 commented Oct 5, 2022

@elasticmachine merge upstream

@kibana-ci
Copy link
Collaborator

💚 Build Succeeded

Metrics [docs]

Module Count

Fewer modules leads to a faster build time

id before after diff
stackConnectors 127 133 +6

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
stackConnectors 324.6KB 327.5KB +3.0KB

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
stackConnectors 20.8KB 20.8KB +2.0B
Unknown metric groups

async chunk count

id before after diff
stackConnectors 48 50 +2

ESLint disabled line counts

id before after diff
stackConnectors 65 67 +2

Total ESLint disabled count

id before after diff
stackConnectors 69 71 +2

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

cc @ymao1

@ymao1 ymao1 merged commit bb8a3c3 into elastic:main Oct 5, 2022
@ymao1 ymao1 deleted the connector-types-split-servicenow branch October 5, 2022 13:56
@kibanamachine kibanamachine added the backport:skip This commit does not require backporting label Oct 5, 2022
WafaaNasr pushed a commit to WafaaNasr/kibana that referenced this pull request Oct 11, 2022
* Splitting public servicenow files

* Splitting server servicenow files

* Fixing checks

* Fixing checks

* Moving xmatters to stack

Co-authored-by: Kibana Machine <[email protected]>
WafaaNasr pushed a commit to WafaaNasr/kibana that referenced this pull request Oct 14, 2022
* Splitting public servicenow files

* Splitting server servicenow files

* Fixing checks

* Fixing checks

* Moving xmatters to stack

Co-authored-by: Kibana Machine <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:skip This commit does not require backporting Feature:Actions/ConnectorTypes Issues related to specific Connector Types on the Actions Framework release_note:skip Skip the PR/issue when compiling release notes Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) v8.6.0
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

6 participants