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

Set GOTMPDIR in withGoEnvWindows #1742

Merged
merged 2 commits into from
Jun 22, 2022
Merged

Conversation

v1v
Copy link
Member

@v1v v1v commented Jun 22, 2022

What does this PR do?

Set GOTMPDIR

Why is it important?

Use the workspace as the tmp place to host the Golang temporary files.

Issue

See golang/go#42224 (comment)

@v1v v1v requested review from a team as code owners June 22, 2022 08:33
@botelastic botelastic bot added the groovy label Jun 22, 2022
@apmmachine
Copy link
Contributor

apmmachine commented Jun 22, 2022

💚 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 preview

Expand to view the summary

Build stats

  • Start Time: 2022-06-22T09:37:07.225+0000

  • Duration: 9 min 58 sec

Test stats 🧪

Test Results
Failed 0
Passed 1474
Skipped 2
Total 1476

💚 Flaky test report

Tests succeeded.

🤖 GitHub comments

To re-run your PR in the CI, just comment with:

  • /test : Re-trigger the build.

  • run infra tests : Run the test-infra test.

  • /test all : Re-trigger the build for all the stages.

  • run elasticsearch-ci/docs : Re-trigger the docs validation. (use unformatted text in the comment!)

@mergify mergify bot assigned v1v Jun 22, 2022
@v1v v1v added bug Something isn't working ready-to-merge labels Jun 22, 2022
@mergify mergify bot merged commit ad345d0 into elastic:main Jun 22, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working groovy ready-to-merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants