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

[Canvas] Disable exporting/importing of templates. Optimize pitch images a bit #83098

Merged
merged 2 commits into from
Nov 19, 2020

Conversation

crob611
Copy link
Contributor

@crob611 crob611 commented Nov 10, 2020

Summary

Fixes #81560

This disallows the exporting and importing of Canvas Workpad Template saved objects from Saved Object Management. We don't really have a reason for allowing these exports now, and if we decide there is a reason in the future we can readdress, but with installation from Fleet coming relatively soon, I'm not sure we will have a reason for that.

I also ran all of the images in the pitch_presentation template through some optimizers to reduce quality a bit, and it has significantly reduced the size of that template.

@crob611 crob611 requested a review from a team as a code owner November 10, 2020 20:07
@crob611 crob611 changed the title Disable exporting/importing of templates. Optimize pitch images a bit [Canvas] Disable exporting/importing of templates. Optimize pitch images a bit Nov 10, 2020
@crob611 crob611 added 7.11.0 Feature:Canvas Team:Presentation Presentation Team for Dashboard, Input Controls, and Canvas v7.10.1 v8.0.0 labels Nov 10, 2020
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-presentation (Team:Presentation)

@crob611 crob611 added the release_note:skip Skip the PR/issue when compiling release notes label Nov 10, 2020
@crob611
Copy link
Contributor Author

crob611 commented Nov 10, 2020

@ryankeairns or @MichaelMarcialis you all will probably want to take a look at the resulting pitch template to make sure the optimized images still look ok. Make sure you start up a fresh Elasticsearch instance to make sure you get the newest version of the template,

@jen-huang jen-huang added v7.11.0 and removed 7.11.0 labels Nov 10, 2020
@ryankeairns ryankeairns self-requested a review November 11, 2020 15:03
@crob611
Copy link
Contributor Author

crob611 commented Nov 17, 2020

@elasticmachine merge upstream

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

History

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

@crob611 crob611 merged commit 235cef7 into elastic:master Nov 19, 2020
crob611 pushed a commit to crob611/kibana that referenced this pull request Nov 19, 2020
…elastic#83098)

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
crob611 pushed a commit to crob611/kibana that referenced this pull request Nov 19, 2020
…elastic#83098)

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
gmmorris added a commit to gmmorris/kibana that referenced this pull request Nov 19, 2020
* master:
  skip "Dashboards linked by a drilldown are both copied to a space" (elastic#83824)
  [alerts] adds action group and date to mustache template variables for actions (elastic#83195)
  skip flaky suite (elastic#79389)
  [DOCS] Reallocates limitations to point-of-use (elastic#79582)
  [Enterprise Search] Engine overview layout stub (elastic#83756)
  Disable exporting/importing of templates.  Optimize pitch images a bit (elastic#83098)
  [DOCS] Consolidates plugins (elastic#83712)
  [ML] Space management UI (elastic#83320)
  test just part of the message to avoid updates (elastic#83703)
  [Data Table] Remove extra column in split mode (elastic#83193)
  Improve snapshot error messages (elastic#83785)
  skip flaky suite (elastic#83773)
  skip flaky suite (elastic#83771)
  skip flaky suite (elastic#65278)
  skip flaky suite (elastic#83793)
  [Task Manager] Ensures retries are inferred from the schedule of recurring tasks (elastic#83682)
  [index patterns] improve index pattern cache (elastic#83368)
  [Fleet] Rename ingestManager plugin ID fleet (elastic#83200)
  fixed pagination in connectors list (elastic#83638)
chrisronline pushed a commit to chrisronline/kibana that referenced this pull request Nov 19, 2020
…elastic#83098)

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
@kibanamachine kibanamachine added the backport missing Added to PRs automatically when the are determined to be missing a backport. label Nov 23, 2020
@kibanamachine
Copy link
Contributor

Looks like this PR has backport PRs but they still haven't been merged. Please merge them ASAP to keep the branches relatively in sync.

crob611 pushed a commit that referenced this pull request Nov 24, 2020
…#83098) (#83819)

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
crob611 pushed a commit that referenced this pull request Nov 24, 2020
…#83098) (#83820)

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
@kibanamachine kibanamachine removed the backport missing Added to PRs automatically when the are determined to be missing a backport. label Nov 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature:Canvas release_note:skip Skip the PR/issue when compiling release notes Team:Presentation Presentation Team for Dashboard, Input Controls, and Canvas v7.10.1 v7.11.0 v8.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Out of the box, Kibana is unable to export and import all saved objects because of Canvas template
6 participants