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

[Security Solution] add handling for endpoint package spec v2 #169901

Merged
merged 1 commit into from
Oct 26, 2023

Conversation

joeypoon
Copy link
Member

@joeypoon joeypoon commented Oct 25, 2023

Summary

Endpoint package is being upgraded to package spec v2. This upgrade causes the package managed transform names to change (now prefixed with an additional logs-). This PR updates all the places we're using the transform names to conditionally either use the v1 transform names or the v2 transform names depending on the package version to maintain backwards compatibility.

Also unskips x-pack/test/security_solution_endpoint_api_int/apis/metadata.ts tests. Flaky test runner: https://buildkite.com/elastic/kibana-flaky-test-suite-runner/builds/3763 (x100 ✅ ).

Checklist

For maintainers

@joeypoon joeypoon added release_note:skip Skip the PR/issue when compiling release notes Team:Defend Workflows “EDR Workflows” sub-team of Security Solution v8.12.0 labels Oct 25, 2023
@joeypoon joeypoon changed the title [Security Solution] and handling for endpoint package spec v2 [Security Solution] add handling for endpoint package spec v2 Oct 25, 2023
endpoint package managed transform names are changing with the endpoint package migration to
package spec v2. this commit updates places where we rely on these transform names. backwards compatible.
@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

@joeypoon joeypoon marked this pull request as ready for review October 26, 2023 16:22
@joeypoon joeypoon requested a review from a team as a code owner October 26, 2023 16:22
@elasticmachine
Copy link
Contributor

Pinging @elastic/security-defend-workflows (Team:Defend Workflows)

@joeypoon joeypoon requested a review from pzl October 26, 2023 16:22
@joeypoon joeypoon merged commit 82880fe into elastic:main Oct 26, 2023
29 checks passed
@kibanamachine kibanamachine added the backport:skip This commit does not require backporting label Oct 26, 2023
@joeypoon joeypoon deleted the feature/endpoint-package-v2 branch October 26, 2023 20:08
jbudz added a commit that referenced this pull request Oct 27, 2023
@jbudz
Copy link
Member

jbudz commented Oct 27, 2023

This was reverted with 638d070 due to an import of kbn/tooling-log, a development dependency, in production code.

Logs can be found at https://buildkite.com/elastic/kibana-artifacts-snapshot/builds/3352#018b6e02-b9a6-4154-b470-0fd1248f5942/852-857

@jbudz jbudz added the reverted label Oct 27, 2023
joeypoon added a commit that referenced this pull request Oct 27, 2023
fkanout pushed a commit to fkanout/kibana that referenced this pull request Oct 30, 2023
fkanout pushed a commit to fkanout/kibana that referenced this pull request Oct 30, 2023
bryce-b pushed a commit to bryce-b/kibana that referenced this pull request Oct 30, 2023
bryce-b pushed a commit to bryce-b/kibana that referenced this pull request Oct 30, 2023
awahab07 pushed a commit to awahab07/kibana that referenced this pull request Oct 31, 2023
awahab07 pushed a commit to awahab07/kibana that referenced this pull request Oct 31, 2023
awahab07 pushed a commit to awahab07/kibana that referenced this pull request Oct 31, 2023
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 release_note:skip Skip the PR/issue when compiling release notes reverted Team:Defend Workflows “EDR Workflows” sub-team of Security Solution v8.12.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants