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

CI: curl seems to be available but where cannot find it #24965

Merged
merged 1 commit into from
Apr 7, 2021

Conversation

v1v
Copy link
Member

@v1v v1v commented Apr 7, 2021

What does this PR do?

Use the curl binary if it's there instead installing, somehow where cannot find it, but as far as I see the curl is already provisioned in the Windows workers.

Why is it important?

Reduce the interactions with chocolatey as much as possible.

@botelastic botelastic bot added the needs_team Indicates that the issue/PR needs a Team:* label label Apr 7, 2021
@v1v v1v added the Team:Automation Label for the Observability productivity team label Apr 7, 2021
@botelastic botelastic bot removed the needs_team Indicates that the issue/PR needs a Team:* label label Apr 7, 2021
@v1v
Copy link
Member Author

v1v commented Apr 7, 2021

Tests

image

image

image

@v1v v1v self-assigned this Apr 7, 2021
@v1v v1v marked this pull request as ready for review April 7, 2021 14:54
@v1v v1v requested a review from a team as a code owner April 7, 2021 14:54
@elasticmachine
Copy link
Collaborator

elasticmachine commented Apr 7, 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: v1v commented: /test

  • Start Time: 2021-04-07T15:42:32.921+0000

  • Duration: 33 min 50 sec

  • Commit: 8cabe5d

Test stats 🧪

Test Results
Failed 0
Passed 728
Skipped 148
Total 876

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 728
Skipped 148
Total 876

@v1v
Copy link
Member Author

v1v commented Apr 7, 2021

/test

@v1v v1v added backport-v7.11.0 Automated backport with mergify backport-v7.12.0 Automated backport with mergify backport-v7.13.0 Automated backport with mergify labels Apr 7, 2021
@v1v v1v merged commit 156ec34 into elastic:master Apr 7, 2021
@v1v v1v deleted the feature/scripts-windows-installing branch April 7, 2021 16:46
mergify bot pushed a commit that referenced this pull request Apr 7, 2021
(cherry picked from commit 156ec34)

# Conflicts:
#	.ci/scripts/install-tools.bat
v1v added a commit to v1v/beats that referenced this pull request Apr 7, 2021
…eline

* upstream/master:
  CI: curl seems to be available but where cannot find it (elastic#24965)
  [mergify] backport automation to 7.x, 7.12 and 7.11 (elastic#24608)
  Only show deprecation warnings for CN-based verification once (elastic#24948)
  CI: use ubuntu-20 (elastic#24963)
  Debug empty root field instead of error (elastic#24966)
  Kubernetes_secrets provider improvements (elastic#24912)
mergify bot pushed a commit that referenced this pull request Apr 7, 2021
(cherry picked from commit 156ec34)

# Conflicts:
#	.ci/scripts/install-tools.bat
mergify bot pushed a commit that referenced this pull request Apr 7, 2021
(cherry picked from commit 156ec34)

# Conflicts:
#	.ci/scripts/install-tools.bat
v1v added a commit that referenced this pull request Jul 12, 2021
v1v added a commit that referenced this pull request Jul 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-v7.11.0 Automated backport with mergify backport-v7.12.0 Automated backport with mergify backport-v7.13.0 Automated backport with mergify Team:Automation Label for the Observability productivity team v7.12.0 v7.13.0 v7.14.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants