Skip to content

Fix test import objects#82767

Merged
LeeDr merged 3 commits intoelastic:masterfrom
LeeDr:fixTestImportObjects
Nov 9, 2020
Merged

Fix test import objects#82767
LeeDr merged 3 commits intoelastic:masterfrom
LeeDr:fixTestImportObjects

Conversation

@LeeDr
Copy link
Copy Markdown

@LeeDr LeeDr commented Nov 5, 2020

Summary

I think this test must have been failing on Cloud but I didn't find an open issue for it. It could only pass when run against a Kibana instance which had this setting --savedObjects.maxImportPayloadBytes=10485760' (from the OSS functional test config file).

That setting isn't in the allow list of Kibana parameters on Cloud so I wrapped the test in a describe and a skipCloud tag.

The test can be run against Kibana with x-pack and security enabled if that setting is added to the config. I added that setting to x-pack/test/functional/config.js so I could run it locally. We don't have any Jenkins jobs running the OSS tests this way now but we may at some point. And it shouldn't impact any other tests (we'll see).

Checklist

Delete any items that are not applicable to this PR.

@LeeDr LeeDr added Team:Core Platform Core services: plugins, logging, config, saved objects, http, ES client, i18n, etc t// test_ui_functional test-cloud Feature:Functional Testing labels Nov 5, 2020
@LeeDr LeeDr requested review from liza-mae and pgayvallet November 5, 2020 18:50
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/kibana-platform (Team:Platform)

@LeeDr LeeDr added release_note:skip Skip the PR/issue when compiling release notes v7.10.0 v7.11.0 v8.0.0 labels Nov 5, 2020
Copy link
Copy Markdown
Contributor

@pgayvallet pgayvallet 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 doing this

@LeeDr
Copy link
Copy Markdown
Author

LeeDr commented Nov 9, 2020

@elasticmachine merge upstream

@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

History

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

@LeeDr LeeDr merged commit 1885dda into elastic:master Nov 9, 2020
@LeeDr LeeDr deleted the fixTestImportObjects branch November 9, 2020 17:01
LeeDr pushed a commit to LeeDr/kibana that referenced this pull request Nov 9, 2020
Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
@LeeDr LeeDr removed the v7.10.0 label Nov 9, 2020
@LeeDr
Copy link
Copy Markdown
Author

LeeDr commented Nov 9, 2020

NOTE: The page object methods this test uses are not in 7.10 branch so I'll also not backport this test change to that branch.

LeeDr pushed a commit that referenced this pull request Nov 9, 2020
Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
gmmorris added a commit to gmmorris/kibana that referenced this pull request Nov 10, 2020
* master: (39 commits)
  Fix ilm navigation (elastic#81664)
  [Lens] Distinct icons for XY and pie chart value labels toolbar (elastic#82927)
  [data.search.aggs] Throw an error when trying to create an agg type that doesn't exist. (elastic#81509)
  Index patterns api - load field list on server (elastic#82629)
  New events resolver (elastic#82170)
  [App Search] Misc naming tech debt (elastic#82770)
  load empty_kibana in test to have clean starting point (elastic#82772)
  Remove data <--> expressions circular dependencies. (elastic#82685)
  Update 8.0 breaking change template to gather information on how to programmatically detect it. (elastic#82905)
  Add alerting as codeowners to related documentation folder (elastic#82777)
  Add captions to user and space grid pages (elastic#82713)
  add alternate path for x-pack/Cloud test for Lens (elastic#82634)
  Uses asCurrentUser in getClusterUuid (elastic#82908)
  [Alerting][Connectors] Add new executor subaction to get 3rd party case fields (elastic#82519)
  Fix test import objects (elastic#82767)
  [ML] Add option for anomaly charts for metric detector should plot min, mean or max as appropriate (elastic#81662)
  Update alert type selection layout to rows instead of grid (elastic#73665)
  Prevent Kerberos and PKI providers from initiating a new session for unauthenticated XHR/API requests. (elastic#82817)
  Update grunt and related packages (elastic#79327)
  Allow the repository to search across all namespaces (elastic#82863)
  ...
@LeeDr LeeDr added the v7.10.1 label Nov 18, 2020
LeeDr pushed a commit to LeeDr/kibana that referenced this pull request Nov 18, 2020
LeeDr pushed a commit that referenced this pull request Nov 18, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Feature:Functional Testing release_note:skip Skip the PR/issue when compiling release notes Team:Core Platform Core services: plugins, logging, config, saved objects, http, ES client, i18n, etc t// test_ui_functional test-cloud v7.10.1 v7.11.0 v8.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants