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

CCS Smoke Test for Remote Clusters and Index Management #142423

Conversation

cuff-links
Copy link
Contributor

This PR adds a functional X-Pack test that uses ftr-remote to add a remote and verify that the data added to the remote is available in index management.

@cuff-links cuff-links added Team:QA Team label for QA Team test_ui_functional test_xpack_functional release_note:skip Skip the PR/issue when compiling release notes Feature:Functional Testing auto-backport Deprecated - use backport:version if exact versions are needed v8.6.0 v8.5.1 labels Oct 1, 2022
Copy link
Contributor

@alisonelizabeth alisonelizabeth 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 addressing my feedback! I left a few more comments in the code, but nothing blocking.

@cuff-links
Copy link
Contributor Author

@elasticmachine merge upstream

@kibana-ci
Copy link
Collaborator

💚 Build Succeeded

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
remoteClusters 84.1KB 84.9KB +819.0B
Unknown metric groups

ESLint disabled in files

id before after diff
osquery 1 2 +1

ESLint disabled line counts

id before after diff
enterpriseSearch 19 21 +2
fleet 57 63 +6
osquery 103 108 +5
securitySolution 439 443 +4
total +17

Total ESLint disabled count

id before after diff
enterpriseSearch 20 22 +2
fleet 65 71 +6
osquery 104 110 +6
securitySolution 516 520 +4
total +18

History

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

@cuff-links cuff-links merged commit f5acf76 into elastic:main Nov 1, 2022
@kibanamachine
Copy link
Contributor

💔 All backports failed

Status Branch Result
8.5 Backport failed because of merge conflicts

Manual backport

To create the backport manually run:

node scripts/backport --pr 142423

Questions ?

Please refer to the Backport tool documentation and see the Github Action logs for details

