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

Avoid cache gsutils temporary location #994

Merged

Conversation

v1v
Copy link
Member

@v1v v1v commented Feb 17, 2021

What does this PR do?

gsutil step can be used ini different OS, therefore the caching cannot be used

Why is it important?

Fixes issues like

Caused /etc/sv/jenkins-swarm/C:\Users\jenkins\workspace\Beats_beats_PR-24085\src\github.com\elastic\beats\build@tmp

image

It can cause issues when using the step in different OS

Caused /etc/sv/jenkins-swarm/C:\Users\jenkins\workspace\Beats_beats_PR-24085\src\github.com\elastic\beats\build@tmp
@v1v v1v added bug Something isn't working backport-to-7.x Automatically backport PRs from master ready-to-merge labels Feb 17, 2021
@v1v v1v requested a review from a team February 17, 2021 15:00
@v1v v1v self-assigned this Feb 17, 2021
@botelastic botelastic bot added the groovy label Feb 17, 2021
@apmmachine
Copy link
Contributor

apmmachine 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 #994 updated

  • Start Time: 2021-02-17T16:36:20.848+0000

  • Duration: 13 min 41 sec

  • Commit: be1342b

Test stats 🧪

Test Results
Failed 0
Passed 1046
Skipped 2
Total 1048

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 1046
Skipped 2
Total 1048

@v1v v1v merged commit 585a77b 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 585a77b)

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 bug Something isn't working groovy ready-to-merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants