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

Use custom GECKODRIVER_CDNURL #41513

Merged
merged 2 commits into from
Jul 19, 2019

Conversation

spalger
Copy link
Contributor

@spalger spalger commented Jul 18, 2019

Tried submitting a PR to 6.8 and can't get it to run without hitting 500's from github trying to download the geckodriver distributable. This PR uses a simple cloud function to access artifacts that are cached in GCS, the function checks if the object exists, downloads it if necessary, and redirects to the public URL for the artifact.

@elasticmachine

This comment has been minimized.

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-operations

@spalger spalger added release_note:skip Skip the PR/issue when compiling release notes test_infra labels Jul 18, 2019
@spalger spalger marked this pull request as ready for review July 18, 2019 22:41
@spalger spalger changed the title test using a GECKODRIVER_CDNURL Use custom GECKODRIVER_CDNURL Jul 18, 2019
@spalger spalger requested a review from tylersmalley July 18, 2019 22:42
@spalger spalger changed the base branch from 6.8 to master July 18, 2019 22:46
@spalger spalger force-pushed the implement/geckodriver-proxy branch from 5698d47 to c05aa39 Compare July 18, 2019 22:52
@spalger spalger requested a review from a team as a code owner July 18, 2019 22:52
Copy link
Contributor

@tylersmalley tylersmalley left a comment

Choose a reason for hiding this comment

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

LGTM!

@spalger
Copy link
Contributor Author

spalger commented Jul 18, 2019

retest

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@spalger spalger merged commit c040914 into elastic:master Jul 19, 2019
spalger pushed a commit to spalger/kibana that referenced this pull request Jul 19, 2019
spalger pushed a commit to spalger/kibana that referenced this pull request Jul 19, 2019
spalger pushed a commit to spalger/kibana that referenced this pull request Jul 19, 2019
spalger pushed a commit to spalger/kibana that referenced this pull request Jul 19, 2019
# Conflicts:
#	.ci/packer_cache.sh
#	src/dev/ci_setup/setup.sh
spalger pushed a commit to spalger/kibana that referenced this pull request Jul 19, 2019
# Conflicts:
#	.ci/packer_cache.sh
#	src/dev/ci_setup/setup.sh
spalger pushed a commit to spalger/kibana that referenced this pull request Jul 19, 2019
# Conflicts:
#	.ci/packer_cache.sh
#	src/dev/ci_setup/setup.sh
spalger pushed a commit that referenced this pull request Jul 19, 2019
# Conflicts:
#	.ci/packer_cache.sh
#	src/dev/ci_setup/setup.sh
spalger pushed a commit that referenced this pull request Jul 19, 2019
# Conflicts:
#	.ci/packer_cache.sh
#	src/dev/ci_setup/setup.sh
spalger pushed a commit that referenced this pull request Jul 19, 2019
# Conflicts:
#	.ci/packer_cache.sh
#	src/dev/ci_setup/setup.sh
@spalger
Copy link
Contributor Author

spalger commented Jul 19, 2019

7.x/7.4: bf21b41
7.3: 7314177
7.2: 25f77f4
7.1: 033092a
7.0: cdb18f1
6.8: a65faaf

@spalger spalger deleted the implement/geckodriver-proxy branch July 19, 2019 07:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants