Skip to content

[7.x] [Security solution] [Endpoint] Fix bad artifact migration (#111294)#111431

Merged
kibanamachine merged 1 commit intoelastic:7.xfrom
kibanamachine:backport/7.x/pr-111294
Sep 7, 2021
Merged

[7.x] [Security solution] [Endpoint] Fix bad artifact migration (#111294)#111431
kibanamachine merged 1 commit intoelastic:7.xfrom
kibanamachine:backport/7.x/pr-111294

Conversation

@kibanamachine
Copy link
Copy Markdown
Contributor

@kibanamachine kibanamachine commented Sep 7, 2021

Backports the following commits to 7.x:

@dasansol92 @paul-tavares @kibanamachine
[Security solution] [Endpoint] Fix bad artifact migration (elastic#11…
f449266 
…1294)

* Working test that validate migrated artifact has same properties as SO artifact
* Checks if artifact is compressed and uncompress it if necessary before creating the new one from fleet

Co-authored-by: Paul Tavares <paul.tavares@elastic.co>
Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
@kibanamachine kibanamachine added the backport This PR is a backport of another PR label Sep 7, 2021
@kibanamachine kibanamachine enabled auto-merge (squash) September 7, 2021 16:39
@kibanamachine kibanamachine mentioned this pull request Sep 7, 2021
Merged
1 task
@kibanamachine
Copy link
Copy Markdown
Contributor Author

kibanamachine commented Sep 7, 2021

💛 Build succeeded, but was flaky

Test Failures

Kibana Pipeline / general / Creates and activates a new ml rule.Detection rules, machine learning Creates and activates a new ml rule

Link to Jenkins

Stack Trace

Failed Tests Reporter:
  - Test has failed 25 times on tracked branches: https://github.com/elastic/kibana/issues/80569

AssertionError: Timed out retrying after 60000ms: Expected to find element: `1`, but never found it. Queried from element: <span.euiBadge.euiBadge--iconLeft>
    at eval (http://localhost:6121/__cypress/tests?p=cypress/integration/detection_rules/machine_learning_rule.spec.ts:21904:81)
at Array.forEach (<anonymous>)
    at Context.eval (http://localhost:6121/__cypress/tests?p=cypress/integration/detection_rules/machine_learning_rule.spec.ts:21903:65)

Metrics [docs]

✅ unchanged

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

cc @dasansol92

@kibanamachine kibanamachine merged commit 4f78254 into elastic:7.x Sep 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@dasansol92 dasansol92

Labels

backport This PR is a backport of another PR

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@kibanamachine @dasansol92