Skip to content

New resolver#64704

Merged
IgorGuz2000 merged 27 commits intoelastic:masterfrom
IgorGuz2000:new_resolver
May 5, 2020
Merged

New resolver#64704
IgorGuz2000 merged 27 commits intoelastic:masterfrom
IgorGuz2000:new_resolver

Conversation

@IgorGuz2000
Copy link
Copy Markdown
Contributor

@IgorGuz2000 IgorGuz2000 commented Apr 28, 2020

Functional Test For Resolver controls and table data

Summarize your PR. If it involves visual changes include a screenshot or gif.

Checklist

Delete any items that are not applicable to this PR.

@IgorGuz2000 IgorGuz2000 requested a review from a team as a code owner April 28, 2020 21:44
Copy link
Copy Markdown
Contributor

@EricDavisX EricDavisX left a comment

Choose a reason for hiding this comment

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

I posted some questions, comments, and potential improvements - if you want to do them now, or later (I hope) or not, I'm ok to merge this if it works. :)

@IgorGuz2000 IgorGuz2000 requested a review from a team as a code owner April 29, 2020 17:38
@bhavyarm
Copy link
Copy Markdown
Contributor

@elasticmachine merge upstream

@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

History

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

@IgorGuz2000 IgorGuz2000 merged commit 4302161 into elastic:master May 5, 2020
@IgorGuz2000 IgorGuz2000 deleted the new_resolver branch May 6, 2020 16:43
@kibanamachine kibanamachine added the backport missing Added to PRs automatically when the are determined to be missing a backport. label May 7, 2020
@kibanamachine
Copy link
Copy Markdown
Contributor

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create backports run node scripts/backport --pr 64704 or prevent reminders by adding the backport:skip label.

6 similar comments
@kibanamachine
Copy link
Copy Markdown
Contributor

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create backports run node scripts/backport --pr 64704 or prevent reminders by adding the backport:skip label.

@kibanamachine
Copy link
Copy Markdown
Contributor

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create backports run node scripts/backport --pr 64704 or prevent reminders by adding the backport:skip label.

@kibanamachine
Copy link
Copy Markdown
Contributor

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create backports run node scripts/backport --pr 64704 or prevent reminders by adding the backport:skip label.

@kibanamachine
Copy link
Copy Markdown
Contributor

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create backports run node scripts/backport --pr 64704 or prevent reminders by adding the backport:skip label.

@kibanamachine
Copy link
Copy Markdown
Contributor

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create backports run node scripts/backport --pr 64704 or prevent reminders by adding the backport:skip label.

@kibanamachine
Copy link
Copy Markdown
Contributor

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create backports run node scripts/backport --pr 64704 or prevent reminders by adding the backport:skip label.

@IgorGuz2000 IgorGuz2000 added the backport:skip This PR does not require backporting label May 18, 2020
@kibanamachine kibanamachine removed the backport missing Added to PRs automatically when the are determined to be missing a backport. label May 18, 2020
@paul-tavares
Copy link
Copy Markdown
Contributor

FYI - I'm getting backport errors due to this PR not being in 7.x

paul-tavares added a commit to paul-tavares/kibana that referenced this pull request Jun 12, 2020
- PR elastic#64704 was never
  backported, thus this change is needed to ensure build is
  successful.
paul-tavares added a commit that referenced this pull request Jun 12, 2020
* [Endpoint] Functional Tests cleanup (#68756)

* Removed several unnecessary disabled eslint rules
* moved common pageobject from endpoint_list to page_utils
* Rename functional_endpoint to security_solution_endpoint
* Delete `functional_endpoint_ingest_failure` no longer applicable

# Conflicts:
#	.github/CODEOWNERS
#	x-pack/test/security_solution_endpoint/apps/endpoint/resolver.ts

* Sync Endpoint Alerts page objects with master

- PR #64704 was never
  backported, thus this change is needed to ensure build is
  successful.

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
IgorGuz2000 added a commit to IgorGuz2000/kibana that referenced this pull request Sep 21, 2020
* New commit

* New commit adding data-test-subj to controls

* Final commit for the resolver tests

* Final commit for the resolver tests

* Final commit for the resolver tests

* Final commit for the resolver tests

* Final commit for the resolver tests

* Removed sleep statements and added "WaitForTableToHaveData"

* Removed sleep statements and added "WaitForTableToHaveData"

* Fixed type check and modified style parsing

* Removed window resize

* Fixed style type errors

* Removed unnecessary sleeps and added verifications for Nodes location

* Removed unnecessary sleeps and added verifications for Nodes location

* Merge branch 'master' of https://github.com/elastic/kibana into new_resolver

* Merge branch 'master' of https://github.com/elastic/kibana into new_resolver

* Merge branch 'master' of https://github.com/elastic/kibana into new_resolver

* Added resize the window for PAss the test

* Added resize the window for PAss the test

* Added resize the window for PAss the test

* Added resize the window for PAss the test

* Added resize the window for PAss the test

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
# Conflicts:
#	x-pack/plugins/endpoint/public/embeddables/resolver/view/graph_controls.tsx
#	x-pack/plugins/endpoint/public/embeddables/resolver/view/process_event_dot.tsx
#	x-pack/test/functional_endpoint/apps/endpoint/index.ts
#	x-pack/test/functional_endpoint/page_objects/endpoint_alerts_page.ts
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:skip This PR does not require backporting release_note:skip Skip the PR/issue when compiling release notes v8.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants