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

Saved Objects export API stable type order #42310

Merged
merged 4 commits into from
Jul 31, 2019

Conversation

kobelb
Copy link
Contributor

@kobelb kobelb commented Jul 30, 2019

The saved object export API uses the types which we "scan" from the various plugins as route validation. We started to see these saved object types be non-deterministic, which led to unstable errors being thrown which were breaking tests.

This PR adds in a simplistic sort to ensure that we're getting stable error message.

Resolves #42305

@kobelb kobelb requested a review from a team as a code owner July 30, 2019 23:43
@kobelb kobelb added release_note:skip Skip the PR/issue when compiling release notes v7.4.0 labels Jul 30, 2019
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@kobelb kobelb requested a review from legrego July 31, 2019 01:24
Copy link
Member

@legrego legrego left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks for the fix, Brandon!

@kobelb kobelb merged commit f2f941f into elastic:master Jul 31, 2019
@kobelb kobelb deleted the fix/stable-so-types branch July 31, 2019 02:28
jloleysens added a commit to jloleysens/kibana that referenced this pull request Jul 31, 2019
…-or-edit-existing-rollup-job

* 'master' of github.com:elastic/kibana: (114 commits)
  [ML] Fixing empty index pattern list (elastic#42299)
  [Markdown] Shim new platform - cleanup plugin (elastic#41760)
  [Code] Enable hierarchicalDocumentSymbolSupport for java language server (elastic#42233)
  Add New Platform mocks for data plugin (elastic#42261)
  Http server route handler implementation (elastic#41894)
  [SR] Allow custom index pattern to be used instead of selectable list when choosing indices to restore (elastic#41534)
  [Code] distributed Code abstraction (elastic#41374)
  [SIEM] Sets page titles to the current page you are on  (elastic#42157)
  Saved Objects export API stable type order (elastic#42310)
  cancellation of interpreter execution (elastic#40238)
  [SIEM] Fixes a crash when Machine Learning influencers is an undefined value (elastic#42198)
  Changed the job to work with a dedicated index (elastic#42297)
  FTR: fix testSubjects.missingOrFail (elastic#42290)
  Increase retry timeout to prevent flaky tests (elastic#42291)
  Spaces - make space a hidden saved object type (elastic#41688)
  Allow applications to register feature privileges which are excluded from the base privileges (elastic#41300)
  Disable flaky log column reorder test (elastic#42285)
  Fixing add element in element reducer (elastic#42276)
  Fix infinite loop (elastic#42228)
  [Maps][File upload] Remove geojson deep clone logic, handle on maps side (elastic#41835)
  ...
kobelb added a commit that referenced this pull request Jul 31, 2019
* Sorting export's valid types

* Updating the saved object api integration export test

* Fixing OSS also
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release_note:skip Skip the PR/issue when compiling release notes v7.4.0
Projects
None yet
3 participants