jloleysens added a commit to jloleysens/kibana that referenced this pull request Nov 1, 2022
* main: (43 commits)
  [Synthetics] Step details page screenshot (elastic#143452)
  [Lens] Datatable expression types improvement. (elastic#144173)
  [packages/kbn-journeys] start apm after browser start and stop after browser is closed (elastic#144267)
  [Files] Make files namespace agnostic (elastic#144019)
  Implement base browser-side logging system (elastic#144107)
  Correct wrong multiplier for byte conversion (elastic#143751)
  [Monaco] Add JSON syntax support to the Monaco editor (elastic#143739)
  CCS Smoke Test for Remote Clusters and Index Management  (elastic#142423)
  [api-docs] Daily api_docs build (elastic#144294)
  chore(NA): include progress on Bazel tasks (elastic#144275)
  [RAM] Allow users to see event logs from all spaces they have access to (elastic#140449)
  [APM] Show recommended minimum size when going below 5 minutes (elastic#144170)
  [typecheck] delete temporary target_types dirs in packages (elastic#144271)
  [Security Solution][Endpoint] adds new alert loading utility and un-skip FTR test for endpoint (elastic#144133)
  [performance/journeys] revert data_stress_test_lens.ts journey step (elastic#144261)
  [TIP] Use search strategies in Threat Intelligence (elastic#143267)
  Optimize react-query dependencies (elastic#144206)
  [babel/node] invalidate cache when synth pkg map is updated (elastic#144258)
  [APM] AWS lambda estimated cost (elastic#143986)
  [Maps] layer group wizard (elastic#144129)
  ...
@kibanamachine
Copy link
Contributor

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

@kibanamachine kibanamachine added the backport missing Added to PRs automatically when the are determined to be missing a backport. label Nov 3, 2022
@kibanamachine
Copy link
Contributor

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

11 similar comments
@kibanamachine
Copy link
Contributor

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

@kibanamachine
Copy link
Contributor

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

@kibanamachine
Copy link
Contributor

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

@kibanamachine
Copy link
Contributor

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

@kibanamachine
Copy link
Contributor

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

@kibanamachine
Copy link
Contributor

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

@kibanamachine
Copy link
Contributor

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

@kibanamachine
Copy link
Contributor

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

@kibanamachine
Copy link
Contributor

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

@kibanamachine
Copy link
Contributor

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

@kibanamachine
Copy link
Contributor

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

cuff-links pushed a commit to cuff-links/kibana that referenced this pull request Nov 21, 2022
* Removed comment of the issue that was referenced for the skip. But the tests were already skipped.

* Added initial tests and page objects for remtoe clusters.

* [CI] Auto-commit changed files from 'node scripts/eslint --no-cache --fix'

* Fixed the test and test names.

* removed exclusive suite.

* [CI] Auto-commit changed files from 'node scripts/eslint --no-cache --fix'

* Fixed i18n issue.

* [CI] Auto-commit changed files from 'node scripts/eslint --no-cache --fix'

* Added more testing stuff.

* [CI] Auto-commit changed files from 'node scripts/precommit_hook.js --ref HEAD~1..HEAD --fix'

* Added more testing stuff.

* [CI] Auto-commit changed files from 'node scripts/precommit_hook.js --ref HEAD~1..HEAD --fix'

* Added test and stuff.

* [CI] Auto-commit changed files from 'node scripts/precommit_hook.js --ref HEAD~1..HEAD --fix'

* Fixed the tests. The only things to update now are the permissions so we stop using super user and also need to fix the cleanup.

* [CI] Auto-commit changed files from 'node scripts/eslint --no-cache --fix'

* Fixed accessibility test to use new ccr page function.

* Fixed an error in checks.

* Restored original settings.

* Adjusted cleanup.

* Removed exclusive suite.

* Removed unused variable.

* Removed unused variable.

* Working with perms.

* Fixes per comments in PR.

* added follower index user.

* [CI] Auto-commit changed files from 'node scripts/precommit_hook.js --ref HEAD~1..HEAD --fix'

* Added comment about super user issue.

* Removed the console.log.

* Fixed nits per PR.

* Removed extra assertion.

Co-authored-by: cuffs <cuffs@cuffss-Office-MacBook-Pro.local>
Co-authored-by: kibanamachine <42973632+kibanamachine@users.noreply.github.com>
(cherry picked from commit f5acf76)
@cuff-links
Copy link
Contributor Author

💔 Some backports could not be created

Status Branch Result
8.6 Cherrypick failed because the selected commit (f5acf76) is empty. Did you already backport this commit?
8.5

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

Manual backport

To create the backport manually run:

node scripts/backport --pr 142423

Questions ?

Please refer to the Backport tool documentation

cuff-links pushed a commit that referenced this pull request Nov 21, 2022
… (#145936)

# Backport

This will backport the following commits from `main` to `8.5`:
- [CCS Smoke Test for Remote Clusters and Index Management
(#142423)](#142423)

<!--- Backport version: 8.9.7 -->

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

<!--BACKPORT [{"author":{"name":"John
Dorlus","email":"jsdorlus@elastic.co"},"sourceCommit":{"committedDate":"2022-11-01T05:14:21Z","message":"CCS
Smoke Test for Remote Clusters and Index Management (#142423)\n\n*
Removed comment of the issue that was referenced for the skip. But the
tests were already skipped.\r\n\r\n* Added initial tests and page
objects for remtoe clusters.\r\n\r\n* [CI] Auto-commit changed files
from 'node scripts/eslint --no-cache --fix'\r\n\r\n* Fixed the test and
test names.\r\n\r\n* removed exclusive suite.\r\n\r\n* [CI] Auto-commit
changed files from 'node scripts/eslint --no-cache --fix'\r\n\r\n* Fixed
i18n issue.\r\n\r\n* [CI] Auto-commit changed files from 'node
scripts/eslint --no-cache --fix'\r\n\r\n* Added more testing
stuff.\r\n\r\n* [CI] Auto-commit changed files from 'node
scripts/precommit_hook.js --ref HEAD~1..HEAD --fix'\r\n\r\n* Added more
testing stuff.\r\n\r\n* [CI] Auto-commit changed files from 'node
scripts/precommit_hook.js --ref HEAD~1..HEAD --fix'\r\n\r\n* Added test
and stuff.\r\n\r\n* [CI] Auto-commit changed files from 'node
scripts/precommit_hook.js --ref HEAD~1..HEAD --fix'\r\n\r\n* Fixed the
tests. The only things to update now are the permissions so we stop
using super user and also need to fix the cleanup.\r\n\r\n* [CI]
Auto-commit changed files from 'node scripts/eslint --no-cache
--fix'\r\n\r\n* Fixed accessibility test to use new ccr page
function.\r\n\r\n* Fixed an error in checks.\r\n\r\n* Restored original
settings.\r\n\r\n* Adjusted cleanup.\r\n\r\n* Removed exclusive
suite.\r\n\r\n* Removed unused variable.\r\n\r\n* Removed unused
variable.\r\n\r\n* Working with perms.\r\n\r\n* Fixes per comments in
PR.\r\n\r\n* added follower index user.\r\n\r\n* [CI] Auto-commit
changed files from 'node scripts/precommit_hook.js --ref HEAD~1..HEAD
--fix'\r\n\r\n* Added comment about super user issue.\r\n\r\n* Removed
the console.log.\r\n\r\n* Fixed nits per PR.\r\n\r\n* Removed extra
assertion.\r\n\r\nCo-authored-by: cuffs
<cuffs@cuffss-Office-MacBook-Pro.local>\r\nCo-authored-by: kibanamachine
<42973632+kibanamachine@users.noreply.github.com>","sha":"f5acf76351faa3a15af993ffad6582a5a29e55fa","branchLabelMapping":{"^v8.6.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["Team:QA","test_ui_functional","test_xpack_functional","release_note:skip","backport
missing","Feature:Functional
Testing","auto-backport","v8.6.0","v8.5.1"],"number":142423,"url":"#142423
Smoke Test for Remote Clusters and Index Management (#142423)\n\n*
Removed comment of the issue that was referenced for the skip. But the
tests were already skipped.\r\n\r\n* Added initial tests and page
objects for remtoe clusters.\r\n\r\n* [CI] Auto-commit changed files
from 'node scripts/eslint --no-cache --fix'\r\n\r\n* Fixed the test and
test names.\r\n\r\n* removed exclusive suite.\r\n\r\n* [CI] Auto-commit
changed files from 'node scripts/eslint --no-cache --fix'\r\n\r\n* Fixed
i18n issue.\r\n\r\n* [CI] Auto-commit changed files from 'node
scripts/eslint --no-cache --fix'\r\n\r\n* Added more testing
stuff.\r\n\r\n* [CI] Auto-commit changed files from 'node
scripts/precommit_hook.js --ref HEAD~1..HEAD --fix'\r\n\r\n* Added more
testing stuff.\r\n\r\n* [CI] Auto-commit changed files from 'node
scripts/precommit_hook.js --ref HEAD~1..HEAD --fix'\r\n\r\n* Added test
and stuff.\r\n\r\n* [CI] Auto-commit changed files from 'node
scripts/precommit_hook.js --ref HEAD~1..HEAD --fix'\r\n\r\n* Fixed the
tests. The only things to update now are the permissions so we stop
using super user and also need to fix the cleanup.\r\n\r\n* [CI]
Auto-commit changed files from 'node scripts/eslint --no-cache
--fix'\r\n\r\n* Fixed accessibility test to use new ccr page
function.\r\n\r\n* Fixed an error in checks.\r\n\r\n* Restored original
settings.\r\n\r\n* Adjusted cleanup.\r\n\r\n* Removed exclusive
suite.\r\n\r\n* Removed unused variable.\r\n\r\n* Removed unused
variable.\r\n\r\n* Working with perms.\r\n\r\n* Fixes per comments in
PR.\r\n\r\n* added follower index user.\r\n\r\n* [CI] Auto-commit
changed files from 'node scripts/precommit_hook.js --ref HEAD~1..HEAD
--fix'\r\n\r\n* Added comment about super user issue.\r\n\r\n* Removed
the console.log.\r\n\r\n* Fixed nits per PR.\r\n\r\n* Removed extra
assertion.\r\n\r\nCo-authored-by: cuffs
<cuffs@cuffss-Office-MacBook-Pro.local>\r\nCo-authored-by: kibanamachine
<42973632+kibanamachine@users.noreply.github.com>","sha":"f5acf76351faa3a15af993ffad6582a5a29e55fa"}},"sourceBranch":"main","suggestedTargetBranches":["8.5"],"targetPullRequestStates":[{"branch":"main","label":"v8.6.0","labelRegex":"^v8.6.0$","isSourceBranch":true,"state":"MERGED","url":"#142423
Smoke Test for Remote Clusters and Index Management (#142423)\n\n*
Removed comment of the issue that was referenced for the skip. But the
tests were already skipped.\r\n\r\n* Added initial tests and page
objects for remtoe clusters.\r\n\r\n* [CI] Auto-commit changed files
from 'node scripts/eslint --no-cache --fix'\r\n\r\n* Fixed the test and
test names.\r\n\r\n* removed exclusive suite.\r\n\r\n* [CI] Auto-commit
changed files from 'node scripts/eslint --no-cache --fix'\r\n\r\n* Fixed
i18n issue.\r\n\r\n* [CI] Auto-commit changed files from 'node
scripts/eslint --no-cache --fix'\r\n\r\n* Added more testing
stuff.\r\n\r\n* [CI] Auto-commit changed files from 'node
scripts/precommit_hook.js --ref HEAD~1..HEAD --fix'\r\n\r\n* Added more
testing stuff.\r\n\r\n* [CI] Auto-commit changed files from 'node
scripts/precommit_hook.js --ref HEAD~1..HEAD --fix'\r\n\r\n* Added test
and stuff.\r\n\r\n* [CI] Auto-commit changed files from 'node
scripts/precommit_hook.js --ref HEAD~1..HEAD --fix'\r\n\r\n* Fixed the
tests. The only things to update now are the permissions so we stop
using super user and also need to fix the cleanup.\r\n\r\n* [CI]
Auto-commit changed files from 'node scripts/eslint --no-cache
--fix'\r\n\r\n* Fixed accessibility test to use new ccr page
function.\r\n\r\n* Fixed an error in checks.\r\n\r\n* Restored original
settings.\r\n\r\n* Adjusted cleanup.\r\n\r\n* Removed exclusive
suite.\r\n\r\n* Removed unused variable.\r\n\r\n* Removed unused
variable.\r\n\r\n* Working with perms.\r\n\r\n* Fixes per comments in
PR.\r\n\r\n* added follower index user.\r\n\r\n* [CI] Auto-commit
changed files from 'node scripts/precommit_hook.js --ref HEAD~1..HEAD
--fix'\r\n\r\n* Added comment about super user issue.\r\n\r\n* Removed
the console.log.\r\n\r\n* Fixed nits per PR.\r\n\r\n* Removed extra
assertion.\r\n\r\nCo-authored-by: cuffs
<cuffs@cuffss-Office-MacBook-Pro.local>\r\nCo-authored-by: kibanamachine
<42973632+kibanamachine@users.noreply.github.com>","sha":"f5acf76351faa3a15af993ffad6582a5a29e55fa"}},{"branch":"8.5","label":"v8.5.1","labelRegex":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->
@kibanamachine kibanamachine added v8.5.2 and removed backport missing Added to PRs automatically when the are determined to be missing a backport. labels Nov 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-backport Deprecated - use backport:version if exact versions are needed Feature:Functional Testing release_note:skip Skip the PR/issue when compiling release notes Team:QA Team label for QA Team test_ui_functional test_xpack_functional v8.5.1 v8.5.2 v8.6.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants