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.

Sign up for GitHub

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

Jump to bottom

Pants-provided-Python for Pex CLI doesn't work in docker environments #20313

Closed
thejcannon opened this issue Dec 19, 2023 · 0 comments · Fixed by #20314
Closed

Pants-provided-Python for Pex CLI doesn't work in docker environments #20313

thejcannon opened this issue Dec 19, 2023 · 0 comments · Fixed by #20314
Assignees
@thejcannon
Labels
bug
Milestone
2.18.x

Comments

@thejcannon
Copy link
Member

Describe the bug
Trying to use docker_environment to run a test on a machine without Python installed will result in an error:

Failed to find a compatible PEX_PYTHON=.python-build-standalone/c12164f0e9228ec20704c1aba97eb31b8e2a482d41943d541cc8e3a9e84f7349/bin/python3.

No interpreters could be found on the system.

Pants version
2.20

OS
Linux host and linux container

Additional info
@thejcannon thejcannon added the bug label Dec 19, 2023
@thejcannon thejcannon self-assigned this Dec 19, 2023
@thejcannon thejcannon added this to the 2.18.x milestone Dec 19, 2023
@thejcannon thejcannon mentioned this issue Dec 19, 2023
Merged
thejcannon added a commit that referenced this issue Dec 26, 2023
@thejcannon
Make docker_environment's Pants-provided-PBS an absolute path (#20314)
550cc24 
Fixes #20313 by ensuring that the `PEX_PYTHON` used for Pex CLI
invocations inside a docker environment is an absolute path.

I also did some drive-by improvements of the test code.
WorkerPants pushed a commit that referenced this issue Dec 26, 2023
@thejcannon @WorkerPants
Make docker_environment's Pants-provided-PBS an absolute path (#20314)
f23e572 
Fixes #20313 by ensuring that the `PEX_PYTHON` used for Pex CLI
invocations inside a docker environment is an absolute path.

I also did some drive-by improvements of the test code.
@WorkerPants WorkerPants mentioned this issue Dec 26, 2023
Merged
huonw pushed a commit that referenced this issue Dec 28, 2023
@WorkerPants @thejcannon
Make docker_environment's Pants-provided-PBS an absolute path (Cherry…
af6864f 
…-pick of #20314) (#20339)

Fixes #20313 by ensuring that the `PEX_PYTHON` used for Pex CLI
invocations inside a docker environment is an absolute path.

I also did some drive-by improvements of the test code.

Co-authored-by: Josh Cannon <[email protected]>
alonsodomin pushed a commit to alonsodomin/pants that referenced this issue Dec 28, 2023
@thejcannon @alonsodomin
Make docker_environment's Pants-provided-PBS an absolute path (pantsb…
879a8d2 
…uild#20314)

Fixes pantsbuild#20313 by ensuring that the `PEX_PYTHON` used for Pex CLI
invocations inside a docker environment is an absolute path.

I also did some drive-by improvements of the test code.
thejcannon added a commit that referenced this issue Dec 30, 2023
@thejcannon
Make docker_environment's Pants-provided-PBS an absolute path (#20314)
73474ac 
Fixes #20313 by ensuring that the `PEX_PYTHON` used for Pex CLI
invocations inside a docker environment is an absolute path.

I also did some drive-by improvements of the test code.
@thejcannon thejcannon mentioned this issue Dec 30, 2023
Merged
thejcannon added a commit that referenced this issue Dec 30, 2023
@thejcannon
Make docker_environment's Pants-provided-PBS an absolute path (Cherry…
695fd60 
…-pick of #20314) (#20349)

Fixes #20313 by ensuring that the `PEX_PYTHON` used for Pex CLI
invocations inside a docker environment is an absolute path.

I also did some drive-by improvements of the test code.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@thejcannon thejcannon

Labels
bug
Projects
None yet
Milestone
2.18.x
Development

Successfully merging a pull request may close this issue.

Make docker_environment's Pants-provided-PBS an absolute path pantsbuild/pants
1 participant
@thejcannon