Skip to content
This repository has been archived by the owner on Aug 9, 2022. It is now read-only.

Fix chromium path for puppeteer #232

Merged
merged 5 commits into from
Dec 1, 2020
Merged

Fix chromium path for puppeteer #232

merged 5 commits into from
Dec 1, 2020

Conversation

joshuali925
Copy link
Contributor

@joshuali925 joshuali925 commented Dec 1, 2020

Issue #, if available:

Description of changes:
check for chromium path for puppeteer to use instead of using a constant path

possible chromium paths:

  './plugins/opendistroReportsKibana/.chromium/headless_shell',
  './plugins/kibana-reports/.chromium/headless_shell',
  './.chromium/headless_shell',
  '../plugins/opendistroReportsKibana/.chromium/headless_shell',
  '../plugins/kibana-reports/.chromium/headless_shell',

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Copy link
Contributor

@davidcui1225 davidcui1225 left a comment

Choose a reason for hiding this comment

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

LGTM

@joshuali925 joshuali925 merged commit 92ab019 into opendistro-for-elasticsearch:dev Dec 1, 2020
joshuali925 added a commit that referenced this pull request Dec 1, 2020
* Fix chromium path

* Add more possible paths

* Add error message

* Use plugin id constant
zhongnansu pushed a commit to zhongnansu/kibana-reports that referenced this pull request Dec 4, 2020
* Fix chromium path

* Add more possible paths

* Add error message

* Use plugin id constant
zhongnansu pushed a commit to zhongnansu/kibana-reports that referenced this pull request Dec 7, 2020
* Fix chromium path

* Add more possible paths

* Add error message

* Use plugin id constant
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants