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

[DOCS] Create open API specification for create/update connector #148691

Merged
merged 8 commits into from
Jan 17, 2023

Conversation

lcawl
Copy link
Contributor

@lcawl lcawl commented Jan 10, 2023

Summary

Relates to #137240

This PR creates the open API specification for:

create connector
update connector

At this point, the spec does not cover every type of connector. It covers only the connectors that were already in the API documentation (IBM resilient, index, Jira, Opsgenie, ServiceNow, Swimlane, server log, Webhook - Case Management); the rest contain an open-ended "additionalProperties:true" for now. Due to this and the fact that the update connector endpoint doesn't contain a discriminator (e.g. connector type) to differentiate the valid options, the example validation failed until I commented out the incomplete connectors.

@lcawl lcawl added Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) docs Feature:Actions/ConnectorsManagement Issues related to Connectors Management UX backport:prev-minor Backport to (9.0) the previous minor version (i.e. one version back from main) v8.7.0 labels Jan 10, 2023
@github-actions
Copy link
Contributor

Documentation preview:

@lcawl lcawl marked this pull request as ready for review January 12, 2023 03:42
@lcawl lcawl requested review from a team as code owners January 12, 2023 03:42
@elasticmachine
Copy link
Contributor

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

@lcawl lcawl enabled auto-merge (squash) January 17, 2023 01:39
Copy link
Contributor

@szabosteve szabosteve left a comment

Choose a reason for hiding this comment

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

LGTM!

@kibana-ci
Copy link
Collaborator

💚 Build Succeeded

Metrics [docs]

✅ unchanged

History

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

@lcawl lcawl merged commit 4f6d0df into elastic:main Jan 17, 2023
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Jan 17, 2023
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.6

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

lcawl added a commit that referenced this pull request Jan 17, 2023
#148691) (#149063)

# Backport

This will backport the following commits from `main` to `8.6`:
- [[DOCS] Create open API specification for create/update connector
(#148691)](#148691)

<!--- Backport version: 8.9.7 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"Lisa
Cawley","email":"[email protected]"},"sourceCommit":{"committedDate":"2023-01-17T17:31:48Z","message":"[DOCS]
Create open API specification for create/update connector
(#148691)","sha":"4f6d0dff28221d449cc7f1631794bd7a619d8033","branchLabelMapping":{"^v8.7.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:enhancement","Team:ResponseOps","docs","Feature:Actions/ConnectorsManagement","backport:prev-minor","v8.7.0"],"number":148691,"url":"https://github.com/elastic/kibana/pull/148691","mergeCommit":{"message":"[DOCS]
Create open API specification for create/update connector
(#148691)","sha":"4f6d0dff28221d449cc7f1631794bd7a619d8033"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v8.7.0","labelRegex":"^v8.7.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/148691","number":148691,"mergeCommit":{"message":"[DOCS]
Create open API specification for create/update connector
(#148691)","sha":"4f6d0dff28221d449cc7f1631794bd7a619d8033"}}]}]
BACKPORT-->

Co-authored-by: Lisa Cawley <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:prev-minor Backport to (9.0) the previous minor version (i.e. one version back from main) docs Feature:Actions/ConnectorsManagement Issues related to Connectors Management UX release_note:enhancement Team:ResponseOps Label for the ResponseOps team (formerly the Cases and Alerting teams) v8.6.1 v8.7.0
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

5 participants