8000 test: change base Debian image from Debian 9 to Debian 11 to include a supported Python3 version for node-gyp by wyk9787 · Pull Request #187 · google/pprof-nodejs · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

test: change base Debian image from Debian 9 to Debian 11 to include a supported Python3 version for node-gyp #187

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 29 commits into from
Oct 14, 2021

Conversation

wyk9787
Copy link
Contributor
@wyk9787 wyk9787 commented Oct 11, 2021

No description provided.

@wyk9787 wyk9787 requested a review from nolanmar511 October 11, 2021 20:58
@wyk9787
Copy link
Contributor Author
wyk9787 commented Oct 11, 2021

@nolanmar511 , are there any other tests other than Github Actions I need to run to make sure this PR works as intended?

@nolanmar511
Copy link
Contributor

The Kokoro tests will run automatically on each new commit.

It looks like one fails. I think changing "python3" to "python3.6" on this line would likely let the failing test pass.

Copy link
Contributor
@nolanmar511 nolanmar511 left a comment

Choose a reason for hiding this comment

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

LGTM pending passing kokoro tests.

@codecov-commenter
Copy link

Codecov Report

Merging #187 (0890b34) into main (40315d3) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #187   +/-   ##
=======================================
  Coverage   42.21%   42.21%           
=======================================
  Files          14       14           
  Lines        2061     2061           
  Branches       42       42           
=======================================
  Hits          870      870           
  Misses       1173     1173           
  Partials       18       18           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 40315d3...0890b34. Read the comment docs.

@wyk9787 wyk9787 changed the title test: install Python 3.6 as a supported dependency for node-gyp to pass the integration tests test: change base Debian image from Debian 9 to Debian 11 to include a supported Python3 version for node-gyp Oct 14, 2021
@wyk9787 wyk9787 requested a review from nolanmar511 October 14, 2021 21:42
@wyk9787 wyk9787 merged commit 408f496 into google:main Oct 14, 2021
aabmass pushed a commit to aabmass/pprof-nodejs that referenced this pull request Jul 21, 2023
aabmass pushed a commit to aabmass/pprof-nodejs that referenced this pull request Jul 21, 2023
aabmass added a commit that referenced this pull request Jul 21, 2023
* chore(deps): update golang docker tag to v1.17 (#182)

* test: change base Debian image from Debian 9 to Debian 11 to include a supported Python3 version for node-gyp (#187)

* fix(deps): update dependency protobufjs to ~7.2.0 [security] (#254)

* fix(deps): update dependency protobufjs to ~7.2.0 [security]

* Pull in protobufjs-cli tools (pbjs and pbts) and regen proto stubs

* Explicitly upgrade protobufjs past vulnerability, regenerate package-lock.json

---------

Co-authored-by: Aaron Abbott <aaronabbott@google.com>

* Update package-lock so there are no prod vulnerabilities

* chore(deps): update dependency sinon to v15 (#237)

* npm run fix with new package-lock.json

* chore: remove testing in windows from workflow (#202)

* Fix system test to use older npm for compat

* test: bump debian version in system test (#251)

* test: specify python3 when building docker image (#192)

* Use alpine version with python < 3.11

---------

Co-authored-by: WhiteSource Renovate <bot@renovateapp.com>
Co-authored-by: Garrett Wang <garrettwang@google.com>
Co-authored-by: Amarin (Um) Phaosawasdi <amchiclet@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants
0