-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
[Security Solution] Auto-bundle Timeline API OpenAPI specs #188844
[Security Solution] Auto-bundle Timeline API OpenAPI specs #188844
Conversation
fd2a7cb
to
89d637e
Compare
89d637e
to
383a756
Compare
Pinging @elastic/security-threat-hunting (Team:Threat Hunting) |
Pinging @elastic/security-solution (Team: SecuritySolution) |
Pinging @elastic/security-threat-hunting-investigations (Team:Threat Hunting:Investigations) |
Files by Code Ownerelastic/kibana-operations
elastic/security-detection-rule-management
elastic/security-solution
elastic/security-threat-hunting-investigations
|
x-pack/plugins/security_solution/common/api/timeline/model/components.gen.ts
Show resolved
Hide resolved
x-pack/plugins/security_solution/common/api/timeline/model/components.gen.ts
Show resolved
Hide resolved
x-pack/plugins/security_solution/common/api/timeline/model/components.gen.ts
Show resolved
Hide resolved
x-pack/plugins/security_solution/common/api/timeline/model/components.gen.ts
Outdated
Show resolved
Hide resolved
x-pack/plugins/security_solution/common/api/timeline/model/components.gen.ts
Outdated
Show resolved
Hide resolved
x-pack/plugins/security_solution/common/api/timeline/model/components.gen.ts
Show resolved
Hide resolved
x-pack/plugins/security_solution/common/api/timeline/model/components.gen.ts
Outdated
Show resolved
Hide resolved
...ck/plugins/security_solution/common/api/timeline/patch_timelines/patch_timeline_route.gen.ts
Outdated
Show resolved
Hide resolved
Close to the actual schema validation
💚 Build Succeeded
Metrics [docs]
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.
Thanks for making the changes!
Reverted 6b64a9f This is causing numerous failures in the https://buildkite.com/elastic/kibana-on-merge/builds/48189#0190efb6-fae2-415a-85dc-1508717e6e5c/1716-2842 |
…lastic#188844)" This reverts commit 6b64a9f.
…189315) **Addresses**: #184428 **Relates to:** #188844 ## Summary The original [PR](#188844) was rolled back by @Ikuni17 due to CI failures in `main` branch. It turned the problem was caused by a bug in `check_for_changed_files()` which is fixed in #189316. This PR reapplies #188844 and it also includes changes to reduce CI reruns (has to be rerun manually in draft PRs) caused by changes to generated files. It's achieved by reducing a number of `check_for_changed_files()` invocations. --------- Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com> Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
Addresses: #184428
Summary
This PR adds scripts for automatic bundling of Timeline API OpenAPI specs as a part of PR pipeline. Corresponding result bundles are automatically committed to the Security Solution plugin
x-pack/plugins/security_solution
in thedocs/openapi/ess/
anddocs/openapi/serverless
folders (similar to #186384).