Skip to content

[7.2] Use kibana-ci-proxy-cache for chrome and gecko drivers (#41581)#41630

Merged
spalger merged 1 commit intoelastic:7.2from
spalger:backport/7.2/pr-41581
Jul 20, 2019
Merged

[7.2] Use kibana-ci-proxy-cache for chrome and gecko drivers (#41581)#41630
spalger merged 1 commit intoelastic:7.2from
spalger:backport/7.2/pr-41581

Conversation

@spalger
Copy link
Contributor

@spalger spalger commented Jul 20, 2019

Backports the following commits to 7.2:

* [ci] download chromedriver and geckodriver through a caching proxy

* restore chromedriver and geckodriver caching in bootstrap cache
@spalger spalger added the backport This PR is a backport of another PR label Jul 20, 2019
@elasticmachine
Copy link
Contributor

💔 Build Failed

@spalger
Copy link
Contributor Author

spalger commented Jul 20, 2019

Looks like we got a 500 because two jobs raced to put the object in the cache at the same time, which should happen very rarely, but will be more likely in situations like this where we're caching a new version of the asset for the first time.

textPayload timestamp
"GCS [kibana-ci-proxy-cache/geckodriver/v0.22.0/geckodriver-v0.22.0-linux64.tar.gz] does not exist, fetching from https://github.com/mozilla/geckodriver/releases/download/v0.22.0/geckodriver-v0.22.0-linux64.tar.gz" 2019-07-20T04:34:04.051Z
"GCS [kibana-ci-proxy-cache/geckodriver/v0.22.0/geckodriver-v0.22.0-linux64.tar.gz] does not exist, fetching from https://github.com/mozilla/geckodriver/releases/download/v0.22.0/geckodriver-v0.22.0-linux64.tar.gz" 2019-07-20T04:34:06.453Z
Uploaded [https://github.com/mozilla/geckodriver/releases/download/v0.22.0/geckodriver-v0.22.0-linux64.tar.gz] to GCS [kibana-ci-proxy-cache/geckodriver/v0.22.0/geckodriver-v0.22.0-linux64.tar.gz] 2019-07-20T04:34:07.802Z
UNCAUGHT HANDLER ERROR: Error: kibana-ci-proxy-cache@elastic-kibana-184716.iam.gserviceaccount.com does not have storage.objects.delete access to kibana-ci-proxy-cache/geckodriver/v0.22.0/geckodriver-v0.22.0-linux64.tar.gz. 2019-07-20T04:34:08.576Z

I'll give the functions the storage.objects.delete permission so that future instances will just overwrite the cache on collision.

Jenkins, test this please.

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@spalger spalger merged commit bcc6da9 into elastic:7.2 Jul 20, 2019
@spalger spalger deleted the backport/7.2/pr-41581 branch July 20, 2019 07:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants