Skip to content

chore(perf): use pyenv to install #2731

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

Merged
merged 7 commits into from
Aug 6, 2021
Merged

Conversation

majorgreys
Copy link
Contributor

Use pyenv to install Python. This is necessary for our performance testing as the binaries in the official docker python images have had their symbols stripped (docker-library/python#483).

@majorgreys majorgreys requested a review from a team as a code owner August 3, 2021 21:29
@majorgreys majorgreys added the changelog/no-changelog A changelog entry is not required for this PR. label Aug 3, 2021
majorgreys and others added 2 commits August 4, 2021 11:43
Co-authored-by: Brett Langdon <me@brett.is>
@codecov-commenter
Copy link

codecov-commenter commented Aug 5, 2021

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 84.94%. Comparing base (c5b5bd2) to head (7860c0e).

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2731      +/-   ##
==========================================
+ Coverage   84.84%   84.94%   +0.10%     
==========================================
  Files         586      589       +3     
  Lines       42771    43036     +265     
==========================================
+ Hits        36290    36558     +268     
+ Misses       6481     6478       -3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@mergify mergify bot merged commit a98bfb3 into DataDog:master Aug 6, 2021
Kyle-Verhoog pushed a commit to Kyle-Verhoog/dd-trace-py that referenced this pull request May 10, 2022
* chore(perf): use pyenv to install

* pin pyenv

* Apply suggestions from code review

Co-authored-by: Brett Langdon <me@brett.is>

* fix

Co-authored-by: Brett Langdon <me@brett.is>
Co-authored-by: Nicolas Vivet <nicolas.vivet@datadoghq.com>
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog/no-changelog A changelog entry is not required for this PR.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy