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

[Detection Engine][Docs] Added response and request descriptions for API docs #205822

Merged
merged 20 commits into from
Jan 16, 2025

Conversation

yctercero
Copy link
Contributor

@yctercero yctercero commented Jan 7, 2025

Summary

As part of the effort to add missing content for Security APIs, this PR introduces a few missing request, response, and parameter examples for Detection Engine Exception APIs.

How to Test

Run a local preview in bump, which shows what the output will look like in our production.

  • Install the bump CLI
  • cd x-pack/solutions/security/packages/kbn-securitysolution-exceptions-common and then run npm run openapi:generate followed by npm run openapi:bundle. That last command gives you the path to the bundled file at the end that is used in the next step.
  • Run bump preview x-pack/solutions/security/packages/kbn-securitysolution-exceptions-common/docs/openapi/ess/security_solution_exceptions_api_2023_10_31.bundled.schema.yaml

@yctercero yctercero requested review from a team as code owners January 7, 2025 22:11
@yctercero yctercero requested review from vitaliidm and a team January 7, 2025 22:11
@yctercero yctercero self-assigned this Jan 7, 2025
@yctercero yctercero added release_note:skip Skip the PR/issue when compiling release notes v9.0.0 Team:Detection Engine Security Solution Detection Engine Area v8.17.0 v8.18.0 labels Jan 7, 2025
@elasticmachine
Copy link
Contributor

Pinging @elastic/security-detection-engine (Team:Detection Engine)

@yctercero yctercero added the backport:version Backport to applied version labels label Jan 7, 2025
Copy link
Contributor

@vitaliidm vitaliidm left a comment

Choose a reason for hiding this comment

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

Thanks for working on this PR, @yctercero

I left a few comments.

Also, I noticed there are changes that enable/disable tests. Are these changes supposed to be in this PR?

@yctercero yctercero requested a review from a team as a code owner January 9, 2025 17:40
@yctercero yctercero removed the request for review from a team January 9, 2025 17:43
@kibanamachine
Copy link
Contributor

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 205822 locally

1 similar comment
@kibanamachine
Copy link
Contributor

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 205822 locally

viduni94 pushed a commit to viduni94/kibana that referenced this pull request Jan 23, 2025
…API docs (elastic#205822)

# Summary

As part of the effort to add missing content for Security APIs, this PR
introduces a few missing request, response, and parameter examples for
Detection Engine Exception APIs.
@kibanamachine
Copy link
Contributor

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 205822 locally

@kibanamachine
Copy link
Contributor

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 205822 locally

5 similar comments
@kibanamachine
Copy link
Contributor

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 205822 locally

@kibanamachine
Copy link
Contributor

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 205822 locally

@kibanamachine
Copy link
Contributor

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 205822 locally

@kibanamachine
Copy link
Contributor

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 205822 locally

@kibanamachine
Copy link
Contributor

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 205822 locally

@kibanamachine
Copy link
Contributor

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create automatically backports add a backport:* label or prevent reminders by adding the backport:skip label.
You can also create backports manually by running node scripts/backport --pr 205822 locally

@yctercero
Copy link
Contributor Author

💚 All backports created successfully

Status Branch Result
8.18
8.17
8.16

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

Questions ?

Please refer to the Backport tool documentation

yctercero added a commit that referenced this pull request Feb 5, 2025
…ns for API docs (#205822) (#209692)

# Backport

This will backport the following commits from `main` to `8.18`:
- [[Detection Engine][Docs] Added response and request descriptions for
API docs (#205822)](#205822)

<!--- Backport version: 9.6.4 -->

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

<!--BACKPORT [{"author":{"name":"Yara
Tercero","email":"yctercero@users.noreply.github.com"},"sourceCommit":{"committedDate":"2025-01-16T20:14:08Z","message":"[Detection
Engine][Docs] Added response and request descriptions for API docs
(#205822)\n\n# Summary\r\n\r\nAs part of the effort to add missing
content for Security APIs, this PR\r\nintroduces a few missing request,
response, and parameter examples for\r\nDetection Engine Exception
APIs.","sha":"94660cf2f51047e09185f4965f94a34ec80915b2","branchLabelMapping":{"^v9.0.0$":"main","^v8.18.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","backport
missing","v9.0.0","Team:Detection
Engine","backport:version","v8.17.0","v8.18.0"],"title":"[Detection
Engine][Docs] Added response and request descriptions for API
docs","number":205822,"url":"https://github.com/elastic/kibana/pull/205822","mergeCommit":{"message":"[Detection
Engine][Docs] Added response and request descriptions for API docs
(#205822)\n\n# Summary\r\n\r\nAs part of the effort to add missing
content for Security APIs, this PR\r\nintroduces a few missing request,
response, and parameter examples for\r\nDetection Engine Exception
APIs.","sha":"94660cf2f51047e09185f4965f94a34ec80915b2"}},"sourceBranch":"main","suggestedTargetBranches":["8.17","8.18"],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/205822","number":205822,"mergeCommit":{"message":"[Detection
Engine][Docs] Added response and request descriptions for API docs
(#205822)\n\n# Summary\r\n\r\nAs part of the effort to add missing
content for Security APIs, this PR\r\nintroduces a few missing request,
response, and parameter examples for\r\nDetection Engine Exception
APIs.","sha":"94660cf2f51047e09185f4965f94a34ec80915b2"}},{"branch":"8.17","label":"v8.17.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.x","label":"v8.18.0","branchLabelMappingKey":"^v8.18.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->
@kibanamachine
Copy link
Contributor

Looks like this PR has backport PRs but they still haven't been merged. Please merge them ASAP to keep the branches relatively in sync.

@yctercero
Copy link
Contributor Author

💚 All backports created successfully

Status Branch Result
8.16

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

Questions ?

Please refer to the Backport tool documentation

yctercero added a commit that referenced this pull request Feb 6, 2025
…ns for API docs (#205822) (#209693)

# Backport

This will backport the following commits from `main` to `8.17`:
- [[Detection Engine][Docs] Added response and request descriptions for
API docs (#205822)](#205822)

<!--- Backport version: 9.6.4 -->

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

<!--BACKPORT [{"author":{"name":"Yara
Tercero","email":"yctercero@users.noreply.github.com"},"sourceCommit":{"committedDate":"2025-01-16T20:14:08Z","message":"[Detection
Engine][Docs] Added response and request descriptions for API docs
(#205822)\n\n# Summary\r\n\r\nAs part of the effort to add missing
content for Security APIs, this PR\r\nintroduces a few missing request,
response, and parameter examples for\r\nDetection Engine Exception
APIs.","sha":"94660cf2f51047e09185f4965f94a34ec80915b2","branchLabelMapping":{"^v9.0.0$":"main","^v8.18.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","backport
missing","v9.0.0","Team:Detection
Engine","backport:version","v8.17.0","v8.18.0"],"title":"[Detection
Engine][Docs] Added response and request descriptions for API
docs","number":205822,"url":"https://github.com/elastic/kibana/pull/205822","mergeCommit":{"message":"[Detection
Engine][Docs] Added response and request descriptions for API docs
(#205822)\n\n# Summary\r\n\r\nAs part of the effort to add missing
content for Security APIs, this PR\r\nintroduces a few missing request,
response, and parameter examples for\r\nDetection Engine Exception
APIs.","sha":"94660cf2f51047e09185f4965f94a34ec80915b2"}},"sourceBranch":"main","suggestedTargetBranches":["8.17","8.18"],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/205822","number":205822,"mergeCommit":{"message":"[Detection
Engine][Docs] Added response and request descriptions for API docs
(#205822)\n\n# Summary\r\n\r\nAs part of the effort to add missing
content for Security APIs, this PR\r\nintroduces a few missing request,
response, and parameter examples for\r\nDetection Engine Exception
APIs.","sha":"94660cf2f51047e09185f4965f94a34ec80915b2"}},{"branch":"8.17","label":"v8.17.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.x","label":"v8.18.0","branchLabelMappingKey":"^v8.18.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->
@yctercero
Copy link
Contributor Author

💚 All backports created successfully

Status Branch Result
8.16

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

Questions ?

Please refer to the Backport tool documentation

yctercero added a commit that referenced this pull request Feb 6, 2025
…ns for API docs (#205822) (#209694)

# Backport

This will backport the following commits from `main` to `8.16`:
- [[Detection Engine][Docs] Added response and request descriptions for
API docs (#205822)](#205822)

<!--- Backport version: 9.6.4 -->

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

<!--BACKPORT [{"author":{"name":"Yara
Tercero","email":"yctercero@users.noreply.github.com"},"sourceCommit":{"committedDate":"2025-01-16T20:14:08Z","message":"[Detection
Engine][Docs] Added response and request descriptions for API docs
(#205822)\n\n# Summary\r\n\r\nAs part of the effort to add missing
content for Security APIs, this PR\r\nintroduces a few missing request,
response, and parameter examples for\r\nDetection Engine Exception
APIs.","sha":"94660cf2f51047e09185f4965f94a34ec80915b2","branchLabelMapping":{"^v9.0.0$":"main","^v8.18.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","backport
missing","v9.0.0","Team:Detection
Engine","backport:version","v8.17.0","v8.18.0"],"title":"[Detection
Engine][Docs] Added response and request descriptions for API
docs","number":205822,"url":"https://github.com/elastic/kibana/pull/205822","mergeCommit":{"message":"[Detection
Engine][Docs] Added response and request descriptions for API docs
(#205822)\n\n# Summary\r\n\r\nAs part of the effort to add missing
content for Security APIs, this PR\r\nintroduces a few missing request,
response, and parameter examples for\r\nDetection Engine Exception
APIs.","sha":"94660cf2f51047e09185f4965f94a34ec80915b2"}},"sourceBranch":"main","suggestedTargetBranches":["8.17","8.18"],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/205822","number":205822,"mergeCommit":{"message":"[Detection
Engine][Docs] Added response and request descriptions for API docs
(#205822)\n\n# Summary\r\n\r\nAs part of the effort to add missing
content for Security APIs, this PR\r\nintroduces a few missing request,
response, and parameter examples for\r\nDetection Engine Exception
APIs.","sha":"94660cf2f51047e09185f4965f94a34ec80915b2"}},{"branch":"8.17","label":"v8.17.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.x","label":"v8.18.0","branchLabelMappingKey":"^v8.18.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->
@kibanamachine kibanamachine added v8.16.4 and removed backport missing Added to PRs automatically when the are determined to be missing a backport. labels Feb 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:version Backport to applied version labels release_note:skip Skip the PR/issue when compiling release notes Team:Detection Engine Security Solution Detection Engine Area v8.16.0 v8.16.4 v8.17.0 v8.17.2 v8.18.0 v9.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants