Skip to content
This repository was archived by the owner on Oct 28, 2024. It is now read-only.

Support curl in addition to wget for gsutil#988

Merged
mergify[bot] merged 1 commit intoelastic:masterfrom
v1v:feature/curl-support
Feb 17, 2021
Merged

Support curl in addition to wget for gsutil#988
mergify[bot] merged 1 commit intoelastic:masterfrom
v1v:feature/curl-support

Conversation

@v1v
Copy link
Copy Markdown
Member

@v1v v1v commented Feb 17, 2021

What does this PR do?

Fallback to curl in case wget is not installed

Why is it important?

As long as gsutil is not widely available in each CI Worker we need to configure the installation, so no all the workers got wget so let's try with curl also

Related issues

Closes #ISSUE\

Tests

$ curl -sSLo gsutil.tar.gz --retry 3 --retry-delay 2 --max-time 10 https://dl.google.com/dl/cloudsdk/channels/rapid/downloads/google-cloud-sdk-319.0.0-linux-x86_64.tar.gz 
$ ls -l gsutil.tar.gz 
-rw-r--r--  1 vmartinez  staff  86364996 17 Feb 09:18 gsutil.tar.gz       
x google-cloud-sdk/.install/.download/
x google-cloud-sdk/.install/anthoscli-linux-x86_64.manifest

@botelastic botelastic Bot added the groovy label Feb 17, 2021
@v1v v1v requested a review from a team February 17, 2021 09:20
@v1v v1v self-assigned this Feb 17, 2021
@v1v v1v added the enhancement New feature or request label Feb 17, 2021
@ghost
Copy link
Copy Markdown

ghost commented Feb 17, 2021

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview

Expand to view the summary

Build stats

  • Build Cause: Pull request #988 opened

  • Start Time: 2021-02-17T09:20:37.531+0000

  • Duration: 12 min 55 sec

  • Commit: 0fffd2e

Test stats 🧪

Test Results
Failed 0
Passed 1048
Skipped 2
Total 1050

Trends 🧪

Image of Build Times

Image of Tests

💚 Flaky test report

Tests succeeded.

Expand to view the summary

Test stats 🧪

Test Results
Failed 0
Passed 1048
Skipped 2
Total 1050

@v1v v1v added backport-to-7.x Automatically backport PRs from master ready-to-merge labels Feb 17, 2021
@mergify mergify Bot merged commit b757105 into elastic:master Feb 17, 2021
mergify Bot pushed a commit that referenced this pull request Feb 17, 2021
mergify Bot added a commit that referenced this pull request Feb 17, 2021
(cherry picked from commit b757105)

Co-authored-by: Victor Martinez <victormartinezrubio@gmail.com>
v1v added a commit that referenced this pull request Mar 18, 2021
* upstream/7.x:
  Fix the getCause and getCauses for the AbortException (#1041) (#1042)
  gsutil for windows with python (#1008) (#1009)
  Report only one DeleteDir step failure (#1006) (#1007)
  [cosmetic] Replace multiline chars in the GitHub PR comment (#998) (#1003)
  [cosmetic] add the author (#999) (#1002)
  chore: use opentelemetry release (#1000) (#1001)
  Avoid cache gsutils temporary location (#994) (#995)
  [JCasC] google cloud engine local setup (#979) (#992)
  Support curl in addition to wget for gsutil (#988) (#989)
  Support windows for gsutil (#985) (#987)
  Add googleStorageUploadExt step (#867) (#982)
  Add releaseNotification step to simplify agent release pipelines (#976) (#981)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

backport-to-7.x Automatically backport PRs from master enhancement New feature or request groovy ready-to-merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants