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

test-infra: add curl validation on windows #1069

Merged
merged 6 commits into from
Apr 7, 2021

Conversation

v1v
Copy link
Member

@v1v v1v commented Apr 7, 2021

What does this PR do?

Validate curl with test-infra

Why is it important?

Somehow Beats scripts don't find the curl installer and try to install it again.

[2021-03-31T07:34:21.427Z] C:\Users\jenkins\workspace\PR-24852-3-953a207b-0e8b-46d4-be83-0a2272f81bed\src\github.com\elastic\beats>where /q curl 
[2021-03-31T07:34:21.689Z] 
[2021-03-31T07:34:21.689Z] C:\Users\jenkins\workspace\PR-24852-3-953a207b-0e8b-46d4-be83-0a2272f81bed\src\github.com\elastic\beats>IF ERRORLEVEL 1 (choco install curl -y --no-progress --skipdownloadcache ) 
[2021-03-31T07:34:22.086Z] Chocolatey v0.10.15
[2021-03-31T07:34:22.086Z] Installing the following packages:
[2021-03-31T07:34:22.086Z] curl
[2021-03-31T07:34:22.086Z] By installing you accept licenses for the packages.
[2021-03-31T07:34:22.346Z] Error retrieving packages from source 'https://chocolatey.org/api/v2/':
[2021-03-31T07:34:22.346Z]  The remote server returned an error: (503) Server Unavailable.
[2021-03-31T07:34:22.606Z] curl not installed. The package was not found with the source(s) listed.
[2021-03-31T07:34:22.607Z]  Source(s): 'https://chocolatey.org/api/v2/'

Hopefully if it's there we can easily change the script from where /q curl to curl and if it fails then do the installation instead.

v1v and others added 4 commits March 18, 2021 21:14
* 'master' of github.com:elastic/apm-pipeline-library: (40 commits)
  Fix and simplify buildStatus step (#1066)
  Add benchmark reporter image to build pipeline (#1065)
  docs: update CHANGELOG.md
  [maven-release-plugin] prepare for next development iteration
  [maven-release-plugin] prepare release v1.1.207
  feat: stage status cache (#1061)
  feat: step hasCommentAuthorWritePermissions (#1064)
  ARM workers are now ephemeral (#1062)
  docs: update metricbeat step docs (#1057)
  Update beats-ci workers for the test-infra validation (#1058)
  feat: allow to remove padding (#1059)
  mergify: priority is now a pro feature (#1060)
  docs: update CHANGELOG.md
  [maven-release-plugin] prepare for next development iteration
  [maven-release-plugin] prepare release v1.1.206
  chore(deps): bump groovy-eclipse-batch from 3.0.7-02 to 3.0.7-03 (#1045)
  chore: bump Elastic Stack versions (#1052)
  Add GCE JCasC (#1053)
  feat: metricbeat step (#1050)
  chore: refactor filebeat step (#1049)
  ...
* 'master' of github.com:elastic/apm-pipeline-library:
  test: add Ubuntu 20.04 to the tests (#1068)
  docs: update CHANGELOG.md
  [maven-release-plugin] prepare for next development iteration
  [maven-release-plugin] prepare release v1.1.208
  Disable unique ephemeral worker by default (#1067)
t checkout -b# the commit.
@v1v v1v requested a review from a team April 7, 2021 12:15
@v1v v1v self-assigned this Apr 7, 2021
@apmmachine
Copy link
Contributor

apmmachine 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: Pull request #1069 updated

  • Start Time: 2021-04-07T15:14:55.988+0000

  • Duration: 34 min 34 sec

  • Commit: a814c3d

Test stats 🧪

Test Results
Failed 0
Passed 1126
Skipped 2
Total 1128

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 1126
Skipped 2
Total 1128

@v1v v1v changed the title Feature/beats curl test-infra: add curl validation on windows Apr 7, 2021
@mergify mergify bot merged commit 4a58b62 into elastic:master Apr 7, 2021
v1v referenced this pull request in v1v/apm-pipeline-library Apr 15, 2021
* 'master' of github.com:elastic/apm-pipeline-library:
  fix: use toJSON instead readJSON (#1071)
  test-infra: add curl validation on windows (#1069)
  CI: skip using ubuntu label in the beats test-infra (#1070)
v1v added a commit that referenced this pull request Apr 18, 2021
* upstream/master: (56 commits)
  docs: update CHANGELOG.md
  [maven-release-plugin] prepare for next development iteration
  [maven-release-plugin] prepare release v1.1.209
  security: disable codecov bash updater
  chore(deps-dev): bump mockito-core from 3.8.0 to 3.9.0 (#1073)
  fix: use toJSON instead readJSON (#1071)
  test-infra: add curl validation on windows (#1069)
  CI: skip using ubuntu label in the beats test-infra (#1070)
  test: add Ubuntu 20.04 to the tests (#1068)
  docs: update CHANGELOG.md
  [maven-release-plugin] prepare for next development iteration
  [maven-release-plugin] prepare release v1.1.208
  Disable unique ephemeral worker by default (#1067)
  Fix and simplify buildStatus step (#1066)
  Add benchmark reporter image to build pipeline (#1065)
  docs: update CHANGELOG.md
  [maven-release-plugin] prepare for next development iteration
  [maven-release-plugin] prepare release v1.1.207
  feat: stage status cache (#1061)
  feat: step hasCommentAuthorWritePermissions (#1064)
  ...
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants