Skip to content

Conversation

Niccolum
Copy link
Contributor

What does this pull request do?

Related issues

closes #ISSUE

@cla-checker-service
Copy link

cla-checker-service bot commented Apr 27, 2021

💚 CLA has been signed

@ghost
Copy link

ghost commented Apr 27, 2021

💔 Build Failed

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: Started by user Benjamin Wohlwend

  • Start Time: 2021-04-27T09:24:26.908+0000

  • Duration: 18 min 21 sec

  • Commit: 03eef6c

Test stats 🧪

Test Results
Failed 0
Passed 7344
Skipped 7899
Total 15243

Trends 🧪

Image of Build Times

Image of Tests

Steps errors 11

Expand to view the steps failures

Show only the first 10 steps failures

Install python3:3.6
  • Took 0 min 7 sec . View more details on here
Install python3:3.6
  • Took 0 min 6 sec . View more details on here
Install python3:3.7
  • Took 0 min 10 sec . View more details on here
Install python3:3.7
  • Took 0 min 7 sec . View more details on here
Install python3:3.7
  • Took 0 min 7 sec . View more details on here
Install python3:3.8
  • Took 0 min 10 sec . View more details on here
Install python3:3.8
  • Took 0 min 7 sec . View more details on here
Install python3:3.8
  • Took 0 min 7 sec . View more details on here
Archive the artifacts
  • Took 0 min 0 sec . View more details on here
  • Description: [2021-04-27T09:42:45.586Z] Archiving artifacts hudson.AbortException: script returned exit code 1
Error signal
  • Took 0 min 0 sec . View more details on here
  • Description: hudson.AbortException: script returned exit code 1

Log output

Expand to view the last 100 lines of log output

[2021-04-27T09:42:11.648Z] + python3 -m coverage combine [2021-04-27T09:42:43.772Z] + python3 -m coverage xml [2021-04-27T09:42:44.740Z] [Cobertura] Skipping Cobertura coverage report as build was not SUCCESS or better ... [2021-04-27T09:42:44.740Z] [2021-04-27T09:42:45.116Z] + curl -sLO https://code.jquery.com/jquery-3.3.1.slim.min.js [2021-04-27T09:42:45.419Z] + curl -sLO https://cdn.jsdelivr.net/npm/htmlson.js@1.0.4/src/htmlson.js [2021-04-27T09:42:45.586Z] Archiving artifacts [2021-04-27T09:42:45.733Z] Stage "Building packages" skipped due to earlier failure(s) [2021-04-27T09:42:45.782Z] Stage "Integration Tests" skipped due to earlier failure(s) [2021-04-27T09:42:45.821Z] Stage "Benchmarks" skipped due to earlier failure(s) [2021-04-27T09:42:45.905Z] Stage "Prepare Release" skipped due to earlier failure(s) [2021-04-27T09:42:45.923Z] Stage "Prepare Release" skipped due to earlier failure(s) [2021-04-27T09:42:45.964Z] Stage "Prepare Release" skipped due to earlier failure(s) [2021-04-27T09:42:46.003Z] Stage "Prepare Release" skipped due to earlier failure(s) [2021-04-27T09:42:46.764Z] Running on Jenkins in /var/lib/jenkins/workspace/hon_apm-agent-python-mbp_PR-1117 [2021-04-27T09:42:46.878Z] [INFO] getVaultSecret: Getting secrets [2021-04-27T09:42:46.929Z] Masking supported pattern matches of $VAULT_ADDR or $VAULT_ROLE_ID or $VAULT_SECRET_ID [2021-04-27T09:42:47.706Z] + chmod 755 generate-build-data.sh [2021-04-27T09:42:47.707Z] + ./generate-build-data.sh https://apm-ci.elastic.co/blue/rest/organizations/jenkins/pipelines/apm-agent-python/apm-agent-python-mbp/PR-1117/ https://apm-ci.elastic.co/blue/rest/organizations/jenkins/pipelines/apm-agent-python/apm-agent-python-mbp/PR-1117/runs/7 FAILURE 1100535 [2021-04-27T09:42:47.707Z] INFO: curl https://apm-ci.elastic.co/blue/rest/organizations/jenkins/pipelines/apm-agent-python/apm-agent-python-mbp/PR-1117/runs/7/steps/?limit=10000 -o steps-info.json [2021-04-27T09:42:52.903Z] INFO: curl https://apm-ci.elastic.co/blue/rest/organizations/jenkins/pipelines/apm-agent-python/apm-agent-python-mbp/PR-1117/runs/7/tests/?status=FAILED -o tests-errors.json 

💚 Flaky test report

Tests succeeded.

Expand to view the summary

Test stats 🧪

Test Results
Failed 0
Passed 7344
Skipped 7899
Total 15243

Copy link
Contributor

@beniwohli beniwohli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great catch and nice fix! Just a small nitpick about naming :)

Niccolum and others added 2 commits April 27, 2021 12:04
Co-authored-by: Benjamin Wohlwend <bw@piquadrat.ch>
Co-authored-by: Benjamin Wohlwend <bw@piquadrat.ch>
@beniwohli
Copy link
Contributor

@Niccolum if you have a second, could you have a look at our CLA and sign it if you agree? That'll allow me to merge your PR :)

https://www.elastic.co/contributor-agreement

@Niccolum
Copy link
Contributor Author

Niccolum commented Apr 27, 2021

@Niccolum if you have a second, could you have a look at our CLA and sign it if you agree? That'll allow me to merge your PR :)

https://www.elastic.co/contributor-agreement

@beniwohli, I signed it. Hope, I did it correct )

@beniwohli
Copy link
Contributor

@Niccolum sorry about the hassle, but could you sign the CLA again using the email you used for the commits? The one matching n...v@e....com

@Niccolum
Copy link
Contributor Author

Niccolum commented Apr 27, 2021

@Niccolum sorry about the hassle, but could you sign the CLA again using the email you used for the commits? The one matching n...v@e....com

@beniwohli Oh, sorry for this
Is it okay?
Снимок экрана от 2021-04-27 14-56-49

@beniwohli
Copy link
Contributor

@Niccolum yes, perfect, thanks! I'll merge the PR in a bit!

@beniwohli beniwohli merged commit 03eef6c into elastic:master Apr 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
3 participants